<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><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#c38">Comment # 38</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 <span class="vcard"><a class="email" href="mailto:martin.peres@ensi-bourges.fr" title="Martin Peres <martin.peres@ensi-bourges.fr>"> <span class="fn">Martin Peres</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=73338#c37">comment #37</a>)
<span class="quote">> Created <span class=""><a href="attachment.cgi?id=106588" name="attach_106588" title="experimental patch for CI (BONAIRE tested) for enabling fan control with hwmon interface">attachment 106588</a> <a href="attachment.cgi?id=106588&action=edit" title="experimental patch for CI (BONAIRE tested) for enabling fan control with hwmon interface">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=73338&attachment=106588'>[review]</a> [review]
> experimental patch for CI (BONAIRE tested) for enabling fan control with
> hwmon interface</span >

Good, I left some comments on the patch :)

<span class="quote">> 
> (In reply to <a href="show_bug.cgi?id=73338#c35">comment #35</a>)
> > Have fun ;)

> So here's the final version of what I've done. I found existing radeon hwmon
> interface and added pwm1* sysfs files to it, so it exposes needed values
> (some like pwm1_max are RO though).

> I need someone aware of radeon driver workflow to review this patch (maybe
> Alex) and to point in a right direction.</span >

Posting the patch on radeon's ML (if there is none, then use dri-devel) and CC
Alex or any other radeon dev working on PM.

<span class="quote">> 
> When all is good, I'll request some help from public (reddit maybe, or local
> LUG) to test it extensively.</span >

When the radeon devs will think the patch is in good-enough shape, they will
likely commit it. Make sure your patch does not change the default behavior and
that should be fine.

Users willing to have manual fan management will be the testers ;)</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>