The else condition is redundant after a return. Remove these redundant else conditions. Signed-off-by: Fred Chou <fred.chou.nd@xxxxxxxxx> --- drivers/staging/vme/devices/vme_pio2_gpio.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/vme/devices/vme_pio2_gpio.c b/drivers/staging/vme/devices/vme_pio2_gpio.c index f00af07..d8a118d 100644 --- a/drivers/staging/vme/devices/vme_pio2_gpio.c +++ b/drivers/staging/vme/devices/vme_pio2_gpio.c @@ -58,14 +58,14 @@ static int pio2_gpio_get(struct gpio_chip *chip, unsigned int offset) if (reg & PIO2_CHANNEL_BIT[offset]) { if (card->bank[PIO2_CHANNEL_BANK[offset]].config != BOTH) return 0; - else - return 1; - } else { - if (card->bank[PIO2_CHANNEL_BANK[offset]].config != BOTH) - return 1; - else - return 0; + + return 1; } + + if (card->bank[PIO2_CHANNEL_BANK[offset]].config != BOTH) + return 1; + + return 0; } static void pio2_gpio_set(struct gpio_chip *chip, unsigned int offset, -- 1.7.9.5 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel