Re: [PATCH 11/18] leds: max77650: switch to device_for_each_child_node_scoped()

From: Bartosz Golaszewski
Date: Fri Sep 27 2024 - 02:37:02 EST


On Fri, Sep 27, 2024 at 1:21 AM Javier Carrasco
<javier.carrasco.cruz@xxxxxxxxx> wrote:
>
> Switch to device_for_each_child_node_scoped() to simplify the code by
> removing the need for calls to fwnode_handle_put() in the error paths.
>
> This also prevents possible memory leaks if new error paths are added
> without the required call to fwnode_handle_put().
>
> After switching to the scoped variant, there is no longer need for a
> jump to 'err_node_out', as an immediate return is possible.
>
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@xxxxxxxxx>
> ---

Acked-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>