]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
powerpc/wii: Simplify with scoped for each OF child loop
authorKrzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Fri, 9 Jan 2026 16:57:49 +0000 (17:57 +0100)
committerRob Herring (Arm) <robh@kernel.org>
Wed, 4 Feb 2026 02:58:13 +0000 (20:58 -0600)
Use scoped for-each loop when iterating over device nodes to make code a
bit simpler.

Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Reviewed-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>
Link: https://patch.msgid.link/20260109-of-for-each-compatible-scoped-v3-5-c22fa2c0749a@oss.qualcomm.com
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
arch/powerpc/platforms/embedded6xx/hlwd-pic.c

index b57e87b0b3ceafd1d4dc9522310c8de79b0cedd5..1522a8bece291dea3fc980486055faec3cc5736d 100644 (file)
@@ -201,11 +201,10 @@ unsigned int hlwd_pic_get_irq(void)
 void __init hlwd_pic_probe(void)
 {
        struct irq_domain *host;
-       struct device_node *np;
        const u32 *interrupts;
        int cascade_virq;
 
-       for_each_compatible_node(np, NULL, "nintendo,hollywood-pic") {
+       for_each_compatible_node_scoped(np, NULL, "nintendo,hollywood-pic") {
                interrupts = of_get_property(np, "interrupts", NULL);
                if (interrupts) {
                        host = hlwd_pic_init(np);
@@ -215,7 +214,6 @@ void __init hlwd_pic_probe(void)
                        irq_set_chained_handler(cascade_virq,
                                                hlwd_pic_irq_cascade);
                        hlwd_irq_host = host;
-                       of_node_put(np);
                        break;
                }
        }