thermal: db8500: demote temperature to debug print

Message ID 20191001074930.8637-1-linus.walleij@linaro.org
State Superseded
Headers show
Series
  • thermal: db8500: demote temperature to debug print
Related show

Commit Message

Linus Walleij Oct. 1, 2019, 7:49 a.m.
There is no need to print this to the console, just a
missed informational debug print.

Cc: Vincent Guittot <vincent.guittot@linaro.org>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
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

Comments

Daniel Lezcano Oct. 1, 2019, 9:01 a.m. | #1
On 01/10/2019 09:49, Linus Walleij wrote:
> There is no need to print this to the console, just a

> missed informational debug print.

> 

> Cc: Vincent Guittot <vincent.guittot@linaro.org>

> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>

> Fixes: 6c375eccded4 ("thermal: db8500: Rewrite to be a pure OF sensor")

> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>


Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>


> ---

>  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

Patch

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;