mtd: rawnand: qcom: Simplify with scoped for each OF child loop

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>
Reviewed-by: Manivannan Sadhasivam <mani@kernel.org>
Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c
index 4dd6f1a4e797857471ea57386dfed715f5b3019d..b7e79b76654d908e4ee8ff832938901c9c08541f 100644 (file)
--- a/drivers/mtd/nand/raw/qcom_nandc.c
+++ b/drivers/mtd/nand/raw/qcom_nandc.c
@@ -2206,16 +2206,14 @@ err:
 static int qcom_probe_nand_devices(struct qcom_nand_controller *nandc)
 {
        struct device *dev = nandc->dev;
-       struct device_node *dn = dev->of_node, *child;
+       struct device_node *dn = dev->of_node;
        struct qcom_nand_host *host;
        int ret = -ENODEV;
 
-       for_each_available_child_of_node(dn, child) {
+       for_each_available_child_of_node_scoped(dn, child) {
                host = devm_kzalloc(dev, sizeof(*host), GFP_KERNEL);
-               if (!host) {
-                       of_node_put(child);
+               if (!host)
                        return -ENOMEM;
-               }
 
                ret = qcom_nand_host_init_and_register(nandc, host, child);
                if (ret) {