On Wed, Jul 20, 2016 at 04:20:39PM +0200, Maxime Ripard wrote:
> From: Thierry Reding <treding@xxxxxxxxxx>
>
> The fdt_for_each_subnode() iterator macro provided by this patch can be
> used to iterate over a device tree node's subnodes. At each iteration a
> loop variable will be set to the next subnode.
>
> Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
> Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx>
Some nits in the comment..
> ---
> libfdt/libfdt.h | 27 +++++++++++++++++++++++++++
> tests/subnode_iterate.c | 8 ++------
> 2 files changed, 29 insertions(+), 6 deletions(-)
>
> diff --git a/libfdt/libfdt.h b/libfdt/libfdt.h
> index 36222fd4a6f4..70ec39c7104c 100644
> --- a/libfdt/libfdt.h
> +++ b/libfdt/libfdt.h
> @@ -168,6 +168,33 @@ int fdt_first_subnode(const void *fdt, int offset);
> */
> int fdt_next_subnode(const void *fdt, int offset);
>
> +/**
> + * fdt_for_each_subnode - iterate over all subnodes of a parent
Parameter descriptions generally go first.
> + *
> + * This is actually a wrapper around a for loop and would be used like so:
> + *
> + * fdt_for_each_subnode(node, fdt, parent) {
> + * if ((node < 0) && (node != -FDT_ERR_NOT_FOUND)) {
> + * Error handling
> + * }
This error handling needs to go *after* the for block.
> + *
> + * Use node
> + * ...
> + * }
> + *
> + * Note that this is implemented as a macro and @node is used as
> + * iterator in the loop. The parent variable be constant or even a
^ insert "may"
> + * literal.
> + *
> + * @node: child node (int)
Probably worth noting it as (int, lvalue).
> + * @fdt: FDT blob (const void *)
> + * @parent: parent node (int)
> + */
> +#define fdt_for_each_subnode(node, fdt, parent) \
> + for (node = fdt_first_subnode(fdt, parent); \
> + node >= 0; \
> + node = fdt_next_subnode(fdt, node))
> +
> /**********************************************************************/
> /* General functions */
> /**********************************************************************/
> diff --git a/tests/subnode_iterate.c b/tests/subnode_iterate.c
> index b9f379d5963c..0fb5c901ebd7 100644
> --- a/tests/subnode_iterate.c
> +++ b/tests/subnode_iterate.c
> @@ -48,9 +48,7 @@ static void test_node(void *fdt, int parent_offset)
> subnodes = cpu_to_fdt32(*prop);
>
> count = 0;
> - for (offset = fdt_first_subnode(fdt, parent_offset);
> - offset >= 0;
> - offset = fdt_next_subnode(fdt, offset))
> + fdt_for_each_subnode(offset, fdt, parent_offset)
> count++;
>
> if (count != subnodes) {
> @@ -65,9 +63,7 @@ static void check_fdt_next_subnode(void *fdt)
> int offset;
> int count = 0;
>
> - for (offset = fdt_first_subnode(fdt, 0);
> - offset >= 0;
> - offset = fdt_next_subnode(fdt, offset)) {
> + fdt_for_each_subnode(offset, fdt, 0) {
> test_node(fdt, offset);
> count++;
> }
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
Attachment:
signature.asc
Description: PGP signature
