Thermal state switching

[Nate Lawson]

Alexandre "Sunny" Kovalenko wrote:
> Good people,
> 
> I was looking at thermal states switching by acpi_thermal.c and it looks
> like follows (provided that temperature raises and falls slowly and
> gradually):
> 
> NONE =up> AC2 =up> AC1 =up> AC0 =down> NONE                       (1)
> 
> I do not know whether this was intentional or not and, for me, something
> along the lines of
> 
> NONE =up> AC2 =up> AC1 =up> AC0 =down> AC1 =down> AC2 =down> NONE (2)
> 
> seemed more natural.
> 
> If (2) and not (1) was indeed the desired behavior attached patch seems
> to do the job. If (1) is what was intended, I do apologize for the
> noise.
> 
> I am running -CURRENT from February 3.

The behavior should be as in #2.  If it isn't, we should fix that. 
However, I'm not sure how your patch would fix this.  It seems more 
correct in that we only set the starting time after switching coolers 
but I don't see how this affects the ACx levels.  Could you explain more?

Thanks,
-- 
Nate