Hi: * author at mordac.netroedge.com <author at mordac.netroedge.com> [2005-03-26 17:44:27 -0800]: > Update of /home/cvs/lm_sensors2/kernel/chips > In directory mordac.netroedge.com:/tmp/cvs-serv26712 > > Modified Files: > w83792d.c > Log Message: > (mds) fix negative fan values. Patch from Chunhao. Why is there a huge 'do {...} while(0)' loop in w83792d_fan()? I guess it's some kind of fan divisor auto-configuration. If that's the case, shouldn't it be 'do {...} while(1)'? Also, why not move the w83792d_update_client() call to the bottom of the loop? Regards, -- Mark M. Hoffman mhoffman at lightlink.com