diff options
author | Joerg Roedel <jroedel@suse.de> | 2016-04-04 15:49:19 (GMT) |
---|---|---|
committer | Rob Herring <robh@kernel.org> | 2016-04-19 22:25:14 (GMT) |
commit | 2021bd01ffccf2728a591070008d7de5bc41306a (patch) | |
tree | 927fe82263ab52e239fdebfef9f8f47ead760442 /drivers/of | |
parent | cd209b412c8a5d632b51af1e45576f0d00b8105f (diff) | |
download | linux-2021bd01ffccf2728a591070008d7de5bc41306a.tar.xz |
of: Remove counting special case from __of_parse_phandle_with_args()
The index = -1 case in __of_parse_phandle_with_args() is
used to just return the number of phandles. That special
case needs extra handling, so move it to the place where it
is needed: of_count_phandle_with_args().
This allows to further simplify __of_parse_phandle_with_args()
later on.
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'drivers/of')
-rw-r--r-- | drivers/of/base.c | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/drivers/of/base.c b/drivers/of/base.c index 69286ec..fcff2b6 100644 --- a/drivers/of/base.c +++ b/drivers/of/base.c @@ -1584,10 +1584,7 @@ static int __of_parse_phandle_with_args(const struct device_node *np, * Unlock node before returning result; will be one of: * -ENOENT : index is for empty phandle * -EINVAL : parsing error on data - * [1..n] : Number of phandle (count mode; when index = -1) */ - if (rc == -ENOENT && index < 0) - rc = cur_index; err: if (it.node) @@ -1723,8 +1720,20 @@ EXPORT_SYMBOL(of_parse_phandle_with_fixed_args); int of_count_phandle_with_args(const struct device_node *np, const char *list_name, const char *cells_name) { - return __of_parse_phandle_with_args(np, list_name, cells_name, 0, -1, - NULL); + struct of_phandle_iterator it; + int rc, cur_index = 0; + + rc = of_phandle_iterator_init(&it, np, list_name, cells_name, 0); + if (rc) + return rc; + + while ((rc = of_phandle_iterator_next(&it)) == 0) + cur_index += 1; + + if (rc != -ENOENT) + return rc; + + return cur_index; } EXPORT_SYMBOL(of_count_phandle_with_args); |