powerpc/mpc5xxx: Use of_get_next_parent to simplify code
authorChristophe Jaillet <christophe.jaillet@wanadoo.fr>
Sun, 11 Oct 2015 20:27:40 +0000 (22:27 +0200)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 15 Oct 2015 09:32:01 +0000 (20:32 +1100)
of_get_next_parent can be used to simplify the while() loop and
avoid the need of a temp variable.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/sysdev/mpc5xxx_clocks.c

index f4f0301..5732926 100644 (file)
@@ -13,7 +13,6 @@
 
 unsigned long mpc5xxx_get_bus_frequency(struct device_node *node)
 {
-       struct device_node *np;
        const unsigned int *p_bus_freq = NULL;
 
        of_node_get(node);
@@ -22,9 +21,7 @@ unsigned long mpc5xxx_get_bus_frequency(struct device_node *node)
                if (p_bus_freq)
                        break;
 
-               np = of_get_parent(node);
-               of_node_put(node);
-               node = np;
+               node = of_get_next_parent(node);
        }
        of_node_put(node);