]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
mtd: rawnand: arasan: Use for_each_child_of_node_scoped()
authorJinjie Ruan <ruanjinjie@huawei.com>
Mon, 26 Aug 2024 09:43:19 +0000 (17:43 +0800)
committerMiquel Raynal <miquel.raynal@bootlin.com>
Fri, 6 Sep 2024 14:59:41 +0000 (16:59 +0200)
Avoids the need for manual cleanup of_node_put() in early exits
from the loop.

Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20240826094328.2991664-2-ruanjinjie@huawei.com
drivers/mtd/nand/raw/arasan-nand-controller.c

index 2ff1d2b13e3c3a4f8f5ab4b6b18ab801b1844662..5436ec4a8fde425f32aac6b51fde56afbb3fa5a2 100644 (file)
@@ -1360,7 +1360,7 @@ static void anfc_chips_cleanup(struct arasan_nfc *nfc)
 
 static int anfc_chips_init(struct arasan_nfc *nfc)
 {
-       struct device_node *np = nfc->dev->of_node, *nand_np;
+       struct device_node *np = nfc->dev->of_node;
        int nchips = of_get_child_count(np);
        int ret;
 
@@ -1370,10 +1370,9 @@ static int anfc_chips_init(struct arasan_nfc *nfc)
                return -EINVAL;
        }
 
-       for_each_child_of_node(np, nand_np) {
+       for_each_child_of_node_scoped(np, nand_np) {
                ret = anfc_chip_init(nfc, nand_np);
                if (ret) {
-                       of_node_put(nand_np);
                        anfc_chips_cleanup(nfc);
                        break;
                }