[PATCH 2.6] I2C chipset drivers use temp_hyst[1-3] V2

Mark M. Hoffman mhoffman at lightlink.com
Thu Nov 20 07:33:09 CET 2003


* Jean Delvare <khali at linux-fr.org> [2003-11-18 20:25:56 +0100]:

(...)

> Summary of the changes:
> adm1021.c: No changes, that chipset uses a real min/max model.
> eeeprom.c: No changes (obviously).
> it87.c:    Remove buggy comments (obviously taken from via686a) about
>            max and min temperature limits being over and hyst. This
>            isn't the case for this driver (min/max model).
> lm75.c:    Simple sysfs file name change (temp_min to temp_hyst).
> lm78.c:    Simple sysfs file name change (temp_min to temp_hyst).
> lm85.c:    No changes needed (min/max model).
> via686a.c: Rename functions and macros from min/max to hyst/over, what
>            it really is. Remove unnecessary comments. Rename sysfs
>            files from temp_min[1-3] to temp_hyst[1-3].
> w83781d.c: Rename variables from temp_min* to temp_hyst* (needed so
>            that the macros keep working). Update macro calls
>            accordingly. Fix writing temp to max and hyst being
>            swapped.

(...)

I just did a quick smoke test... the patch looks ok for ASB100 (w83781d).
Writing to temp_max? and temp_hyst? seems to behave as expected.  I'll
try to do more testing (including lm78) soon.

OBTW: Temps for w83781d are in 1/100 C... I thought that was fixed
(i.e. 1/1000 C) already?  I tested this with straight 2.6.0-test9.

Regards,

-- 
Mark M. Hoffman
mhoffman at lightlink.com



More information about the lm-sensors mailing list