watchdog: sc1200: Mark expected switch fall-through
authorGustavo A. R. Silva <gustavo@embeddedor.com>
Thu, 14 Feb 2019 20:20:08 +0000 (14:20 -0600)
committerWim Van Sebroeck <wim@linux-watchdog.org>
Sat, 2 Mar 2019 14:28:21 +0000 (15:28 +0100)
commit51c802f57a0c4ce17f472d8fb718ecd94e23c5d3
tree873aab305471e7b271444fac21b926f738003d2d
parent8baee57218f6dcd976bb5cb13d8262fbbd153282
watchdog: sc1200: Mark expected switch fall-through

In preparation to enabling -Wimplicit-fallthrough, mark switch
cases where we are expecting to fall through.

This patch fixes the following warning:

drivers/watchdog/sc1200wdt.c: In function ‘sc1200wdt_ioctl’:
drivers/watchdog/sc1200wdt.c:241:3: warning: this statement may fall through [-Wimplicit-fallthrough=]
   sc1200wdt_write_data(WDTO, timeout);
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/watchdog/sc1200wdt.c:244:2: note: here
  case WDIOC_GETTIMEOUT:
  ^~~~

Warning level 3 was used: -Wimplicit-fallthrough=3

Notice that, in this particular case, the code comment is modified
in accordance with what GCC is expecting to find.

This patch is part of the ongoing efforts to enable
-Wimplicit-fallthrough.

Signed-off-by: Gustavo A. R. Silva <gustavo@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@linux-watchdog.org>
drivers/watchdog/sc1200wdt.c