[lm-sensors] [PATCH 22.214.171.124] f71882.c driver, Added PWM/FAN control.
Mark van Doesburg
mark.vandoesburg at hetnet.nl
Wed Jun 11 21:41:29 CEST 2008
Your proposal sounds fine to me. I wasn't too happy with the fan?_target
pwm? conflict either. So I will implement the following:
1. Remove fan?_target.
2. Add fan?_full_speed.
3. Interpret pwm? as the duty cycle when in duty cycle mode.
4. Scale pwm? from 0..255 to 0..fan?_full_speed when in rpm mode.
5. Change the behavior of pwm?_enable not to change pwm vs. duty
5. Add module option to select rpm/duty cycle mode.
6. Add pwm?_auto*
This will take some time of course. (Although being able to read the
temperature of my "Asus EAH3650 silent" would speed things up ;-)
My BIOS doesn't put a usefull value in the full speed register. There is
also no way to set the case fan to anything but a fixed PWM value. Then
again I build the computer from seperate components, I can imagine a
fully assembled system could do more.
Mark van Doesburg.
More information about the lm-sensors