summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGustavo A. R. Silva <gustavo@embeddedor.com>2019-07-29 07:13:20 -0400
committerDavid S. Miller <davem@davemloft.net>2019-07-29 12:38:34 -0400
commit56f37a3fa2190732957882e10ec20adf83c6cb93 (patch)
tree10e3f4a9fc435f346af97d512ab73a93796cce8f
parentf3eb2c334a6d3f5e704ecd0b150d5cfe6e3aca40 (diff)
arcnet: com90io: Mark expected switch fall-throughs
Mark switch cases where we are expecting to fall through. This patch fixes the following warnings (Building: powerpc allyesconfig): drivers/net/arcnet/com90io.c: In function 'com90io_setup': include/linux/printk.h:304:2: warning: this statement may fall through [-Wimplicit-fallthrough=] printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__) ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/net/arcnet/com90io.c:365:3: note: in expansion of macro 'pr_err' pr_err("Too many arguments\n"); ^~~~~~ drivers/net/arcnet/com90io.c:366:2: note: here case 2: /* IRQ */ ^~~~ drivers/net/arcnet/com90io.c:367:7: warning: this statement may fall through [-Wimplicit-fallthrough=] irq = ints[2]; ~~~~^~~~~~~~~ drivers/net/arcnet/com90io.c:368:2: note: here case 1: /* IO address */ ^~~~ Reported-by: Stephen Rothwell <sfr@canb.auug.org.au> Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com> Reviewed-by: Kees Cook <keescook@chromium.org> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--drivers/net/arcnet/com90io.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/arcnet/com90io.c b/drivers/net/arcnet/com90io.c
index 2c546013a980..186bbf87bc84 100644
--- a/drivers/net/arcnet/com90io.c
+++ b/drivers/net/arcnet/com90io.c
@@ -363,8 +363,10 @@ static int __init com90io_setup(char *s)
363 switch (ints[0]) { 363 switch (ints[0]) {
364 default: /* ERROR */ 364 default: /* ERROR */
365 pr_err("Too many arguments\n"); 365 pr_err("Too many arguments\n");
366 /* Fall through */
366 case 2: /* IRQ */ 367 case 2: /* IRQ */
367 irq = ints[2]; 368 irq = ints[2];
369 /* Fall through */
368 case 1: /* IO address */ 370 case 1: /* IO address */
369 io = ints[1]; 371 io = ints[1];
370 } 372 }