ARM: BCM63xx: remove useless goto statement
authorDario Binacchi <dario.binacchi@amarulasolutions.com>
Tue, 6 Dec 2022 17:09:13 +0000 (18:09 +0100)
committerFlorian Fainelli <f.fainelli@gmail.com>
Tue, 10 Jan 2023 21:46:59 +0000 (13:46 -0800)
Between the 'goto out' statement and the 'out' label there are no other
statements, so it is useless to check the return value of the
bcm63xx_pmb_power_on_cpu() function. Then, let's remove the statements
that are unnecessarily executed.

Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Link: https://lore.kernel.org/r/20221206170913.3316205-1-dario.binacchi@amarulasolutions.com
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
arch/arm/mach-bcm/bcm63xx_smp.c

index 641e1f8fcf5e7c9b92bc205f5a2414a7f67c1a2a..18d0ffc621aa98c099d06ea9bcd3ce518cbcc5b9 100644 (file)
@@ -142,8 +142,7 @@ static int bcm63138_smp_boot_secondary(unsigned int cpu,
         */
        ret = bcm63xx_pmb_power_on_cpu(dn);
        of_node_put(dn);
-       if (ret)
-               goto out;
+
 out:
        iounmap(bootlut_base);