Re: [PATCH v5 1/9] iio: temp: ltc2983: Use __free(fwnode_handle) and device_for_each_node_scoped()

[Jonathan Cameron <jic23@kernel.org>]

On Mon, 26 Feb 2024 09:35:49 +0100
Nuno Sá <noname.nuno@gmail.com> wrote:

> On Sat, 2024-02-24 at 12:32 +0000, Jonathan Cameron wrote:
> > From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > 
> > This use of the new cleanup.h scope based freeing infrastructure allows
> > us to exit directly from error conditions and in the good path with
> > the reference obtained from fwnode_find_reference() (which may be an error
> > pointer) automatically released.
> > 
> > Similarly the _scoped() version of device_for_each_child_node()
> > removes the need for the manual calling of fwnode_handl_put() in
> > paths where the code exits the loop early.
> > 
> > Tidy up some unusual indentation in a dev_dbg() whilst here.
> > 
> > Cc: Cosmin Tanislav <cosmin.tanislav@analog.com>
> > Cc: Nuno Sá <nuno.sa@analog.com>
> > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > 
> > ---
> > v5: Add the device_for_each_child_node_scoped() change (Nuno)
> > ---  
> 
> Reviewed-by: Nuno Sa <nuno.sa@analog.com>
I'll pick up this now as whilst I hope someone will check the others, I know
you are building on this and it will make life easier if this is already
queued up.

Thanks,

Jonathan