Hi Grant, > > > automatic fan clock divider > > > ```````````````````````````` > > > * User sets 0 to fan_min limit > > > - low speed alarm is disabled > > > - fan clock divider not changed > > > - auto fan clock adjuster disabled > > > > Why would we want to disable the auto fan clock adjuster in this > > case? > > Oops, I just posted new driver... Bad timing -- I'll need to check, > this implies enabling bidirectional fan adjustment to do it right? No, it doesn't. > > You also missed one case, which is the main reason why the whole > > thing is (IMHO) needed: > > > > * User doesn't set any limit > > - auto fan clock adjuster enabled > > Again, this implies bidirectional adjustment (...) Again, it doesn't. Automatic fan clock adjustment is really automatic fan divider increment, in order to prevent zero speed readings when the fan actually spins. Decreasing would only possibly improve accuracy, and that's not our problem when the user did not set a proper limit. -- Jean Delvare
