From patchwork Sat May 4 13:20:17 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Peng Fan \(OSS\)" X-Patchwork-Id: 794801 Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2087.outbound.protection.outlook.com [40.107.22.87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 81A31364DC; Sat, 4 May 2024 13:17:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.22.87 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714828662; cv=fail; b=cEvUC+T/ODzuhakcap7p79BIr4DUR0vEOlwsGID3DQB+4NlE3bbMmgsCrkSWUYkWE3JThjhHIxUMNc7dXjOBCwqWTfev8TVljZ9QWwF0VD6BINBsx17zorRzmy1jDYHsT4klLp8virT/tpmNjENhvr45E7bQm9XIunmHUo8Q1jU= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714828662; c=relaxed/simple; bh=FyFseGQJmsq1rScaeKVJgP424XsxB+ntR+56HIl8ufU=; h=From:Date:Subject:Content-Type:Message-Id:References:In-Reply-To: To:Cc:MIME-Version; b=m+HwH+SHcuAvTGNpnWR8l7tXHCyDlg3IbXKryo5Y9Trou7YAY2YK5hnYPdu1DRBWvgM/c7gii6IQ5YZRRaiDuGiWSAtilwli/SfCj0u/iX99kiKmhUROKw4n5aLkrZDqy8LAt5WWAH0BdAnr3vLHVaslURGL7bDxm/q8gUsSSR8= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com; spf=pass smtp.mailfrom=oss.nxp.com; dkim=pass (1024-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b=U0oBYf6u; arc=fail smtp.client-ip=40.107.22.87 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b="U0oBYf6u" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kYxiRSbXrlxM0QIZzB2JKkGv1WJuFfsZ5UeNw2hEhj3s2RugeUNPmnruOs3KpMnTUl3wLULJaA9kWr825f0PL1V65U6T2VhJMZNg6Ph0uzezGyCOafoYdFqTRIhQlDiDpUXoQEGJUL4lErnu2rfYTpG+nhVzmvG1aHI3lvEULEOfMiIOVDnLQjkHBRf4LY4rE3vqHoQFP7dQQDVpZQTDZrvPf10iDSA3cniKUGIVHYDdDXKVOSWAzyNS5hYV++eY0ZwfDgQBdvuSrkqtFMqjYwLvMmiRkprFbK3D5aNugridWj1DX/Q806+Vl5faqD4ovOn++GuoFSqEgaPcixcWHg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=rPCG6xZzK0LC/23B+oYJdueIO/buCemfRXEoPX5rvQQ=; b=JTpmd+rqr42czEZb91zi5Jo2aSufwuMAvEL7P+ECrNA9kvs+uFlC3Gxy9vxOQyI3sGAPsnd63WdGDUTMPc4tAHZ6sT8MYx7Ggkuz4D18WQyO8t7vyFCMf50L4Gx3wDbm0FtBo5oANfUu3Q97gmLteAKjDQWVnh/gFvZSdvrJ//7knKIc4IK0NdDx+GLzeaHTKfsi9GFQtchFeWOvT+WnsJUjZnsQvWsNL3D/1xHK4kulYh6lfwpVp1N3hvhBTHaEfm4S25JQ2RqmZ1fZUE/cOAlDy3WY35zRWC+CalNrv7abUyspyaH+5c5EVea/DgLqQbj1FeXDUD+oXMCgoIMPPg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oss.nxp.com; dmarc=pass action=none header.from=oss.nxp.com; dkim=pass header.d=oss.nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NXP1.onmicrosoft.com; s=selector2-NXP1-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=rPCG6xZzK0LC/23B+oYJdueIO/buCemfRXEoPX5rvQQ=; b=U0oBYf6uD96SvqELHvZ1Dm4fRhXclJNCzV5vD9LAKg2FJer/qQAgCy33yfbFfxL6OCfoJmrvNotAl9NB5G7lmt08HgP2vvNUlIgL7KkHi/vOcbuwTCWrK0PGMFYkZMqgRM6FVRLBRnpVS9FZeYPfntla8UGRo1RGes+KL9BWCOc= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oss.nxp.com; Received: from DU0PR04MB9417.eurprd04.prod.outlook.com (2603:10a6:10:358::11) by AM9PR04MB8633.eurprd04.prod.outlook.com (2603:10a6:20b:43c::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7519.34; Sat, 4 May 2024 13:17:37 +0000 Received: from DU0PR04MB9417.eurprd04.prod.outlook.com ([fe80::d30b:44e7:e78e:662d]) by DU0PR04MB9417.eurprd04.prod.outlook.com ([fe80::d30b:44e7:e78e:662d%4]) with mapi id 15.20.7544.036; Sat, 4 May 2024 13:17:37 +0000 From: "Peng Fan (OSS)" Date: Sat, 04 May 2024 21:20:17 +0800 Subject: [PATCH v2 19/20] pinctrl: k210: Use scope based of_node_put() cleanups Message-Id: <20240504-pinctrl-cleanup-v2-19-26c5f2dc1181@nxp.com> References: <20240504-pinctrl-cleanup-v2-0-26c5f2dc1181@nxp.com> In-Reply-To: <20240504-pinctrl-cleanup-v2-0-26c5f2dc1181@nxp.com> To: Linus Walleij , Thierry Reding , Jonathan Hunter , Dvorkin Dmitry , Wells Lu , Maxime Coquelin , Alexandre Torgue , Emil Renner Berthing , Jianlong Huang , Hal Feng , Orson Zhai , Baolin Wang , Chunyan Zhang , Viresh Kumar , Shiraz Hashim , soc@kernel.org, Krzysztof Kozlowski , Sylwester Nawrocki , Alim Akhtar , Geert Uytterhoeven , Patrice Chotard , Heiko Stuebner , Damien Le Moal , Ludovic Desroches , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Dong Aisheng , Fabio Estevam , Shawn Guo , Jacky Bai , Pengutronix Kernel Team , Chester Lin , Matthias Brugger , Ghennadi Procopciuc , Sean Wang , Matthias Brugger , AngeloGioacchino Del Regno , Sascha Hauer , Andrew Jeffery , Joel Stanley , Dan Carpenter , Tony Lindgren , Stephen Warren 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 X-Mailer: b4 0.12.3 X-Developer-Signature: v=1; a=ed25519-sha256; t=1714828815; l=1076; i=peng.fan@nxp.com; s=20230812; h=from:subject:message-id; bh=58KovdEVBZIsn2lfhZ/T0dqiN9oElq31ckJF6Ac0RQ8=; b=Z+wSwh1JPoQAPchmd5Ic4kf6A6+OCSxNrWN3N1KhVLu4RjS5VBGAWLrX0O4aqR9c7Yhj1583S ps7l8SBklTdDPmAjAcjmF7u6Z9ImB7S+gJdZ3GjS/jXZ/LXZiyUzK46 X-Developer-Key: i=peng.fan@nxp.com; a=ed25519; pk=I4sJg7atIT1g63H7bb5lDRGR2gJW14RKDD0wFL8TT1g= X-ClientProxiedBy: SG2PR06CA0230.apcprd06.prod.outlook.com (2603:1096:4:ac::14) To DU0PR04MB9417.eurprd04.prod.outlook.com (2603:10a6:10:358::11) Precedence: bulk X-Mailing-List: linux-samsung-soc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DU0PR04MB9417:EE_|AM9PR04MB8633:EE_ X-MS-Office365-Filtering-Correlation-Id: 656e3a5c-55ec-45f0-bfc5-08dc6c3c916b 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|52116005|7416005|376005|1800799015|366007|38350700005|921011; X-Microsoft-Antispam-Message-Info: =?utf-8?q?/VrCvMeJXuFQRXwKLXMtp5kmy9ufzXk?= =?utf-8?q?nQPBttvQ6TMo5Nlx5UD1h+Y0tz+k3jBcA+EWdpBp6xyDGU6wFLf78K/cQyNwTXJni?= =?utf-8?q?DqT6F42sOlPJ+ddp9uawkarNGh2cAPqd+uf5fzzupPEWA1wUmoDn+yJvIH6g3fPX0?= =?utf-8?q?Nu7LMq4Z+mTtp07nPILUGMrXXsUNdB3zUb9/wIBTEuTqlNgHrEpeB74UK6pBfOocc?= =?utf-8?q?1tsOWY448PgMj3wWLCuYxG+PLl88dXjYBHWD48B9W5eih/Qzjj3Tbb0+tnE/sN6Jf?= =?utf-8?q?UWCX9RyoyghtuKxrm42KDOrheQ4gIYyI52ydjC2KV0ckx/xvwDHw0OpnB5bOLsEf+?= =?utf-8?q?AHJZSVZmnHH9UxvhcO67FY7+B/h3miURjfFSiy2imnCt6Jr1TfjwWYqYUPElidNIU?= =?utf-8?q?z7Bb6ABc9a6gMk7NO7uHUS3xv5CIaPro4vjz69Nt+ayr06yoVpIxgklfguliW6Jp8?= =?utf-8?q?yxkzTYCAM9LLeC7T5yla56lR+4O74L5ErQTfThCYtOOK50l43gpF6TnzYOKNQz5Tu?= =?utf-8?q?FcVfyNsBqq2gUpy/eW4BjSrSELpda4LpoyOlGLkH/30AJ07Dj0/xoXckD8FRUA4d8?= =?utf-8?q?YL9T3pQiVaxOVPCdTZHp4BzSzJ6LWIlFCPNgN0bM9Pn672CZnllF5DeWLKm/YxLE5?= =?utf-8?q?XLEFBdeLwQkfCXZJsjgu0oGGq3Vky+b7UgRfqEuYXo8usC6N3sfoVoqI9wwCZeqOM?= =?utf-8?q?KlZUo0GQWGJOJWCfpsoszza6AySHAiHx2LZXEMq7pg3O5PCq0nr4XAm9irPeji/vi?= =?utf-8?q?eRe9VlvXDaaLwGhTT23Opi+S/6wGE7AdPy6afnQUIr5ynmODgWJ9aOsZzzccQgktN?= =?utf-8?q?Ic6gV0+sGNR2sWUtLgVo+5D5cltQ+uMXCUwJ7SVJa5TbR8JLDj+CH6Mhd/bN7SOG7?= =?utf-8?q?la36anhHyvDQnza1P3RpGWhjcqefaXki/fJW7RdDhsUedTxFcmM1uZmBFsovpTKPH?= =?utf-8?q?E51QmnBMA7W8/aFM8qhUVyqA60XWKugt+LnAIkX1m3qX52my05ZmNbT6xbSWVvmVf?= =?utf-8?q?YRDFk+YBxSZIl+k8dvX+OPXHp2yxPiQux7hVco6h7k9uIJc5j61hOlO3ibsLNk4SA?= =?utf-8?q?HRxZjZWQnO+HudsdHMwXJ8FXcRNDaeJ548ea5mVWwv6eeHTpjC+yS0Teq9K+yHh1M?= =?utf-8?q?6VUcgOdkLDyXxSAr5RvTwEUhS8brPatlnrWt4FPBvHZspzR+LK2x7NpIoGhReqxdT?= =?utf-8?q?HosizniIbvlY4Nz/BZ8uyh5Fmr24fHnSgKtpPG/P9VhlL4uIEFAzPDXhephUaYNBm?= =?utf-8?q?f3n+Kz4EMEajEZFwb+oLPbON/1NL46Q3gcH6TAb6FqxKvSYUCBnoNRSY=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)(52116005)(7416005)(376005)(1800799015)(366007)(38350700005)(921011); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?9rv2Z1vYxrBcb0nKYucoSLamTerZ?= =?utf-8?q?bgXzW0Ex3p72cJ6uU0RowNkvYbhDqx+YpoXKMlYgAqH6SooCXD9k4q9TSmyQ1Eet+?= =?utf-8?q?Voq3QfImqvdNJYStBsDy4UPwpec/id2ObzWwgGiKRcB7BhOn0+IKFlsvHz5BtHIkt?= =?utf-8?q?FXLnIYyY40N/f6Zmt+UnCCLlaU89y2G6iexVe+iE4NxcZ/jXNVT5Wf/WwCGS8+Dv/?= =?utf-8?q?770knUSEVl615PS1JrVRpABYZKALInoJkaDLO3FXcWUhj+ZbrXHEp90zUuLn5BqUI?= =?utf-8?q?WNmbA2OyWu6dIunfL06cPNg2oLD1dEUBLz+rg7KBGaK9qx0cHz5yGVT1ksA42Etcl?= =?utf-8?q?PF1x1A4zlH9mKjLskYFxBuq3ZB5f/MtrX5vQLaCO/uEncKLoXp1apL5uo6WIdbN7F?= =?utf-8?q?HzCLruNt6AXGrqLEY4TGCN+YIdtBLiRw2d0Ia6zNIcmIka5mIMhf2s0ASnl7BnL68?= =?utf-8?q?0HBsxZ+/bA7BZDC4FaoISx5RVj8AgkJVa7GmgPFvLIAHVnOdObZfY/0Qm/SNtS/Cn?= =?utf-8?q?GeY4kbwJihWWwFxSdCpxWo3fT79T7d7jloAO8K5HSEhFU06/e+kYYV8hRGfSH92Fr?= =?utf-8?q?7mv9XqVCtJgGDWO8UVPGUg84o37ScJzT0TP0uv04QDfMcOpnbGVC+CV7AFQumbPD6?= =?utf-8?q?RS+zCtpB7OzaJ7bDFgPmauXBJO6DbUDT48wsDi4OJYretGlH3qy9/X6WuosWzdOeV?= =?utf-8?q?sKPUGyF+KAXXpWdEwl7a8dIhI2dH7cTOp8BStWnW/nN5spdCS4rvIpYaQDvzxjp48?= =?utf-8?q?6gnIglild6rXB0MWxOUVzw5gPawVbAdaFmsuR1z/uzYB4MsUD+Gk/7n5V9Dra1+9d?= =?utf-8?q?cCaTlLGDE5Uxbimdp5OWQo2Q2Ittu+neO3VKxzPQslkHh7yMGNWAkt6+a4mipzggw?= =?utf-8?q?qDtltaH5jqEAY62UTUyj2RQO231TwVcYqXOJPDEUz9XxKbvvanfbpbs0qY/VOida9?= =?utf-8?q?NjcB+awMqA174QqOMxgeImi6zTo5GiS3BuQ1tIjBS3HredfxR3GA16hWsw/h/AgE2?= =?utf-8?q?oK0STgzPWGOaqJabg/i/wzTHtEe9GqP8FZql+rJxORv5DIT/hsGUHnvP/MpAgLcQE?= =?utf-8?q?nQHn8LANtiReFm1a8AAM76Q66BdrNpnYYg7hDmXBGEQu6S2fNE+bK6JHRzFJz5V1Z?= =?utf-8?q?H4Spg7KaknI93RFEYdUni8AexSVe0QjI1HYRk7S62Sc2SqzbKOjBtyUeE0nhkbtbI?= =?utf-8?q?OxvFNKSoGOSG1+sRWwJ3aR8+RgnJ0IMpZf+iO1BpO6iSi3c0GTHa6mihlPwCLbw/Z?= =?utf-8?q?i68FN8SdN0KYXU7X9nqGWG42yMYfxFI1x4G7bOYPGE/1GESQRocIYN/MXr64Mo07P?= =?utf-8?q?bmmL4YixVGljoA+rtC2Ez5N0cUDM0UBriniyyNIqNjOlm664BrTHRVLXHwJCIQEX5?= =?utf-8?q?FNyGdAGHGpy6peekrCmTLYY+n78nEOzjFkFcB7X20zNvJlVQo/1bSccwCY7qV4v7K?= =?utf-8?q?l6zGLWlctc5I5dFPoLqTgB9ZIoHtWvzdiB52HsQoEWrnqSxR5rd9goR+YVBGb8ozJ?= =?utf-8?q?VYfDY99w9qAi?= X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 656e3a5c-55ec-45f0-bfc5-08dc6c3c916b X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 May 2024 13:17:37.4396 (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: g6nupdqB+zySfjS5aEdMaK45BkDcQFF3xV64E9YCjbGQMY976BhNaluoO2HLwNgSGP71nunEjSbX24YIjKnQog== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8633 From: Peng Fan Use scope based of_node_put() cleanup to simplify code. Signed-off-by: Peng Fan --- drivers/pinctrl/pinctrl-k210.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/pinctrl/pinctrl-k210.c b/drivers/pinctrl/pinctrl-k210.c index b6d1ed9ec9a3..2753e14c3e38 100644 --- a/drivers/pinctrl/pinctrl-k210.c +++ b/drivers/pinctrl/pinctrl-k210.c @@ -849,7 +849,6 @@ static int k210_pinctrl_dt_node_to_map(struct pinctrl_dev *pctldev, unsigned int *num_maps) { unsigned int reserved_maps; - struct device_node *np; int ret; reserved_maps = 0; @@ -861,13 +860,11 @@ static int k210_pinctrl_dt_node_to_map(struct pinctrl_dev *pctldev, if (ret < 0) goto err; - for_each_available_child_of_node(np_config, np) { + for_each_available_child_of_node_scoped(np_config, np) { ret = k210_pinctrl_dt_subnode_to_map(pctldev, np, map, &reserved_maps, num_maps); - if (ret < 0) { - of_node_put(np); + if (ret < 0) goto err; - } } return 0;