gpio: sprd: Add missing break in switch statement
authorGustavo A. R. Silva <gustavo@embeddedor.com>
Mon, 11 Feb 2019 18:50:01 +0000 (12:50 -0600)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 13 Feb 2019 09:39:50 +0000 (10:39 +0100)
Fix the following warning by adding a missing break:

drivers/gpio/gpio-eic-sprd.c: In function ‘sprd_eic_irq_set_type’:
drivers/gpio/gpio-eic-sprd.c:403:3: warning: this statement may fall through [-Wimplicit-fallthrough=]
   switch (flow_type) {
   ^~~~~~
drivers/gpio/gpio-eic-sprd.c:435:2: note: here
  default:
  ^~~~~~~

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

Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Reviewed-by: Baolin Wang <baolin.wang@linaro.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-eic-sprd.c

index e0d6a0a..64cc87d 100644 (file)
@@ -420,6 +420,7 @@ static int sprd_eic_irq_set_type(struct irq_data *data, unsigned int flow_type)
                default:
                        return -ENOTSUPP;
                }
+               break;
        default:
                dev_err(chip->parent, "Unsupported EIC type.\n");
                return -ENOTSUPP;