| Message ID | 2de2d178380525191d99d6c3619ad8c45efdbab4.1547458732.git.amit.kucheria@linaro.org |
|---|---|
| State | Accepted |
| Commit | bf78f133cd39e0ed41551150909e41513958a738 |
| Headers | show |
| Series | None | expand |
On 14-01-19, 15:51, Amit Kucheria wrote: > 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 <amit.kucheria@linaro.org> > --- > drivers/thermal/cpu_cooling.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c > index dfd23245f778..6fff16113628 100644 > --- 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; > } Always keep such cleanup patches at the top, so the maintainers can pick them up easily even if the entire series doesn't get in. Acked-by: Viresh Kumar <viresh.kumar@linaro.org> -- viresh
On Thu, Jan 17, 2019 at 6:57 AM Viresh Kumar <viresh.kumar@linaro.org> wrote: > > On 14-01-19, 15:51, Amit Kucheria wrote: > > 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 <amit.kucheria@linaro.org> > > --- > > drivers/thermal/cpu_cooling.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c > > index dfd23245f778..6fff16113628 100644 > > --- 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; > > } > > Always keep such cleanup patches at the top, so the maintainers can > pick them up easily even if the entire series doesn't get in. Actually, if nothing in the series depends on this change and this change doesn't depend on anything in the series, why is it part of the series at all?
diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index dfd23245f778..6fff16113628 100644 --- 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; }
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 <amit.kucheria@linaro.org> --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.17.1