[PATCH 0/2] pinctrl: intel: platform: fix error path in device_for_each_child_node()

Javier Carrasco posted 2 patches 2 months ago
drivers/pinctrl/intel/pinctrl-intel-platform.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
[PATCH 0/2] pinctrl: intel: platform: fix error path in device_for_each_child_node()
Posted by Javier Carrasco 2 months ago
This series fixes an error path where the reference of a child node is
not decremented upon early return. When at it, a trivial comma/semicolon
substitution I found by chance has been added to improve code clarity.

Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
---
Javier Carrasco (2):
      pinctrl: intel: platform: fix error path in device_for_each_child_node()
      pinctrl: intel: platform: use semicolon instead of comma in ncommunities assignment

 drivers/pinctrl/intel/pinctrl-intel-platform.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
---
base-commit: 92fc9636d1471b7f68bfee70c776f7f77e747b97
change-id: 20240926-intel-pinctrl-platform-scoped-68d000ba3c1d

Best regards,
-- 
Javier Carrasco <javier.carrasco.cruz@gmail.com>
Re: [PATCH 0/2] pinctrl: intel: platform: fix error path in device_for_each_child_node()
Posted by Andy Shevchenko 1 month, 4 weeks ago
On Thu, Sep 26, 2024 at 04:11:01PM +0200, Javier Carrasco wrote:
> This series fixes an error path where the reference of a child node is
> not decremented upon early return. When at it, a trivial comma/semicolon
> substitution I found by chance has been added to improve code clarity.
> 
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
> ---
> Javier Carrasco (2):
>       pinctrl: intel: platform: fix error path in device_for_each_child_node()
>       pinctrl: intel: platform: use semicolon instead of comma in ncommunities assignment
> 
>  drivers/pinctrl/intel/pinctrl-intel-platform.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> ---
> base-commit: 92fc9636d1471b7f68bfee70c776f7f77e747b97

Try hard to avoid basing your commits on Linux Next. This is a specific
subsystem and you can just use Intel pinctrl tree for that.

No need to resend this time, I fixed the issue manually.

-- 
With Best Regards,
Andy Shevchenko
Re: [PATCH 0/2] pinctrl: intel: platform: fix error path in device_for_each_child_node()
Posted by Mika Westerberg 2 months ago
On Thu, Sep 26, 2024 at 04:11:01PM +0200, Javier Carrasco wrote:
> This series fixes an error path where the reference of a child node is
> not decremented upon early return. When at it, a trivial comma/semicolon
> substitution I found by chance has been added to improve code clarity.
> 
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
> ---
> Javier Carrasco (2):
>       pinctrl: intel: platform: fix error path in device_for_each_child_node()
>       pinctrl: intel: platform: use semicolon instead of comma in ncommunities assignment

Both,

Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com>