diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-05-07 12:06:19 (GMT) |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2013-05-14 12:04:14 (GMT) |
commit | 18442e65d424dc84a4a4a3b635eb1a52de3cb6b4 (patch) | |
tree | 15d5cc6c17223c64e2d44da452e4af082c24c8fa /drivers | |
parent | 63ad9cbff5896dadfee510f631ffcb0176984722 (diff) | |
download | linux-fsl-qoriq-18442e65d424dc84a4a4a3b635eb1a52de3cb6b4.tar.xz |
pinctrl: single: fix error return code in pcs_parse_one_pinctrl_entry()
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Introduced by commit 9dddb4df90d136429b6d6ddefceb49a9b93f6cd1
(pinctrl: single: support generic pinconf)
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Acked-by: Haojian Zhuang <haojian.zhuang@linaro.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/pinctrl/pinctrl-single.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c index 5f2d2bf..b9fa046 100644 --- a/drivers/pinctrl/pinctrl-single.c +++ b/drivers/pinctrl/pinctrl-single.c @@ -1166,7 +1166,8 @@ static int pcs_parse_one_pinctrl_entry(struct pcs_device *pcs, (*map)->data.mux.function = np->name; if (pcs->is_pinconf) { - if (pcs_parse_pinconf(pcs, np, function, map)) + res = pcs_parse_pinconf(pcs, np, function, map); + if (res) goto free_pingroups; *num_maps = 2; } else { |