From: David Hunter <david.hunter.linux@gmail.com>
Date: Tue, 9 Jul 2024 14:35:53 +0000 (-0400)
Subject: powerpc-km82xx.c: replace of_node_put() with __free
X-Git-Tag: v6.13-rc1~98^2~8
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=4aa5cc1e0012f784bc7f637458e597564833b425;p=thirdparty%2Fkernel%2Flinux.git

powerpc-km82xx.c: replace of_node_put() with __free

The use of the __free macro allows the cleanup to be based on scope
instead of on another function called later. This makes the cleanup
automatic and less susceptible to errors later.

Signed-off-by: David Hunter <david.hunter.linux@gmail.com>
[mpe: Fix over-long line & change log wording]
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://patch.msgid.link/20240709143553.117053-1-david.hunter.linux@gmail.com
---

diff --git a/arch/powerpc/platforms/82xx/km82xx.c b/arch/powerpc/platforms/82xx/km82xx.c
index c86da3f2b74bd..99f0f0f418767 100644
--- a/arch/powerpc/platforms/82xx/km82xx.c
+++ b/arch/powerpc/platforms/82xx/km82xx.c
@@ -27,15 +27,15 @@
 
 static void __init km82xx_pic_init(void)
 {
-	struct device_node *np = of_find_compatible_node(NULL, NULL,
-							"fsl,pq2-pic");
+	struct device_node *np __free(device_node);
+	np = of_find_compatible_node(NULL, NULL, "fsl,pq2-pic");
+
 	if (!np) {
 		pr_err("PIC init: can not find cpm-pic node\n");
 		return;
 	}
 
 	cpm2_pic_init(np);
-	of_node_put(np);
 }
 
 struct cpm_pin {