Commit 5324fda7 authored by Soren Brinkmann's avatar Soren Brinkmann Committed by Mike Turquette

clk: Fix notifier documentation

Contradicting to documenation, the notifier callbacks do receive
the original clock rate in struct clk_notifier_data.old_rate and the new
frequency struct clk_notifier_data.new_rate, independent of the
notification reason.

This behavior also seems to make more sense, since callbacks can use the
same code to deterimine whether clocks are scaled up or down. Something
which would not even possible in the post-rate-change case if the
behavior was as documented.
Signed-off-by: default avatarSoren Brinkmann <soren.brinkmann@xilinx.com>
Signed-off-by: default avatarMike Turquette <mturquette@linaro.org>
parent 6d0abeca
...@@ -2259,20 +2259,11 @@ void __clk_put(struct clk *clk) ...@@ -2259,20 +2259,11 @@ void __clk_put(struct clk *clk)
* re-enter into the clk framework by calling any top-level clk APIs; * re-enter into the clk framework by calling any top-level clk APIs;
* this will cause a nested prepare_lock mutex. * this will cause a nested prepare_lock mutex.
* *
* Pre-change notifier callbacks will be passed the current, pre-change * In all notification cases cases (pre, post and abort rate change) the
* rate of the clk via struct clk_notifier_data.old_rate. The new, * original clock rate is passed to the callback via struct
* post-change rate of the clk is passed via struct * clk_notifier_data.old_rate and the new frequency is passed via struct
* clk_notifier_data.new_rate. * clk_notifier_data.new_rate.
* *
* Post-change notifiers will pass the now-current, post-change rate of
* the clk in both struct clk_notifier_data.old_rate and struct
* clk_notifier_data.new_rate.
*
* Abort-change notifiers are effectively the opposite of pre-change
* notifiers: the original pre-change clk rate is passed in via struct
* clk_notifier_data.new_rate and the failed post-change rate is passed
* in via struct clk_notifier_data.old_rate.
*
* clk_notifier_register() must be called from non-atomic context. * clk_notifier_register() must be called from non-atomic context.
* Returns -EINVAL if called with null arguments, -ENOMEM upon * Returns -EINVAL if called with null arguments, -ENOMEM upon
* allocation failure; otherwise, passes along the return value of * allocation failure; otherwise, passes along the return value of
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment