mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-25 10:19:47 +00:00
gpio: of: Don't return 0 on dt_gpio_count()
It's unusual to have error checking like (ret <= 0) in cases when counting GPIO resources. In case when it's mandatory we propagate the error (-ENOENT), otherwise we don't use the result. This makes consistent behaviour across all possible variants called in gpiod_count(). Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
This commit is contained in:
parent
4ed55016d7
commit
4033d4a4f5
1 changed files with 2 additions and 2 deletions
|
@ -3122,10 +3122,10 @@ static int dt_gpio_count(struct device *dev, const char *con_id)
|
||||||
gpio_suffixes[i]);
|
gpio_suffixes[i]);
|
||||||
|
|
||||||
ret = of_gpio_named_count(dev->of_node, propname);
|
ret = of_gpio_named_count(dev->of_node, propname);
|
||||||
if (ret >= 0)
|
if (ret > 0)
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
return ret;
|
return ret ? ret : -ENOENT;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int platform_gpio_count(struct device *dev, const char *con_id)
|
static int platform_gpio_count(struct device *dev, const char *con_id)
|
||||||
|
|
Loading…
Reference in a new issue