soc: bcm: brcmstb: biuctrl: Avoid double of_node_put()
authorFlorian Fainelli <f.fainelli@gmail.com>
Thu, 15 Sep 2022 19:01:59 +0000 (12:01 -0700)
committerFlorian Fainelli <f.fainelli@gmail.com>
Thu, 15 Sep 2022 21:30:29 +0000 (14:30 -0700)
Commit 9a073d4fbb18 ("soc: bcm: brcmstb: biuctrl: Add missing
of_node_put()") added what was thought to be a missing of_node_put() but
now causes a double of_node_put() to be called, once from
setup_hifcpubiuctrl_regs() and another time from brcmstb_biuctrl_init().

Ensure that setup_hifcpubiuctrl_regs() is not calling of_node_put()
since it is not obvious it does that on one of its parameters.

Fixes: 9a073d4fbb18 ("soc: bcm: brcmstb: biuctrl: Add missing of_node_put()")
Reported-by: Jim Quinlan <jim2101024@gmail.com>
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Change-Id: I1c405c36c2f06c8b8c0f684143b7a52db7e809f0

drivers/soc/bcm/brcmstb/biuctrl.c

index 1467bbd..e1d7b45 100644 (file)
@@ -288,7 +288,6 @@ static int __init setup_hifcpubiuctrl_regs(struct device_node *np)
        if (BRCM_ID(family_id) == 0x7260 && BRCM_REV(family_id) == 0)
                cpubiuctrl_regs = b53_cpubiuctrl_no_wb_regs;
 out:
-       of_node_put(np);
        return ret;
 }