Received-SPF: pass (google.com: domain of linux-kernel+bounces-165271-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
Date: Wed, 01 May 2024 20:56:01 +0800
Subject: [PATCH 03/21] pinctrl: sunplus: Use scope based of_node_put()
 cleanups
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Message-Id: <20240501-pinctrl-cleanup-v1-3-797ceca46e5c@nxp.com>
References: <20240501-pinctrl-cleanup-v1-0-797ceca46e5c@nxp.com>
In-Reply-To: <20240501-pinctrl-cleanup-v1-0-797ceca46e5c@nxp.com>
To: Linus Walleij <linus.walleij@linaro.org>, 
 Thierry Reding <thierry.reding@gmail.com>, 
 Jonathan Hunter <jonathanh@nvidia.com>, Dvorkin Dmitry <dvorkin@tibbo.com>, 
 Wells Lu <wellslutw@gmail.com>, Maxime Coquelin <mcoquelin.stm32@gmail.com>, 
 Alexandre Torgue <alexandre.torgue@foss.st.com>, 
 Emil Renner Berthing <kernel@esmil.dk>, 
 Jianlong Huang <jianlong.huang@starfivetech.com>, 
 Hal Feng <hal.feng@starfivetech.com>, Orson Zhai <orsonzhai@gmail.com>, 
 Baolin Wang <baolin.wang@linux.alibaba.com>, 
 Chunyan Zhang <zhang.lyra@gmail.com>, Viresh Kumar <vireshk@kernel.org>, 
 Shiraz Hashim <shiraz.linux.kernel@gmail.com>, soc@kernel.org, 
 Krzysztof Kozlowski <krzk@kernel.org>, 
 Sylwester Nawrocki <s.nawrocki@samsung.com>, 
 Alim Akhtar <alim.akhtar@samsung.com>, 
 Geert Uytterhoeven <geert+renesas@glider.be>, 
 Patrice Chotard <patrice.chotard@foss.st.com>, 
 Heiko Stuebner <heiko@sntech.de>, Damien Le Moal <dlemoal@kernel.org>, 
 Ludovic Desroches <ludovic.desroches@microchip.com>, 
 Nicolas Ferre <nicolas.ferre@microchip.com>, 
 Alexandre Belloni <alexandre.belloni@bootlin.com>, 
 Claudiu Beznea <claudiu.beznea@tuxon.dev>, 
 Dong Aisheng <aisheng.dong@nxp.com>, Fabio Estevam <festevam@gmail.com>, 
 Shawn Guo <shawnguo@kernel.org>, Jacky Bai <ping.bai@nxp.com>, 
 Pengutronix Kernel Team <kernel@pengutronix.de>, 
 Chester Lin <chester62515@gmail.com>, Matthias Brugger <mbrugger@suse.com>, 
 Ghennadi Procopciuc <ghennadi.procopciuc@oss.nxp.com>, 
 Sean Wang <sean.wang@kernel.org>, Matthias Brugger <matthias.bgg@gmail.com>, 
 AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, 
 Sascha Hauer <s.hauer@pengutronix.de>, 
 Andrew Jeffery <andrew@codeconstruct.com.au>, Joel Stanley <joel@jms.id.au>, 
 Dan Carpenter <dan.carpenter@linaro.org>
Cc: linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, 
 linux-tegra@vger.kernel.org, linux-arm-kernel@lists.infradead.org, 
 linux-stm32@st-md-mailman.stormreply.com, linux-samsung-soc@vger.kernel.org, 
 linux-renesas-soc@vger.kernel.org, linux-rockchip@lists.infradead.org, 
 linux-riscv@lists.infradead.org, linux-mediatek@lists.infradead.org, 
 imx@lists.linux.dev, linux-aspeed@lists.ozlabs.org, 
 openbmc@lists.ozlabs.org, Peng Fan <peng.fan@nxp.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: DU0PR04MB9417:EE_|DB9PR04MB9284:EE_
X-MS-Office365-Filtering-Correlation-Id: 065f0460-04e2-428c-56a7-08dc69dd1913
X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam:
	BCL:0;ARA:13230031|1800799015|376005|52116005|7416005|366007|38350700005|921011;
X-Microsoft-Antispam-Message-Info:
	=?utf-8?B?dWgxa29RSWQrb1I4M1FZKzIweXZqakNKUGNjc3ZMQlhzbEZEZU9TMi9WTmJy?=
 =?utf-8?B?TzZFdWRvV3hZMlFDTWZ4SjJQcHhrVVhnWHBEN010OXNzS2tacDc5SkhtaG1r?=
 =?utf-8?B?NXU2WjhGL1BscmdpUGUvZy9tRUNBV0VJU3FPczd0MW1QN2grZzU0TkwzYURo?=
 =?utf-8?B?Q1BUbXZwcmpXYmY1QnRWbm16dHBRcWJzMWpMb29BaGhBRXdzQmczemFsRnRQ?=
 =?utf-8?B?WkczaFNscGlXU2J6ckNBSU9lK1NJTEwxem5odUVHM3I1MWVKZldjVTdWUFRN?=
 =?utf-8?B?S2ZxaWY1b05rTHEzN2ViMmk0RGM4QUUzOURVancrb3JFcWo3RnVJUEVwc0Rm?=
 =?utf-8?B?ZUhHZlJSMXJ3cjlTbjZnTHZibzFoYWlkbDc5L0hjM2FDWTZxTnQrd05vR24v?=
 =?utf-8?B?S0pwWEhXS2Qrak5DTlZyaGpLcXpTMXdORW44eFpXMjFHQUtKNTJrWU9YVmR6?=
 =?utf-8?B?ZTNmbmo4TC84VUZrNkU0R2JhSmlHaVlMUC9JUjVrNTV5eldYeW9uZU8zc3pD?=
 =?utf-8?B?RFNZS3lsSHQxQndHcXNtTG85RVBUbzFEQ2t1ays0cHhLeE5IUGtqTHZYRnR3?=
 =?utf-8?B?ZVMxOHZjd0VxZGc2ZUJvdi9tc3owL29yOVhoN3czTUIwMk9FNnZ4M0RqbU5o?=
 =?utf-8?B?ZHBoejhiblhUV3JhcTRIdmZZYnozR2NBaTRHNmtQSm8yaHBrSXpVNGdWQVN4?=
 =?utf-8?B?Z2cvaVFoQmdvQndNNTJPR09GUXY0OEJwNjcwcmZXNUtWNW9DY0Y4dG0zcGVR?=
 =?utf-8?B?a0E0b0lUUG43dUtOc3kydnI5NVk1RStFcFpWUFpkeThaZGpXQ3VxanhaNmVL?=
 =?utf-8?B?QUoxWEthOWNLd0dkc2JHeWFRdEdncHp3ZHE0a1g3ZXdnNXFhOGJUY040V1BV?=
 =?utf-8?B?RFBXQStVTXQ0ay8rRXIyZmdxQVVtYURObUNVNncyT2Y1dWVYNzFVR21OUmhE?=
 =?utf-8?B?d1QyNThSMHFNSVNEbDNwUitIK2dia0VIVkR0YmFiWUJXODlyN3ZRMnREWWR3?=
 =?utf-8?B?ZjdDcDhzOXZtYlV5NndCT3BNSkQ2MURFN0d4aXVYK0JFSE1WSnBXU1ZNenhn?=
 =?utf-8?B?aFNiWk16aDE1cFEvL2RuUDQ2a0FsYkwxUHltSzB3NVFtWmZSRTArcDcxK21I?=
 =?utf-8?B?QUpUb2N2czdKaFNDVnFhV3BhZDhjdWdVbVlFNnA5OWRudXppK1NQa2dJZEk4?=
 =?utf-8?B?Q1lSRU1pV1JtTnpjNU93Y0lxZ25mdVlYQnhYc0Z4bXZDZU91UEMyYjRkbFE5?=
 =?utf-8?B?VFdJUk5tMVFiWCt6NHc2UmoyVmwyeW80bHc5SmNtT05TVzRMYTBNcnRUblk3?=
 =?utf-8?B?cHlwVExGUkxZL2dGR0MvbUpZdzg2VnFleExIN1VjNVltNkphM0JqdnVaYysy?=
 =?utf-8?B?ZXNabEwzZFJVdWRXdmw5RmhtWUNzSm9Gczl3aU5tcXFPbU5hdVh3ekJCVjRH?=
 =?utf-8?B?azNPbElOUEpYalBBSlVPenVQMTYvY1VhQ3Bsc1VoTlVCc2RETFc1SmtIdXkr?=
 =?utf-8?B?NVhsSDBGWWIyOS9LN1FaM0JpN2hxdWl2MWRYNWRWaElEaHprY1l0NzRZTksz?=
 =?utf-8?B?YzZXbTcxSDBFQjZGVnFQeTI2dGt1aWppK0RvU0pMTVp2ZUg1VVZldktzbTRk?=
 =?utf-8?B?K0ZQL2ZnRzFINTFTS1lnRU5jS2pmUEpiVFZHYmpmZHRiVzMzSEJaeXJCL0R1?=
 =?utf-8?B?NWRQamloQ2xKeC9zcXhScWQvNjFVamlXd294bDJGem50ZHRkMFlOWlM5cWtx?=
 =?utf-8?B?Mnc3OXY0OWU1aTB0MnpQM3lMUkRqb0hQc0xkM1dXME9PMkJJWkFoWlFHdnR2?=
 =?utf-8?Q?3UCC1vhaTalv4F3aFfhdRU5kwAIVYGktXVNPU=3D?=
X-Forefront-Antispam-Report:
	CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU0PR04MB9417.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(1800799015)(376005)(52116005)(7416005)(366007)(38350700005)(921011);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?RncyM2k1NklpYnNOWUdwTWtuVFN3ZFl3QUxRYmJqbTZDSGd6RzQ2dHBWU05C?=
 =?utf-8?B?MnNwcDAvZG9CNEZQd1pIbDFiWnJlUnAyYU1yaFdiYUVIMm1ISk5KUG0xb0Fp?=
 =?utf-8?B?S0ZYR2ltelpXVk5ibEhmNVo5OUd4aG9UMWJTUE13dHRJZVp3OGxSVEdLNGJJ?=
 =?utf-8?B?S3k2NjEwcHN4ZW5zN3NjVXhsU3NZNFpyaGNXTzZjUWFHbXZobEJ0QlNVblV1?=
 =?utf-8?B?SHJaTW5rM2ZHWjN4c3NpNjJJSlE1dWZuZU5PaVJoRjN5WG5BeEZuM0RXbC8r?=
 =?utf-8?B?c0JlUjd5OUZPTlBHcjRuRmNsZzcrYjBJbHZkaVJaWEl6S243N3FzMlQvajNl?=
 =?utf-8?B?dGxjUTRxb1RBN05MTnZrdjU0L1ZPMnhqMWphTGJ1SkRRdFRNZkZCYm5YeG4y?=
 =?utf-8?B?VUpEV0NsZjgyVXlpQVpLWVM5VWhsUStWN2tRRFBUSzZ4Yjc1Ui82clhnaWVZ?=
 =?utf-8?B?MmFLSFo3WHdCaTJBR0lZZ2VObXExT2NJQkJvRFo1M2N1UXJURTFZaVYrWkRz?=
 =?utf-8?B?TWpLeEpkVk5EbDcybHlHamtSR3k2dXRMSzJlSzkyZXhoamZnV3RNQ1VsVGhF?=
 =?utf-8?B?VWQ2bDFjTHpuTVk4OTRmSDM4VE9PWTVUQU42NzV3TllHaEpYaWRpSHdIY2Nr?=
 =?utf-8?B?L1RkTEMwMVovUEVWVTdWSGZvK0hudTJUdGtSTlBvR2luOTV5VVBVUTgwVE0z?=
 =?utf-8?B?OUJWTWM3THN5aWNKd3lRR1VnU0JOV1MxMUNwZHdQSVdkakYzUE1mdGw1YjJS?=
 =?utf-8?B?eVIxOXkraC81dktuZ01rK1BTQmM1WUpGUjRBLzNOMDhnc2FRU0ZFc2JrYnB0?=
 =?utf-8?B?S0ppNE5jRC9GNXRkbFZ0c0xZdW41UnFGNUd0a0JwV2ZEb2E4cm1mZ3QxVTlO?=
 =?utf-8?B?TWNDYWVrbXU0NCsxZTVzMkYrWFlCSVozZUEwcmZ0b2ZSb1ptWnJyNUh1Zmpl?=
 =?utf-8?B?Mk8wOU93alpKaDFIUmlNWXJwdTVvRmYvNko5bDNmb1B3TTZYd0xvN21ZY1Q0?=
 =?utf-8?B?ZUJXS3czOTIvRjBzODFESEhmdSs5MEc0QlJ2UnhWRkl1dE1TNnRSdFJrQnpj?=
 =?utf-8?B?NVFoMUxONEkyNHIwT1RZM3VWSnhnMlEzelh0eTFVc3pTTUx4MFMzb2JVYzhh?=
 =?utf-8?B?NWRaS2tCQ3NOSVlYbWI2R2xCMmR3QkpqNGxFUDJ3eEQ4UWg5ZnJubHlCb0tn?=
 =?utf-8?B?NjJHR21kQzVzRUhQbjF3ekhNVjRKaFJqdWhIYTIvM3hUVnFZLzdDRzY4V0xC?=
 =?utf-8?B?RzBIUzFJamlGSHpuLy9TdUdYRzY2MmlHQXFKa3BiUTMzVkh5djQ5ZUI5WGM4?=
 =?utf-8?B?bXBzT2pSNk0xcll5eVhnNXZBT09zZDg1VGg0SWd0cHc2MHJKa0o1NWg5SzNr?=
 =?utf-8?B?blV5UjNkK1ozdEtYZG5UTzB4Z0R1Q3B5VkNpMlRrUTk1NDZzNmNjaDhjRzIv?=
 =?utf-8?B?S2VKQy9sV0RUSUprYW1lZDl2TWxwbk56ZHc4c0VMNmVad2JsczdXMDZGWlZZ?=
 =?utf-8?B?bHRUTVlIeUlzVXBtZWF0Y3BVZ2lHWFJNV3ZTanZKeVdjUG5XWVlJYmJDbVJs?=
 =?utf-8?B?Q2NXZnpSVFZyZS9MeFY0MWxSdktaMStUSktpa2E5SDFzaXlJYU9UUENyNnNa?=
 =?utf-8?B?b3Rtei8ydFpyU0M1cndHMXVuNVQ2L0VBMkhVWm9qMElKWmRXQXRIRmRXZnVL?=
 =?utf-8?B?WlVmdXB4R2tnYWkrNWNkQVh1UkF4SGlYeU54MmdPa0VnMjF4dk1maXJ6Z0hr?=
 =?utf-8?B?VXZZUlpHZXREVDh0V0FiUW5SVnF6VFhZN0J3MVhYU1hYVGdGWHNoS1R6VjFm?=
 =?utf-8?B?OFluYUNzekRWNUJEMDI5RHN0WnBtSzA4MllKbnVLTEpNcytDSHdvU1diSHBW?=
 =?utf-8?B?eEM1TDZPSk91aE0yY0tSMHdTUlhITWZEOTliZ0dYNVBrOWdBU0VJdUJhK2E0?=
 =?utf-8?B?OXVLcFFiZVJCNFV2VzkwRVc5bTF4UCtwcWdxZERMTGxUakVPWVRMQVJFc3J0?=
 =?utf-8?B?SVZ2RUJYZmptNTBROW5ISTNCOXFlaENpWHpaVis5WHFENFlYVXBZTzk5aVE0?=
 =?utf-8?B?bHJyc0xyb0wycGFkbXVYNFN6NGhSLzdmT2x6RlpFa3MzUUZlbFFCL0tWNG85?=
 =?utf-8?Q?PpH8nE3WoGiTxUR1vvJ9wtWDC?=
X-OriginatorOrg: oss.nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 065f0460-04e2-428c-56a7-08dc69dd1913
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 May 2024 12:49:11.0055
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: kvq5FEOPiyJ+1kQ64iVQwxsQD3eMnMe+tmTitUmgyqQLUOxYlxzgOM82g+xPwvaQXsBROZvmNsKkMC2YFHNfdw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB9284

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/sunplus/sppctl.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/pinctrl/sunplus/sppctl.c b/drivers/pinctrl/sunplus/sppctl.c
index ae156f779a16..604256ec9079 100644
--- a/drivers/pinctrl/sunplus/sppctl.c
+++ b/drivers/pinctrl/sunplus/sppctl.c
@@ -823,7 +823,7 @@ static int sppctl_dt_node_to_map(struct pinctrl_dev *pctldev, struct device_node
 	int nmG = of_property_count_strings(np_config, "groups");
 	const struct sppctl_func *f = NULL;
 	u8 pin_num, pin_type, pin_func;
-	struct device_node *parent;
+	struct device_node *parent __free(device_node) = NULL;
 	unsigned long *configs;
 	struct property *prop;
 	const char *s_f, *s_g;
@@ -964,7 +964,6 @@ static int sppctl_dt_node_to_map(struct pinctrl_dev *pctldev, struct device_node
 		}
 	}
 
-	of_node_put(parent);
 	dev_dbg(pctldev->dev, "%d pins mapped\n", *num_maps);
 	return 0;
 
@@ -973,7 +972,6 @@ static int sppctl_dt_node_to_map(struct pinctrl_dev *pctldev, struct device_node
 		if ((*map)[i].type == PIN_MAP_TYPE_CONFIGS_PIN)
 			kfree((*map)[i].data.configs.configs);
 	kfree(*map);
-	of_node_put(parent);
 	return -ENOMEM;
 }
 

-- 
2.37.1