Skip to content

Commit 5b60697

Browse files
vireshkrafaeljw
authored andcommitted
PM / OPP: OF: Use pr_debug() instead of pr_err() while adding OPP table
Some platforms add the OPPs dynamically from platform specific drivers instead of getting them statically from DT. The cpufreq-dt driver already ignores the return value of dev_pm_opp_of_cpumask_add_table() to not error out for such cases, but we still end up printing error message from that routine. That's not nice. Convert the print message to use pr_debug() instead. Reported-by: Mason <[email protected]> Signed-off-by: Viresh Kumar <[email protected]> Reviewed-by: Stephen Boyd <[email protected]> Signed-off-by: Rafael J. Wysocki <[email protected]>
1 parent 5771a8c commit 5b60697

File tree

1 file changed

+6
-2
lines changed
  • drivers/base/power/opp

1 file changed

+6
-2
lines changed

drivers/base/power/opp/of.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -539,8 +539,12 @@ int dev_pm_opp_of_cpumask_add_table(const struct cpumask *cpumask)
539539

540540
ret = dev_pm_opp_of_add_table(cpu_dev);
541541
if (ret) {
542-
pr_err("%s: couldn't find opp table for cpu:%d, %d\n",
543-
__func__, cpu, ret);
542+
/*
543+
* OPP may get registered dynamically, don't print error
544+
* message here.
545+
*/
546+
pr_debug("%s: couldn't find opp table for cpu:%d, %d\n",
547+
__func__, cpu, ret);
544548

545549
/* Free all other OPPs */
546550
dev_pm_opp_of_cpumask_remove_table(cpumask);

0 commit comments

Comments
 (0)