[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-tegra
Subject: Re: [PATCH v2 03/20] pinctrl: stm32: Use scope based of_node_put() cleanups
From: Patrice CHOTARD <patrice.chotard () foss ! st ! com>
Date: 2024-05-17 6:11:50
Message-ID: 124b75a6-d8da-4e29-8dce-79d18c355ba2 () foss ! st ! com
[Download RAW message or body]
On 5/4/24 15:20, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> Use scope based of_node_put() cleanup to simplify code.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> drivers/pinctrl/stm32/pinctrl-stm32.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/pinctrl/stm32/pinctrl-stm32.c b/drivers/pinctrl/stm32/pinctrl-stm32.c
> index 978ccdbaf3d3..a8673739871d 100644
> --- a/drivers/pinctrl/stm32/pinctrl-stm32.c
> +++ b/drivers/pinctrl/stm32/pinctrl-stm32.c
> @@ -670,7 +670,6 @@ static int stm32_pctrl_dt_node_to_map(struct pinctrl_dev *pctldev,
> struct device_node *np_config,
> struct pinctrl_map **map, unsigned *num_maps)
> {
> - struct device_node *np;
> unsigned reserved_maps;
> int ret;
>
> @@ -678,12 +677,11 @@ static int stm32_pctrl_dt_node_to_map(struct pinctrl_dev *pctldev,
> *num_maps = 0;
> reserved_maps = 0;
>
> - for_each_child_of_node(np_config, np) {
> + for_each_child_of_node_scoped(np_config, np) {
> ret = stm32_pctrl_dt_subnode_to_map(pctldev, np, map,
> &reserved_maps, num_maps);
> if (ret < 0) {
> pinctrl_utils_free_map(pctldev, *map, *num_maps);
> - of_node_put(np);
> return ret;
> }
> }
>
Hi
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Thanks
Patrice
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic