On 07/07/17 04:58, Simon Glass wrote: Hi Simon,
> On 2 July 2017 at 18:59, Andre Przywara <andre.przyw...@arm.com> wrote: >> In some bindings a property points to multiple nodes, using a list of >> phandles. A prominent example are UART pinctrl nodes, which use one node >> to contain the RX/TX pins and another node to describe the lines used >> for the hardware handshake. >> The current fdtdec_lookup_phandle() helper function to chase a phandle >> is quite convienent, but can only lookup the first of those handles. >> >> Introduce an extra function fdtdec_lookup_phandle_index() to take an >> index parameter and implement fdtdec_lookup_phandle() as a special case >> of that. >> >> Signed-off-by: Andre Przywara <andre.przyw...@arm.com> >> --- >> include/fdtdec.h | 12 ++++++++++++ >> lib/fdtdec.c | 16 ++++++++++++---- >> 2 files changed, 24 insertions(+), 4 deletions(-) > > Can you please: > > - Add a dev_read... version of this API > - Also ofnode_... Mmmh, I am not sure I follow here. I find that both dev_read_phandle_with_args() and ofnode_parse_phandle_with_args() take an index parameter already and from briefly looking at the code seem to do the right thing already. So I guess that's not what you meant? What am I missing? Cheers, Andre. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
