| Message ID | 20200326100916.22532-7-kever.yang@rock-chips.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v3,1/8] tool: Move ALIGN_MASK to header as common MACRO | expand |
Hi Kever, On Thu, 26 Mar 2020 at 04:09, Kever Yang <kever.yang at rock-chips.com> wrote: > > The tool need to use fdtdec_get_child_count(), make it available for > HOST_CC. > > Signed-off-by: Kever Yang <kever.yang at rock-chips.com> > --- > > Changes in v3: None > Changes in v2: None > > include/fdt_support.h | 1 + > lib/fdtdec.c | 11 ----------- > lib/fdtdec_common.c | 11 +++++++++++ > 3 files changed, 12 insertions(+), 11 deletions(-) > > diff --git a/include/fdt_support.h b/include/fdt_support.h > index 3f4bc643d4..9b88fd1723 100644 > --- a/include/fdt_support.h > +++ b/include/fdt_support.h > @@ -329,6 +329,7 @@ int fdt_get_cells_len(const void *blob, char *nr_cells_name); > #ifdef USE_HOSTCC > int fdtdec_get_int(const void *blob, int node, const char *prop_name, > int default_val); > +int fdtdec_get_child_count(const void *blob, int node); Please can you add a function comment for this? > #endif > #ifdef CONFIG_FMAN_ENET > int fdt_update_ethernet_dt(void *blob); > diff --git a/lib/fdtdec.c b/lib/fdtdec.c > index 61af3472e6..c55c77a31c 100644 > --- a/lib/fdtdec.c > +++ b/lib/fdtdec.c > @@ -807,17 +807,6 @@ 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(subnode, blob, node) > - num++; > - > - return num; > -} > - > int fdtdec_get_byte_array(const void *blob, int node, const char *prop_name, > u8 *array, int count) > { > diff --git a/lib/fdtdec_common.c b/lib/fdtdec_common.c > index 088e9e9063..5775992ef3 100644 > --- a/lib/fdtdec_common.c > +++ b/lib/fdtdec_common.c > @@ -53,3 +53,14 @@ unsigned int fdtdec_get_uint(const void *blob, int node, const char *prop_name, > debug("(not found)\n"); > return default_val; > } > + > +int fdtdec_get_child_count(const void *blob, int node) > +{ > + int subnode; > + int num = 0; > + > + fdt_for_each_subnode(subnode, blob, node) > + num++; > + > + return num; > +} > -- > 2.17.1 > Regards, SImon
diff --git a/include/fdt_support.h b/include/fdt_support.h index 3f4bc643d4..9b88fd1723 100644 --- a/include/fdt_support.h +++ b/include/fdt_support.h @@ -329,6 +329,7 @@ int fdt_get_cells_len(const void *blob, char *nr_cells_name); #ifdef USE_HOSTCC int fdtdec_get_int(const void *blob, int node, const char *prop_name, int default_val); +int fdtdec_get_child_count(const void *blob, int node); #endif #ifdef CONFIG_FMAN_ENET int fdt_update_ethernet_dt(void *blob); diff --git a/lib/fdtdec.c b/lib/fdtdec.c index 61af3472e6..c55c77a31c 100644 --- a/lib/fdtdec.c +++ b/lib/fdtdec.c @@ -807,17 +807,6 @@ 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(subnode, blob, node) - num++; - - return num; -} - int fdtdec_get_byte_array(const void *blob, int node, const char *prop_name, u8 *array, int count) { diff --git a/lib/fdtdec_common.c b/lib/fdtdec_common.c index 088e9e9063..5775992ef3 100644 --- a/lib/fdtdec_common.c +++ b/lib/fdtdec_common.c @@ -53,3 +53,14 @@ unsigned int fdtdec_get_uint(const void *blob, int node, const char *prop_name, debug("(not found)\n"); return default_val; } + +int fdtdec_get_child_count(const void *blob, int node) +{ + int subnode; + int num = 0; + + fdt_for_each_subnode(subnode, blob, node) + num++; + + return num; +}
The tool need to use fdtdec_get_child_count(), make it available for HOST_CC. Signed-off-by: Kever Yang <kever.yang at rock-chips.com> --- Changes in v3: None Changes in v2: None include/fdt_support.h | 1 + lib/fdtdec.c | 11 ----------- lib/fdtdec_common.c | 11 +++++++++++ 3 files changed, 12 insertions(+), 11 deletions(-)