watchdog: alim1535_wdt: mark expected switch fall-through
authorGustavo A. R. Silva <garsilva@embeddedor.com>
Thu, 2 Nov 2017 19:25:09 +0000 (14:25 -0500)
committerWim Van Sebroeck <wim@iguana.be>
Thu, 28 Dec 2017 19:45:28 +0000 (20:45 +0100)
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.

Notice that in this particular case I replaced "Fall" with a proper
"fall through" comment, which is what GCC is expecting to find.

Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
drivers/watchdog/alim1535_wdt.c

index 3a17fbd..60f0c2e 100644 (file)
@@ -223,8 +223,8 @@ static long ali_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
                if (ali_settimer(new_timeout))
                        return -EINVAL;
                ali_keepalive();
-               /* Fall */
        }
+               /* fall through */
        case WDIOC_GETTIMEOUT:
                return put_user(timeout, p);
        default: