From: Sascha Hauer <s.hauer@pengutronix.de>
Date: Fri, 9 Aug 2013 12:20:53 +0000 (+0200)
Subject: pinctrl: imx: do not fail when parsing a function fails
X-Git-Tag: upstream/snapshot3+hdmi~4441^2~38
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7ea46e0f9df44ce3dea5a54f7c182f98eeef7bee;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git

pinctrl: imx: do not fail when parsing a function fails

When parsing a function fails this is no reason to make the whole
driver fail. Just continue with the next function.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Acked-by: Shawn Guo <shawn.guo@linaro.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---

diff --git a/drivers/pinctrl/pinctrl-imx.c b/drivers/pinctrl/pinctrl-imx.c
index b8077d9..81ffb17 100644
--- a/drivers/pinctrl/pinctrl-imx.c
+++ b/drivers/pinctrl/pinctrl-imx.c
@@ -550,7 +550,6 @@ static int imx_pinctrl_probe_dt(struct platform_device *pdev,
 {
 	struct device_node *np = pdev->dev.of_node;
 	struct device_node *child;
-	int ret;
 	u32 nfuncs = 0;
 	u32 i = 0;
 
@@ -577,13 +576,8 @@ static int imx_pinctrl_probe_dt(struct platform_device *pdev,
 	if (!info->groups)
 		return -ENOMEM;
 
-	for_each_child_of_node(np, child) {
-		ret = imx_pinctrl_parse_functions(child, info, i++);
-		if (ret) {
-			dev_err(&pdev->dev, "failed to parse function\n");
-			return ret;
-		}
-	}
+	for_each_child_of_node(np, child)
+		imx_pinctrl_parse_functions(child, info, i++);
 
 	return 0;
 }