wimax: usb-fw: mark expected switch fall-through
authorGustavo A. R. Silva <gustavo@embeddedor.com>
Thu, 9 Aug 2018 15:39:44 +0000 (10:39 -0500)
committerDavid S. Miller <davem@davemloft.net>
Sat, 11 Aug 2018 18:29:36 +0000 (11:29 -0700)
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.

Notice that in this particular case, I placed the "fall through"
annotation at the bottom of the case, which is what GCC is expecting
to find.

Addresses-Coverity-ID: 1369529 ("Missing break in switch")
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/wimax/i2400m/usb-fw.c

index 502c346..529ebca 100644 (file)
@@ -130,12 +130,12 @@ retry:
                        dev_err(dev, "BM-CMD: too many stalls in "
                                "URB; resetting device\n");
                        usb_queue_reset_device(i2400mu->usb_iface);
-                       /* fallthrough */
                } else {
                        usb_clear_halt(i2400mu->usb_dev, pipe);
                        msleep(10);     /* give the device some time */
                        goto retry;
                }
+               /* fall through */
        case -EINVAL:                   /* while removing driver */
        case -ENODEV:                   /* dev disconnect ... */
        case -ENOENT:                   /* just ignore it */