[PATCH 3/3] memory: ti-aemif: don't needlessly iterate over child nodes

Bartosz Golaszewski posted 3 patches 1 year, 6 months ago
[PATCH 3/3] memory: ti-aemif: don't needlessly iterate over child nodes
Posted by Bartosz Golaszewski 1 year, 6 months ago
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

When populating devices from a specific node, we don't need to call
of_platform_populate() for every child node manually - the routine
will already do it. We can call it directly from the platform device's
OF node. While at it: use the managed variant of of_platform_populate().

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
---
 drivers/memory/ti-aemif.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/drivers/memory/ti-aemif.c b/drivers/memory/ti-aemif.c
index fbe0786f5ac4..49e7cbe98fdb 100644
--- a/drivers/memory/ti-aemif.c
+++ b/drivers/memory/ti-aemif.c
@@ -383,13 +383,9 @@ static int aemif_probe(struct platform_device *pdev)
 	 * child will be probed after the AEMIF timing parameters are set.
 	 */
 	if (np) {
-		for_each_available_child_of_node(np, child_np) {
-			ret = of_platform_populate(child_np, NULL, NULL, dev);
-			if (ret < 0) {
-				of_node_put(child_np);
-				return ret;
-			}
-		}
+		ret = devm_of_platform_populate(dev);
+		if (ret)
+			return ret;
 	}
 
 	return 0;

-- 
2.43.0