gpio: twl4030: Remove redundant assignment
authorAlexander Shiyan <shc_work@mail.ru>
Sat, 22 Mar 2014 08:32:15 +0000 (12:32 +0400)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 27 Mar 2014 09:06:34 +0000 (10:06 +0100)
Variable "status" is never used, so remove it and add warning if
any error happen.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-twl4030.c

index 8b88ca2..3ebb1a5 100644 (file)
@@ -139,7 +139,6 @@ static u8 cached_leden;
 static void twl4030_led_set_value(int led, int value)
 {
        u8 mask = LEDEN_LEDAON | LEDEN_LEDAPWM;
-       int status;
 
        if (led)
                mask <<= 1;
@@ -148,8 +147,9 @@ static void twl4030_led_set_value(int led, int value)
                cached_leden &= ~mask;
        else
                cached_leden |= mask;
-       status = twl_i2c_write_u8(TWL4030_MODULE_LED, cached_leden,
-                                 TWL4030_LED_LEDEN_REG);
+
+       WARN_ON_ONCE(twl_i2c_write_u8(TWL4030_MODULE_LED, cached_leden,
+                                     TWL4030_LED_LEDEN_REG));
 }
 
 static int twl4030_set_gpio_direction(int gpio, int is_input)