can: slcan: clean up if/else
authorMarc Kleine-Budde <mkl@pengutronix.de>
Sun, 3 Jul 2022 09:26:58 +0000 (11:26 +0200)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Tue, 19 Jul 2022 18:30:14 +0000 (20:30 +0200)
Remove braces after if() for single statement blocks, also remove else
after return() in if() block.

Link: https://lore.kernel.org/all/20220704125954.1587880-6-mkl@pengutronix.de
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/slcan/slcan-core.c

index 70a6d1b..d9bf75a 100644 (file)
@@ -469,19 +469,19 @@ static void slcan_unesc(struct slcan *sl, unsigned char s)
 {
        if ((s == '\r') || (s == '\a')) { /* CR or BEL ends the pdu */
                if (!test_and_clear_bit(SLF_ERROR, &sl->flags) &&
-                   (sl->rcount > 4))  {
+                   sl->rcount > 4)
                        slc_bump(sl);
-               }
+
                sl->rcount = 0;
        } else {
                if (!test_bit(SLF_ERROR, &sl->flags))  {
                        if (sl->rcount < SLC_MTU)  {
                                sl->rbuff[sl->rcount++] = s;
                                return;
-                       } else {
-                               sl->dev->stats.rx_over_errors++;
-                               set_bit(SLF_ERROR, &sl->flags);
                        }
+
+                       sl->dev->stats.rx_over_errors++;
+                       set_bit(SLF_ERROR, &sl->flags);
                }
        }
 }
@@ -1104,9 +1104,8 @@ static void __exit slcan_exit(void)
                        continue;
 
                sl = netdev_priv(dev);
-               if (sl->tty) {
+               if (sl->tty)
                        netdev_err(dev, "tty discipline still running\n");
-               }
 
                slc_close(dev);
                unregister_candev(dev);