Hi Joe On 12/09/2017 08:28 PM, Joe Hershberger wrote: > On Mon, Dec 4, 2017 at 2:05 AM, Patrice CHOTARD <patrice.chot...@st.com> > wrote: >> Hi Simon >> >> On 12/02/2017 04:28 AM, Simon Glass wrote: >>> Hi Patrice, >>> >>> On 29 November 2017 at 01:06, <patrice.chot...@st.com> wrote: >>>> From: Patrice Chotard <patrice.chot...@st.com> >>>> >>>> Add missing dev_count_phandle_with_args() to avoid >>>> compilation issue. >>>> >>>> Signed-off-by: Patrice Chotard <patrice.chot...@st.com> >>>> --- >>>> drivers/core/read.c | 7 +++++++ >>>> 1 file changed, 7 insertions(+) >>>> >>>> diff --git a/drivers/core/read.c b/drivers/core/read.c >>>> index 5d440ce..f346cc1 100644 >>>> --- a/drivers/core/read.c >>>> +++ b/drivers/core/read.c >>>> @@ -103,6 +103,13 @@ int dev_read_phandle_with_args(struct udevice *dev, >>>> const char *list_name, >>>> out_args); >>>> } >>>> >>>> +int dev_count_phandle_with_args(struct udevice *dev, const char >>>> *list_name, >>>> + const char *cells_name) >>> >>> Is there a declaration of this in the header file? >> >> yes >> > > It seems that the function is only defined if > CONFIG_DM_DEV_READ_INLINE is defined. Otherwise, there is just a > prototype, but no implementation in include/dm/read.h.
The implementation is located in include/dm/read.h line 499. For information, it's needed to not break compilation for a few aarch64 platforms (puma-rk3399 and lion-rk3368). Patrice > > Reviewed-by: Joe Hershberger <joe.hershber...@ni.com> > >> >>> >>>> +{ >>>> + return ofnode_count_phandle_with_args(dev_ofnode(dev), list_name, >>>> + cells_name); >>>> +} >>>> + >>>> int dev_read_addr_cells(struct udevice *dev) >>>> { >>>> return ofnode_read_addr_cells(dev_ofnode(dev)); >>>> -- >>>> 1.9.1 >>>> >>> >>> Regards, >>> Simon >>> >> _______________________________________________ >> U-Boot mailing list >> U-Boot@lists.denx.de >> https://lists.denx.de/listinfo/u-boot _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot