Re: Thermal managment broken

Dominik Brodowski
2002-09-16T20:55:18Z

Hi Pavel, Andrew, list;

> First is strict bugfix, please take. Second is needed, please take it,
> too.
While I'd "vote" for the second patch "in general", there are two aspects
which should be mentioned first:

> + tz->trips.critical.temperature = CELSIUS_TO_KELVIN(critical);
> + tz->trips.hot.temperature = CELSIUS_TO_KELVIN(hot);
> + tz->trips.passive.temperature = CELSIUS_TO_KELVIN(passive);
> + tz->trips.active[0].temperature = CELSIUS_TO_KELVIN(active0);
> + tz->trips.active[1].temperature = CELSIUS_TO_KELVIN(active1);

You explicitely want to support _two_ active trip points here. While this
seems to be the amount of active trip points seen on most systems, I'd
prefer you'd do a more flexible interface [zero to ACPI_THERMAL_MAX_ACTIVE
would be best].

Additionally, it might be better to specify a "starting" and
"stopping" trip point - so that you don't have a fan going
on-off-on-off-on-off repeatedly. (IIRC someone complained about this a few
months ago on this list). But even without this "feature", I'd like to see
the user-specified trip points in the acpi patch.

Dominik

-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.