Re: [PATCH] device property: introduce fwnode_for_each_child_node_scoped()

From: Andy Shevchenko
Date: Thu May 02 2024 - 14:04:49 EST


On Thu, May 02, 2024 at 09:03:09PM +0300, Andy Shevchenko wrote:
> On Thu, May 02, 2024 at 07:58:26PM +0200, Javier Carrasco wrote:
> > On 5/2/24 17:34, Andy Shevchenko wrote:
> > > On Thu, May 02, 2024 at 12:55:40PM +0200, Javier Carrasco wrote:

..

> > >> This macro has been tested with a patch series that has not been
> > >> applied yet and is under discussion in input [1], which makes use of the
> > >> non-scoped version of the loop.
> > >
> > > So, why should we apply a dead code?
> >
> > I will add this patch to the series I mentioned, so there is a first use
> > case.
>
> Sounds like a good plan.

Ah, note that IIO has already some patches against device property APIs. Maybe
it's already done by Jonathan.
Cc'ed to him.

> > Even if the _available variant is preferred, the other one is more
> > widely used, and having a scoped version will allow for safer code.

--
With Best Regards,
Andy Shevchenko