diff options
| author | Alexander Shiyan <shc_work@mail.ru> | 2014-03-22 04:32:15 -0400 |
|---|---|---|
| committer | Linus Walleij <linus.walleij@linaro.org> | 2014-03-27 05:06:34 -0400 |
| commit | fd0f885b79d1eac9b8aed7e04fe641b8cfd4ebf0 (patch) | |
| tree | 345cb35a2e88832f23758aab9173628a8e8a8430 | |
| parent | a54aef5072325ad3e5bd3e7c2858ae208fb4454d (diff) | |
gpio: twl4030: Remove redundant assignment
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>
| -rw-r--r-- | drivers/gpio/gpio-twl4030.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpio/gpio-twl4030.c b/drivers/gpio/gpio-twl4030.c index 8b88ca2eda9c..3ebb1a5ff22e 100644 --- a/drivers/gpio/gpio-twl4030.c +++ b/drivers/gpio/gpio-twl4030.c | |||
| @@ -139,7 +139,6 @@ static u8 cached_leden; | |||
| 139 | static void twl4030_led_set_value(int led, int value) | 139 | static void twl4030_led_set_value(int led, int value) |
| 140 | { | 140 | { |
| 141 | u8 mask = LEDEN_LEDAON | LEDEN_LEDAPWM; | 141 | u8 mask = LEDEN_LEDAON | LEDEN_LEDAPWM; |
| 142 | int status; | ||
| 143 | 142 | ||
| 144 | if (led) | 143 | if (led) |
| 145 | mask <<= 1; | 144 | mask <<= 1; |
| @@ -148,8 +147,9 @@ static void twl4030_led_set_value(int led, int value) | |||
| 148 | cached_leden &= ~mask; | 147 | cached_leden &= ~mask; |
| 149 | else | 148 | else |
| 150 | cached_leden |= mask; | 149 | cached_leden |= mask; |
| 151 | status = twl_i2c_write_u8(TWL4030_MODULE_LED, cached_leden, | 150 | |
| 152 | TWL4030_LED_LEDEN_REG); | 151 | WARN_ON_ONCE(twl_i2c_write_u8(TWL4030_MODULE_LED, cached_leden, |
| 152 | TWL4030_LED_LEDEN_REG)); | ||
| 153 | } | 153 | } |
| 154 | 154 | ||
| 155 | static int twl4030_set_gpio_direction(int gpio, int is_input) | 155 | static int twl4030_set_gpio_direction(int gpio, int is_input) |
