thermal: cpu_cooling: Clarify error message

idlethread · Eduardo Valentin · commit bf78f133cd39 · 2019-02-05T15:50:13.000-08:00
Make it clear that it is a failure if the cpufreq driver was unable to
register as a cooling device. Makes it easier to find in logs and
grepping for words like fail, err, warn.

Signed-off-by: Amit Kucheria &lt;amit.kucheria@linaro.org&gt;
Acked-by: Viresh Kumar &lt;viresh.kumar@linaro.org&gt;
Signed-off-by: Eduardo Valentin &lt;edubezval@gmail.com&gt;
diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c
@@ -774,7 +774,7 @@ of_cpufreq_cooling_register(struct cpufreq_policy *policy)
 
 		cdev = __cpufreq_cooling_register(np, policy, capacitance);
 		if (IS_ERR(cdev)) {
-			pr_err("cpu_cooling: cpu%d is not running as cooling device: %ld\n",
+			pr_err("cpu_cooling: cpu%d failed to register as cooling device: %ld\n",
 			       policy->cpu, PTR_ERR(cdev));
 			cdev = NULL;
 		}

Original file line number	Diff line number	Diff line change
`@@ -774,7 +774,7 @@ of_cpufreq_cooling_register(struct cpufreq_policy *policy)`
`774`	`774`
`775`	`775`	`cdev = __cpufreq_cooling_register(np, policy, capacitance);`
`776`	`776`	`if (IS_ERR(cdev)) {`
`777`		`- pr_err("cpu_cooling: cpu%d is not running as cooling device: %ld\n",`
	`777`	`+ pr_err("cpu_cooling: cpu%d failed to register as cooling device: %ld\n",`
`778`	`778`	`policy->cpu, PTR_ERR(cdev));`
`779`	`779`	`cdev = NULL;`
`780`	`780`	`}`