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 - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html