Re: [lm-sensors] The new thermal management sysfs class, and hwmon

Zhang, Rui
2008-02-22T08:35:27Z

Hi, Hans,

On Fri, 2008-02-22 at 16:00 +0800, Hans de Goede wrote:

> I think all that is really needed and asked for is for the new thermal
> ACPI
> code to:
> 1) provide temp readings in the same format as hwmon (so milli degrees
> celcius,
> not degrees celcius
Agree.
> 2) provide a hwmon interface so that tools like (but not limited too):
> * net-snmp
> * mrtg
> * sensors
> * sensors-applet (gnome)
> * xfce-sensors-applet
> * ksysguard
> * ksensors
> * gkrellm
>
> Can provide temp and fan readings without having to be modified.
hmm, for fan device, maybe something like this?
pwm[1-*]_enable = 1 : manual fan control (using pwm[1-*])
2+: automatic fan control (by acpi thermal driver)
pwm[1-*] = 0 : fan is off.
pwm[1-*] = 255: fan is on.
pwm[1-*] has only two valid values as ACPI fan only support
two states, ON/OFF. and it doesn't need fan[1-*]_input because the fan
speed is not available.
Yes, it can work for ACPI fan although I don't think the existing pwm
hwmon I/F maps well to what we need and it seems like a "forced fit" to
use it. Any better ideas? :)

Thanks,
Rui

> To be clear I am suggesting that the new code exports both:
> 1) the current full fledged thermal acpi zone interface, as designed
> for
> both reading and configuration
> 2) a read only hwmon interface for reporting temps and fans