<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - Fan speed in idle at 40% with radeonsi and at 18% with catalyst" href="https://bugs.freedesktop.org/show_bug.cgi?id=73338#c27">Comment # 27</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - Fan speed in idle at 40% with radeonsi and at 18% with catalyst" href="https://bugs.freedesktop.org/show_bug.cgi?id=73338">bug 73338</a> from <a class="email" href="mailto:martin.peres@ensi-bourges.fr" title="Martin Peres <martin.peres@ensi-bourges.fr>"> Martin Peres</a> <pre>(In reply to <a href="show_bug.cgi?id=73338#c26">comment #26</a>) > (In reply to <a href="show_bug.cgi?id=73338#c25">comment #25</a>) > > > I am the Nouveau developer who added fan management support. I would really > > advice you follow the hwmon recommendations and name this file pwm1. > > Sorry, I'm novice, didn't know about these. Don't feel bad, you cannot know possibly know everything ;) > Though it's not a hw interface > really, the exchange is done through io-mapped SMC registers... Not sure how more hw-oriented the interface could get :D That is still irrelevant to the discussion though. HWMON is software interface for the userspace. > > Ok, I'll make necessary changes. I assume it will allow lm-sensors to pick > it up automatically, is it correct? Yes, it will! Just to make sure we understood each others: - pwm1 (RW) exposes the current pwm value (what you called power) - pwm1_min(RW) min exposes the mininimum power that should be used by the fan - pwm1_max(RW) min exposes the maximum power that should be used by the fan - pwm1_enable (RW) exposes the current fan mode. 0 = DISABLED, 1 = MANUAL (the user writes the fan power he/she wants to pwm1), 2 = AUTOMATIC (linear fan management?). Here is Nouveau's documentation on fan management: <a href="http://cgit.freedesktop.org/nouveau/linux-2.6/tree/Documentation/thermal/nouveau_thermal">http://cgit.freedesktop.org/nouveau/linux-2.6/tree/Documentation/thermal/nouveau_thermal</a> I know it is a lot of changes I am asking you to do, but consistency across drivers is great feature!</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>