diff options
author | Axel Lin <axel.lin@ingics.com> | 2013-05-19 10:00:47 +0400 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2013-05-20 22:27:30 +0400 |
commit | cfb10898efe1bc1f3eb8d8f37f164d9e2ac8b43a (patch) | |
tree | 4633a990efc5bcd9090bb594acc4a9923a0194cc /drivers/gpio/gpio-viperboard.c | |
parent | 90dae4ebf03063a70d992aad00d5f5a607c31db8 (diff) | |
download | linux-cfb10898efe1bc1f3eb8d8f37f164d9e2ac8b43a.tar.xz |
gpio: Don't override the error code in probe error handling
Otherwise, we return 0 in probe error paths when gpiochip_remove() returns 0.
Also show error message if gpiochip_remove() fails.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Cc: Tomoya MORINAGA <tomoya.rohm@gmail.com>
Cc: Denis Turischev <denis@compulab.co.il>
Cc: Lars Poeschel <poeschel@lemonage.de>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/gpio/gpio-viperboard.c')
-rw-r--r-- | drivers/gpio/gpio-viperboard.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/gpio/gpio-viperboard.c b/drivers/gpio/gpio-viperboard.c index 095ab14cea4d..5ac2919197fe 100644 --- a/drivers/gpio/gpio-viperboard.c +++ b/drivers/gpio/gpio-viperboard.c @@ -446,7 +446,8 @@ static int vprbrd_gpio_probe(struct platform_device *pdev) return ret; err_gpiob: - ret = gpiochip_remove(&vb_gpio->gpioa); + if (gpiochip_remove(&vb_gpio->gpioa)) + dev_err(&pdev->dev, "%s gpiochip_remove failed\n", __func__); err_gpioa: return ret; |