[PATCH v2 1/2] core: fdtaddr: add devfdt_get_addr_size_index_ptr function

[Michael Nazzareno Trimarchi]

Hi Johan

On Sat, Feb 25, 2023 at 12:47 PM Michael Nazzareno Trimarchi
<michael at amarulasolutions.com> wrote:
>
> Hi
>
> On Sat, Feb 25, 2023 at 12:43 PM Johan Jonker <jbx6244 at gmail.com> wrote:
> >
> > Add devfdt_get_addr_size_index_ptr function with the same
> > functionality as devfdt_get_addr_size_index, but instead
> > a return pointer is given.
> >
> > Suggested-by: Michael Nazzareno Trimarchi <michael at amarulasolutions.com>
> > Signed-off-by: Johan Jonker <jbx6244 at gmail.com>
> > ---
> >  drivers/core/fdtaddr.c |  8 ++++++++
> >  include/dm/fdtaddr.h   | 15 +++++++++++++++
> >  2 files changed, 23 insertions(+)
> >
> > diff --git a/drivers/core/fdtaddr.c b/drivers/core/fdtaddr.c
> > index 91bcd1a2..84bb8d8b 100644

[snip]

> > +/**
> > + * devfdt_get_addr_size_index_ptr() - Return indexed pointer to the address of the
> > + *                                    reg property of a device
> > + *
> > + * @dev: Pointer to a device
> > + * @index: the 'reg' property can hold a list of <addr, size> pairs
> > + *        and @index is used to select which one is required
> > + * @size: Pointer to size varible - this function returns the size
> > + *        specified in the 'reg' property here
> > + *
> > + * Return: Pointer to addr, or NULL if there is no such property
> > + */
> > +void *devfdt_get_addr_size_index_ptr(const struct udevice *dev, int index,
> > +                                    fdt_size_t *size);
> > +
> >  /**
> >   * devfdt_get_addr_name() - Get the reg property of a device, indexed by name
> >   *
> > --
> > 2.20.1
> >

git grep "== FDT_ADDR_T_NONE"

I think using both function some of use cases can be rewritten now

MIchael

-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael at amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info at amarulasolutions.com
www.amarulasolutions.com