[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240821010739.15293-1-david.hunter.linux@gmail.com>
Date: Tue, 20 Aug 2024 21:07:39 -0400
From: David Hunter <david.hunter.linux@...il.com>
To: sboyd@...nel.org
Cc: david.hunter.linux@...il.com,
javier.carrasco.cruz@...il.com,
linux-clk@...r.kernel.org,
linux-kernel@...r.kernel.org,
mturquette@...libre.com,
shuah@...nel.org
Subject: [PATCH 1/1 V2] Driver: clk-qoriq.c: replace of_node_put with _free improves cleanup
Use _free() to have automatic cleanup instead of calling of_node_put()
manually.
Compiled without errors or warnings.
Signed-off-by: David Hunter <david.hunter.linux@...il.com>
---
The following commit has information on _free(device_node):
9448e55d032d99af8e23487f51a542d51b2f1a48
V1 --> V2
- Improved message body
- Use change log to give more information about _free()
---
drivers/clk/clk-qoriq.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/clk/clk-qoriq.c b/drivers/clk/clk-qoriq.c
index 4dcde305944c..941a0372db4b 100644
--- a/drivers/clk/clk-qoriq.c
+++ b/drivers/clk/clk-qoriq.c
@@ -1065,11 +1065,8 @@ static void __init _clockgen_init(struct device_node *np, bool legacy);
static void __init legacy_init_clockgen(struct device_node *np)
{
if (!clockgen.node) {
- struct device_node *parent_np;
-
- parent_np = of_get_parent(np);
+ struct device_node __free(device_node) *parent_np = of_get_parent(np);
_clockgen_init(parent_np, true);
- of_node_put(parent_np);
}
}
--
2.43.0
Powered by blists - more mailing lists