mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-12 13:55:32 +00:00
clk: add pr_debug & kerneldoc around clk notifiers
Both the pr_err and the additional kerneldoc aim to help when debugging errors thrown from within a clock rate-change notifier callback. Reported-by: Sören Brinkmann <soren.brinkmann@xilinx.com> Acked-by: Sören Brinkmann <soren.brinkmann@xilinx.com> Signed-off-by: Mike Turquette <mturquette@linaro.org>
This commit is contained in:
parent
05135b4cb4
commit
86bcfa2e87
2 changed files with 18 additions and 1 deletions
|
@ -1339,8 +1339,11 @@ static int __clk_speculate_rates(struct clk *clk, unsigned long parent_rate)
|
|||
if (clk->notifier_count)
|
||||
ret = __clk_notify(clk, PRE_RATE_CHANGE, clk->rate, new_rate);
|
||||
|
||||
if (ret & NOTIFY_STOP_MASK)
|
||||
if (ret & NOTIFY_STOP_MASK) {
|
||||
pr_debug("%s: clk notifier callback for clock %s aborted with error %d\n",
|
||||
__func__, clk->name, ret);
|
||||
goto out;
|
||||
}
|
||||
|
||||
hlist_for_each_entry(child, &clk->children, child_node) {
|
||||
ret = __clk_speculate_rates(child, new_rate);
|
||||
|
|
|
@ -78,8 +78,22 @@ struct clk_notifier_data {
|
|||
unsigned long new_rate;
|
||||
};
|
||||
|
||||
/**
|
||||
* clk_notifier_register: register a clock rate-change notifier callback
|
||||
* @clk: clock whose rate we are interested in
|
||||
* @nb: notifier block with callback function pointer
|
||||
*
|
||||
* ProTip: debugging across notifier chains can be frustrating. Make sure that
|
||||
* your notifier callback function prints a nice big warning in case of
|
||||
* failure.
|
||||
*/
|
||||
int clk_notifier_register(struct clk *clk, struct notifier_block *nb);
|
||||
|
||||
/**
|
||||
* clk_notifier_unregister: unregister a clock rate-change notifier callback
|
||||
* @clk: clock whose rate we are no longer interested in
|
||||
* @nb: notifier block which will be unregistered
|
||||
*/
|
||||
int clk_notifier_unregister(struct clk *clk, struct notifier_block *nb);
|
||||
|
||||
/**
|
||||
|
|
Loading…
Reference in a new issue