The else condition is redundant after a return. Remove these redundant else 
conditions.

Signed-off-by: Fred Chou <fred.chou...@gmail.com>
---
 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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to