Re: [PATCH] ACPI: Remove possible recursion from thermal driver

2007-05-17 Thread Len Brown
On Tuesday 15 May 2007 22:19, Alexey Starikovskiy wrote: > Please read the spec 11.1.2 and 11.1.2.3. Trip points change _only_ to > create a hysteresis loop, not by themselves. This means that we will > get 0x81 type _only_ in response to us changing active state, the > moment then we have temperat

Re: [PATCH] ACPI: Remove possible recursion from thermal driver

2007-05-15 Thread Alexey Starikovskiy
Please read the spec 11.1.2 and 11.1.2.3. Trip points change _only_ to create a hysteresis loop, not by themselves. This means that we will get 0x81 type _only_ in response to us changing active state, the moment then we have temperature and just decided to move away from this trip point, not clos

Re: [PATCH] ACPI: Remove possible recursion from thermal driver

2007-05-15 Thread Len Brown
> --- a/drivers/acpi/thermal.c > +++ b/drivers/acpi/thermal.c > @@ -1106,7 +1106,6 @@ static void acpi_thermal_notify(acpi_handle handle, u32 > event, void *data) > break; > case ACPI_THERMAL_NOTIFY_THRESHOLDS: > acpi_thermal_get_trip_points(tz); >-

[PATCH] ACPI: Remove possible recursion from thermal driver

2007-05-14 Thread Alexey Starikovskiy
Hi, This patch helps to fight infinite recursion on some ACER laptops, see bugzilla #8385. Thanks, Alex. Thermal check may result in thermal points to be moved down. Don't do thermal check again in this case. From: Alexey Starikovskiy <[EMAIL PROTECTED]> Signed-off-by: Alexey Starikovskiy <[EM