Return the value obtained from of_property_count_strings() instead of -EINVAL.
Silences the following smatch warning: drivers/pinctrl/pinctrl-samsung.c:529 samsung_pinctrl_parse_dt_pins() info: why not propagate '*npins' from of_property_count_strings() instead of -22? Cc: Thomas Abraham <thomas...@samsung.com> Signed-off-by: Sachin Kamat <sachin.ka...@linaro.org> --- drivers/pinctrl/pinctrl-samsung.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/pinctrl/pinctrl-samsung.c b/drivers/pinctrl/pinctrl-samsung.c index 8a24223..824fda9 100644 --- a/drivers/pinctrl/pinctrl-samsung.c +++ b/drivers/pinctrl/pinctrl-samsung.c @@ -526,7 +526,7 @@ static int __init samsung_pinctrl_parse_dt_pins(struct platform_device *pdev, *npins = of_property_count_strings(cfg_np, "samsung,pins"); if (*npins < 0) { dev_err(dev, "invalid pin list in %s node", cfg_np->name); - return -EINVAL; + return *npins; } *pin_list = devm_kzalloc(dev, *npins * sizeof(**pin_list), GFP_KERNEL); -- 1.7.4.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/