pinctrl: th1520: Fix return value for unknown pin error
authorDrew Fustini <dfustini@tenstorrent.com>
Wed, 9 Oct 2024 01:16:46 +0000 (18:16 -0700)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 9 Oct 2024 12:42:07 +0000 (14:42 +0200)
Fix th1520_pinctrl_dt_node_to_map() to set the return value before
jumping to free_configs when an unknown pin error occurs.

Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
Closes: https://lore.kernel.org/r/202410022336.xyWlV0Tf-lkp@intel.com/
Signed-off-by: Drew Fustini <dfustini@tenstorrent.com>
Link: https://lore.kernel.org/20241008-th1520-pinctrl-fixes-v3-1-5b60db446a49@tenstorrent.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-th1520.c

index 9331f44..265a75a 100644 (file)
@@ -499,6 +499,7 @@ static int th1520_pinctrl_dt_node_to_map(struct pinctrl_dev *pctldev,
                                nmaps = rollback;
                                dev_err(thp->pctl->dev, "%pOFn.%pOFn: unknown pin '%s'\n",
                                        np, child, pinname);
+                               ret = -EINVAL;
                                goto free_configs;
                        }