diff options
author | abdoulaye berthe <berthe.ab@gmail.com> | 2014-07-12 16:30:12 -0400 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2014-07-22 10:39:26 -0400 |
commit | 9f5132ae82fdbb047cc187bf689a81c8cc0de7fa (patch) | |
tree | 3d5ce5c1a1ff25fdbec52416ad21eb1ad14ca25f /drivers/gpio/gpio-mc33880.c | |
parent | 38ed0187d2da7f219f9f84d8921b5c95b266b34b (diff) |
gpio: remove all usage of gpio_remove retval in driver/gpio
Signed-off-by: abdoulaye berthe <berthe.ab@gmail.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/gpio/gpio-mc33880.c')
-rw-r--r-- | drivers/gpio/gpio-mc33880.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/drivers/gpio/gpio-mc33880.c b/drivers/gpio/gpio-mc33880.c index 553a80a5eaf3..4e3e160e5db2 100644 --- a/drivers/gpio/gpio-mc33880.c +++ b/drivers/gpio/gpio-mc33880.c | |||
@@ -149,20 +149,15 @@ exit_destroy: | |||
149 | static int mc33880_remove(struct spi_device *spi) | 149 | static int mc33880_remove(struct spi_device *spi) |
150 | { | 150 | { |
151 | struct mc33880 *mc; | 151 | struct mc33880 *mc; |
152 | int ret; | ||
153 | 152 | ||
154 | mc = spi_get_drvdata(spi); | 153 | mc = spi_get_drvdata(spi); |
155 | if (mc == NULL) | 154 | if (mc == NULL) |
156 | return -ENODEV; | 155 | return -ENODEV; |
157 | 156 | ||
158 | ret = gpiochip_remove(&mc->chip); | 157 | gpiochip_remove(&mc->chip); |
159 | if (!ret) | 158 | mutex_destroy(&mc->lock); |
160 | mutex_destroy(&mc->lock); | ||
161 | else | ||
162 | dev_err(&spi->dev, "Failed to remove the GPIO controller: %d\n", | ||
163 | ret); | ||
164 | 159 | ||
165 | return ret; | 160 | return 0; |
166 | } | 161 | } |
167 | 162 | ||
168 | static struct spi_driver mc33880_driver = { | 163 | static struct spi_driver mc33880_driver = { |