[PATCH] of/platform: Simplify with scoped for each OF child loop

Krzysztof Kozlowski posted 1 patch 1 month, 1 week ago
drivers/of/platform.c | 15 +++++++--------
1 file changed, 7 insertions(+), 8 deletions(-)
[PATCH] of/platform: Simplify with scoped for each OF child loop
Posted by Krzysztof Kozlowski 1 month, 1 week ago
Use scoped for-each loop when iterating over device nodes to make code a
bit simpler.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
---
 drivers/of/platform.c | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/drivers/of/platform.c b/drivers/of/platform.c
index f77cb19973a5..d4e169b3b5af 100644
--- a/drivers/of/platform.c
+++ b/drivers/of/platform.c
@@ -394,7 +394,6 @@ int of_platform_bus_probe(struct device_node *root,
 			  const struct of_device_id *matches,
 			  struct device *parent)
 {
-	struct device_node *child;
 	int rc = 0;
 
 	root = root ? of_node_get(root) : of_find_node_by_path("/");
@@ -407,13 +406,13 @@ int of_platform_bus_probe(struct device_node *root,
 	/* Do a self check of bus type, if there's a match, create children */
 	if (of_match_node(matches, root)) {
 		rc = of_platform_bus_create(root, matches, NULL, parent, false);
-	} else for_each_child_of_node(root, child) {
-		if (!of_match_node(matches, child))
-			continue;
-		rc = of_platform_bus_create(child, matches, NULL, parent, false);
-		if (rc) {
-			of_node_put(child);
-			break;
+	} else {
+		for_each_child_of_node_scoped(root, child) {
+			if (!of_match_node(matches, child))
+				continue;
+			rc = of_platform_bus_create(child, matches, NULL, parent, false);
+			if (rc)
+				break;
 		}
 	}
 
-- 
2.51.0
Re: [PATCH] of/platform: Simplify with scoped for each OF child loop
Posted by Rob Herring (Arm) 1 month, 1 week ago
On Wed, 31 Dec 2025 13:09:27 +0100, Krzysztof Kozlowski wrote:
> Use scoped for-each loop when iterating over device nodes to make code a
> bit simpler.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> ---
>  drivers/of/platform.c | 15 +++++++--------
>  1 file changed, 7 insertions(+), 8 deletions(-)
> 

Applied, thanks!