diff options
author | Christophe Jaillet <christophe.jaillet@wanadoo.fr> | 2015-10-11 16:27:40 -0400 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2015-10-15 05:32:01 -0400 |
commit | b340587e68b479e52039f800d7c60abd417e1975 (patch) | |
tree | 68d481a8fed7f94cae6ca442ab8b97536f5b037c | |
parent | 1def37586fb1f3bbbedeaa64bf047595958dfc66 (diff) |
powerpc/mpc5xxx: Use of_get_next_parent to simplify code
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>
-rw-r--r-- | arch/powerpc/sysdev/mpc5xxx_clocks.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/arch/powerpc/sysdev/mpc5xxx_clocks.c b/arch/powerpc/sysdev/mpc5xxx_clocks.c index f4f0301b9a60..573292663cf2 100644 --- a/arch/powerpc/sysdev/mpc5xxx_clocks.c +++ b/arch/powerpc/sysdev/mpc5xxx_clocks.c | |||
@@ -13,7 +13,6 @@ | |||
13 | 13 | ||
14 | unsigned long mpc5xxx_get_bus_frequency(struct device_node *node) | 14 | unsigned long mpc5xxx_get_bus_frequency(struct device_node *node) |
15 | { | 15 | { |
16 | struct device_node *np; | ||
17 | const unsigned int *p_bus_freq = NULL; | 16 | const unsigned int *p_bus_freq = NULL; |
18 | 17 | ||
19 | of_node_get(node); | 18 | of_node_get(node); |
@@ -22,9 +21,7 @@ unsigned long mpc5xxx_get_bus_frequency(struct device_node *node) | |||
22 | if (p_bus_freq) | 21 | if (p_bus_freq) |
23 | break; | 22 | break; |
24 | 23 | ||
25 | np = of_get_parent(node); | 24 | node = of_get_next_parent(node); |
26 | of_node_put(node); | ||
27 | node = np; | ||
28 | } | 25 | } |
29 | of_node_put(node); | 26 | of_node_put(node); |
30 | 27 | ||