clk: Always notify whole subtree when reparenting

A clock's notifier count only reflects notifiers which are registered
directly for that clock. A reparent operation though affects the whole
subtree because of a potential rate change.
When issuing the pre rate change notifications only the notifier count
for the clock to be changed is considered and notifiers for subclocks
may never be called. Resulting in clocks in the subtree which have
registered notifiers, may receive a POST_- or ABORT_RATE_CHANGE
notification, without a PRE_RATE_CHANGE_NOTIFICATION.
Therefore always traverse the whole subtree when issueing pre rate
change notifications during a reparent operation.

Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
Signed-off-by: Mike Turquette <mturquette@linaro.org>

diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
index 2e669a8..edf3fe1 100644 (file)
--- a/drivers/clk/clk.c
+++ b/drivers/clk/clk.c
@@ -1493,8 +1493,7 @@ int clk_set_parent(struct clk *clk, struct clk *parent)
        }
 
        /* propagate PRE_RATE_CHANGE notifications */
-       if (clk->notifier_count)
-               ret = __clk_speculate_rates(clk, p_rate);
+       ret = __clk_speculate_rates(clk, p_rate);
 
        /* abort if a driver objects */
        if (ret & NOTIFY_STOP_MASK)