diff options
| author | Axel Lin <axel.lin@ingics.com> | 2013-05-19 13:58:37 +0800 |
|---|---|---|
| committer | Linus Walleij <linus.walleij@linaro.org> | 2013-05-27 15:37:08 +0200 |
| commit | 97fc463769f1564e8eda2e2f70d3b6e92a25ff16 (patch) | |
| tree | f1ee749b141b7055f9995afa506c83539bc6382e /drivers/pinctrl/pinctrl-coh901.c | |
| parent | b134dc3feaa5136b376c7d2658bbe156bea19e63 (diff) | |
| download | linux-97fc463769f1564e8eda2e2f70d3b6e92a25ff16.tar.gz linux-97fc463769f1564e8eda2e2f70d3b6e92a25ff16.tar.bz2 linux-97fc463769f1564e8eda2e2f70d3b6e92a25ff16.zip | |
pinctrl: 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>
Acked-by: Tony Prisk <linux@prisktech.co.nz>
Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pinctrl/pinctrl-coh901.c')
| -rw-r--r-- | drivers/pinctrl/pinctrl-coh901.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/pinctrl/pinctrl-coh901.c b/drivers/pinctrl/pinctrl-coh901.c index a67af419f531..d6b41747d687 100644 --- a/drivers/pinctrl/pinctrl-coh901.c +++ b/drivers/pinctrl/pinctrl-coh901.c @@ -830,7 +830,8 @@ static int __init u300_gpio_probe(struct platform_device *pdev) return 0; err_no_range: - err = gpiochip_remove(&gpio->chip); + if (gpiochip_remove(&gpio->chip)) + dev_err(&pdev->dev, "failed to remove gpio chip\n"); err_no_chip: err_no_domain: err_no_port: |