[lm-sensors] [PATCH] Add MAX6650 support
hjk at linutronix.de
Mon Mar 5 12:34:21 CET 2007
I updated the driver once again. Basically, I made these changes:
* "mode" is now not only a module parameter, but also a sysfs file. This is
needed to simply switch the fan fully on/off. Depending on the other
settings, this might not be possible with the "speed" attribute alone.
* Setting "speed" really only sets the speed value. The original
implementation changed "mode" between "off" and "closed loop" when speed was
0 or >0, respectively. I consider this bad style, a driver should do what
it's told to do and not make guesses about what the user might wish.
* Updated documentation.
I left the other values as module parameters. I think it's unlikely that
somebody wants to set these to different values if he has more than one chip.
Of course we could make all of them sysfs files, too, but I think this is not
necessary. Let's wait and see if somebody actually wants that.
Signed-off-by: Hans J. Koch <hjk at linutronix.de>
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 22299 bytes
Desc: not available
Url : http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20070305/63bbcc3e/attachment.bin
More information about the lm-sensors