Re: [PATCH v2] ARM: dts: sun8i-a83t: Add thermal trip points/cooling maps

[OndÅej Jirman]

On Mon, Feb 24, 2020 at 06:56:18PM +0100, Daniel Lezcano wrote:
> On 24/02/2020 18:39, OndÅej Jirman wrote:
> > On Mon, Feb 24, 2020 at 06:23:28PM +0100, megous hlavni wrote:
> > 
> > To be more clear, new temperatures are available from the thermal sensor driver
> > at the rate of 4 per second, which should be enough to do quick adjustments to
> > the thermal zone/cooling device even for quick temperature rises.
> > 
> > https://elixir.bootlin.com/linux/v5.6-rc3/source/drivers/thermal/sun8i_thermal.c#L442
> > 
> > There's no slow/fast period depending on whether the cooling is active.
> > It's always fast and no polling of the thermal sensor is needed.
> 
> Thanks for the clarification. All sensors have their specificity.
> 
> Does the sensor allow to create a threshold temperature where an
> interrupt fires when crossing the boundary? That would be interesting
> for performance and energy saving to disable the interrupts until
> 'cpu0_hot' is reached, no?

I think so. I don't think it would affect this binding though. It would still
require no polling, and thermal driver would probably just have to be updated
to get the relevant information about trip points from the thermal zone and
notify it of changes/trip point crossing.

I don't think it would affect performance or energy saving much though.
4 interrupts per second is barely noticeable, and there are much bigger
fish to fry, when it comes to power savings on A83T at this point.

thank you and regards,
	o.


> -- 
>  <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
>