diff mbox

[U-Boot,V2] fdt: introduce fdtdec_get_child_count

Message ID 1454304675-3768-1-git-send-email-van.freenix@gmail.com
State Accepted
Commit 1889a7e275c82cfa120004595a652e834c3723e1
Delegated to: Anatolij Gustschin
Headers show

Commit Message

Peng Fan Feb. 1, 2016, 5:31 a.m. UTC
Introduce fdtdec_get_child_count for get the number of subnodes
of one parent node.

Signed-off-by: Peng Fan <van.freenix@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
---

V2:
 correct spell error.
 renamed to fdtdec_get_child_count, move code to lib/fdtdec.c and add
 prototype in include/fdtdec.h

 include/fdtdec.h |  9 +++++++++
 lib/fdtdec.c     | 11 +++++++++++
 2 files changed, 20 insertions(+)

Comments

Simon Glass Feb. 2, 2016, 12:05 a.m. UTC | #1
On 31 January 2016 at 22:31, Peng Fan <van.freenix@gmail.com> wrote:
> Introduce fdtdec_get_child_count for get the number of subnodes
> of one parent node.
>
> Signed-off-by: Peng Fan <van.freenix@gmail.com>
> Cc: Simon Glass <sjg@chromium.org>
> ---
>
> V2:
>  correct spell error.
>  renamed to fdtdec_get_child_count, move code to lib/fdtdec.c and add
>  prototype in include/fdtdec.h
>
>  include/fdtdec.h |  9 +++++++++
>  lib/fdtdec.c     | 11 +++++++++++
>  2 files changed, 20 insertions(+)

Acked-by: Simon Glass <sjg@chromium.org>
Anatolij Gustschin Feb. 6, 2016, 1:49 p.m. UTC | #2
On Mon,  1 Feb 2016 13:31:15 +0800
Peng Fan van.freenix@gmail.com wrote:

> Introduce fdtdec_get_child_count for get the number of subnodes
> of one parent node.
> 
> Signed-off-by: Peng Fan <van.freenix@gmail.com>
> Cc: Simon Glass <sjg@chromium.org>
> ---
> 
> V2:
>  correct spell error.
>  renamed to fdtdec_get_child_count, move code to lib/fdtdec.c and add
>  prototype in include/fdtdec.h
> 
>  include/fdtdec.h |  9 +++++++++
>  lib/fdtdec.c     | 11 +++++++++++
>  2 files changed, 20 insertions(+)

applied to u-boot-staging, thanks!

--
Anatolij
diff mbox

Patch

diff --git a/include/fdtdec.h b/include/fdtdec.h
index dd82916..d1c29a8 100644
--- a/include/fdtdec.h
+++ b/include/fdtdec.h
@@ -722,6 +722,15 @@  const u32 *fdtdec_locate_array(const void *blob, int node,
  */
 int fdtdec_get_bool(const void *blob, int node, const char *prop_name);
 
+/*
+ * Count child nodes of one parent node.
+ *
+ * @param blob	FDT blob
+ * @param node	parent node
+ * @return number of child node; 0 if there is not child node
+ */
+int fdtdec_get_child_count(const void *blob, int node);
+
 /**
  * Look in the FDT for a config item with the given name and return its value
  * as a 32-bit integer. The property must have at least 4 bytes of data. The
diff --git a/lib/fdtdec.c b/lib/fdtdec.c
index 0eb56af..1b1ca02 100644
--- a/lib/fdtdec.c
+++ b/lib/fdtdec.c
@@ -831,6 +831,17 @@  int fdtdec_parse_phandle_with_args(const void *blob, int src_node,
 	return rc;
 }
 
+int fdtdec_get_child_count(const void *blob, int node)
+{
+	int subnode;
+	int num = 0;
+
+	fdt_for_each_subnode(blob, subnode, node)
+		num++;
+
+	return num;
+}
+
 int fdtdec_get_byte_array(const void *blob, int node, const char *prop_name,
 		u8 *array, int count)
 {