| Message ID | 20191119074650.2664-1-linus.walleij@linaro.org |
|---|---|
| State | Accepted |
| Commit | c56dcfa3d4d0f49f0c37cd24886aa86db7aa7f30 |
| Headers | show |
| Series | thermal: db8500: Depromote debug print | expand |
On Tue, Nov 19, 2019 at 08:46:50AM +0100, Linus Walleij wrote: > We are not interested in getting this debug print on our > console all the time. > > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > Cc: Stephan Gerhold <stephan@gerhold.net> > Fixes: 6c375eccded4 ("thermal: db8500: Rewrite to be a pure OF sensor") > Signed-off-by: Linus Walleij <linus.walleij@linaro.org> I noticed those as well. Thanks for the patch! FWIW: Reviewed-by: Stephan Gerhold <stephan@gerhold.net> > --- > drivers/thermal/db8500_thermal.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/thermal/db8500_thermal.c b/drivers/thermal/db8500_thermal.c > index 372dbbaaafb8..21d4d6e6409a 100644 > --- a/drivers/thermal/db8500_thermal.c > +++ b/drivers/thermal/db8500_thermal.c > @@ -152,8 +152,8 @@ static irqreturn_t prcmu_high_irq_handler(int irq, void *irq_data) > db8500_thermal_update_config(th, idx, THERMAL_TREND_RAISING, > next_low, next_high); > > - dev_info(&th->tz->device, > - "PRCMU set max %ld, min %ld\n", next_high, next_low); > + dev_dbg(&th->tz->device, > + "PRCMU set max %ld, min %ld\n", next_high, next_low); > } else if (idx == num_points - 1) > /* So we roof out 1 degree over the max point */ > th->interpolated_temp = db8500_thermal_points[idx] + 1; > -- > 2.21.0 >
On 19/11/2019 08:46, Linus Walleij wrote: > We are not interested in getting this debug print on our > console all the time. > > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > Cc: Stephan Gerhold <stephan@gerhold.net> > Fixes: 6c375eccded4 ("thermal: db8500: Rewrite to be a pure OF sensor") > Signed-off-by: Linus Walleij <linus.walleij@linaro.org> Applied, thanks! > --- > drivers/thermal/db8500_thermal.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/thermal/db8500_thermal.c b/drivers/thermal/db8500_thermal.c > index 372dbbaaafb8..21d4d6e6409a 100644 > --- a/drivers/thermal/db8500_thermal.c > +++ b/drivers/thermal/db8500_thermal.c > @@ -152,8 +152,8 @@ static irqreturn_t prcmu_high_irq_handler(int irq, void *irq_data) > db8500_thermal_update_config(th, idx, THERMAL_TREND_RAISING, > next_low, next_high); > > - dev_info(&th->tz->device, > - "PRCMU set max %ld, min %ld\n", next_high, next_low); > + dev_dbg(&th->tz->device, > + "PRCMU set max %ld, min %ld\n", next_high, next_low); > } else if (idx == num_points - 1) > /* So we roof out 1 degree over the max point */ > th->interpolated_temp = db8500_thermal_points[idx] + 1; > -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
diff --git a/drivers/thermal/db8500_thermal.c b/drivers/thermal/db8500_thermal.c index 372dbbaaafb8..21d4d6e6409a 100644 --- a/drivers/thermal/db8500_thermal.c +++ b/drivers/thermal/db8500_thermal.c @@ -152,8 +152,8 @@ static irqreturn_t prcmu_high_irq_handler(int irq, void *irq_data) db8500_thermal_update_config(th, idx, THERMAL_TREND_RAISING, next_low, next_high); - dev_info(&th->tz->device, - "PRCMU set max %ld, min %ld\n", next_high, next_low); + dev_dbg(&th->tz->device, + "PRCMU set max %ld, min %ld\n", next_high, next_low); } else if (idx == num_points - 1) /* So we roof out 1 degree over the max point */ th->interpolated_temp = db8500_thermal_points[idx] + 1;
We are not interested in getting this debug print on our console all the time. Cc: Daniel Lezcano <daniel.lezcano@linaro.org> Cc: Stephan Gerhold <stephan@gerhold.net> Fixes: 6c375eccded4 ("thermal: db8500: Rewrite to be a pure OF sensor") Signed-off-by: Linus Walleij <linus.walleij@linaro.org> --- drivers/thermal/db8500_thermal.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.21.0