[U-Boot] [PATCH 3/6] libfdt: add fdtdec_lookup_phandle_index()

[André Przywara]

On 07/07/17 04:58, Simon Glass wrote:

Hi Simon,

> On 2 July 2017 at 18:59, Andre Przywara <andre.przywara at 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.przywara at 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.