Re: (subset) [PATCH 0/3] gpio: switch to device_for_each_chilld_node_scoped()
From: Bartosz Golaszewski
Date: Tue Oct 01 2024 - 14:54:37 EST
From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>
On Sat, 28 Sep 2024 21:47:34 +0200, Javier Carrasco wrote:
> This series switches from the device_for_each_child_node() macro to its
> scoped variant. This makes the code more robust if new early exits are
> added to the loops, because there is no need for explicit calls to
> fwnode_handle_put(), which also simplifies existing code.
>
> The non-scoped macros to walk over nodes turn error-prone as soon as
> the loop contains early exits (break, goto, return), and patches to
> fix them show up regularly, sometimes due to new error paths in an
> existing loop [1].
>
> [...]
Applied, thanks!
[1/3] gpio: dwapb: switch to device_for_each_child_node_scoped()
commit: 0a53be8e080b53ef922e90204999f4ccef29cd57
[2/3] gpio: sim: switch to device_for_each_child_node_scoped()
commit: d64d0287f4bc7013c60b07e34e43c3fc558e3808
Best regards,
--
Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>