Re: [PATCH 1/6] property: add fwnode_property_read_string_index()

From: Clément Léger
Date: Mon Mar 21 2022 - 06:35:14 EST


Le Mon, 21 Mar 2022 12:31:58 +0200,
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> a écrit :

> >
> > IMHO, it would indeed be better. However,
> > fwnode_property_match_string() also allocates memory to do the same
> > kind of operation. Would you also like a callback for this one ?
>
> But matching string will need all of them to cover all possible cases.
> So, it doesn't rely on the certain index and needs allocation anyway.

Acked, it makes sense to keep it that way.

--
Clément Léger,
Embedded Linux and Kernel engineer at Bootlin
https://bootlin.com