diff options
author | Sean Anderson <sean.anderson@seco.com> | 2021-04-20 10:50:54 -0400 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2021-05-04 07:57:18 -0400 |
commit | 430e136dc1398546ad2b1db08692e9edac480cc8 (patch) | |
tree | 9bd127f80bae5166b76c77123e8f2bafe644352f /drivers | |
parent | 8ddaf943589756442bba21e5be645cd47526d82b (diff) | |
download | u-boot-430e136dc1398546ad2b1db08692e9edac480cc8.zip u-boot-430e136dc1398546ad2b1db08692e9edac480cc8.tar.gz u-boot-430e136dc1398546ad2b1db08692e9edac480cc8.tar.bz2 |
dm: gpio: Fix gpio_get_list_count failing with livetree
of_parse_phandle_with_args (called by dev_read_phandle_with_args) does not
support getting the length of a phandle list by using the index -1.
Instead, use dev_count_phandle_with_args which supports exactly this
use-case.
Fixes: 8558217153 ("gpio: Convert to use APIs which support live DT")
Signed-off-by: Sean Anderson <sean.anderson@seco.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/gpio/gpio-uclass.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpio/gpio-uclass.c b/drivers/gpio/gpio-uclass.c index e4e7f58..131099c 100644 --- a/drivers/gpio/gpio-uclass.c +++ b/drivers/gpio/gpio-uclass.c @@ -1215,9 +1215,9 @@ int gpio_get_list_count(struct udevice *dev, const char *list_name) { int ret; - ret = dev_read_phandle_with_args(dev, list_name, "#gpio-cells", 0, -1, - NULL); - if (ret) { + ret = dev_count_phandle_with_args(dev, list_name, "#gpio-cells", + -ENOENT); + if (ret < 0) { debug("%s: Node '%s', property '%s', GPIO count failed: %d\n", __func__, dev->name, list_name, ret); } |