<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - backlight: clamp_brightness not suitable for devices in led subsystem (kbd backlight)"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=77092#c2">Comment # 2</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - backlight: clamp_brightness not suitable for devices in led subsystem (kbd backlight)"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=77092">bug 77092</a>
              from <span class="vcard"><a class="email" href="mailto:josh@freedesktop.org" title="Josh Triplett <josh@freedesktop.org>"> <span class="fn">Josh Triplett</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=77092#c1">comment #1</a>)
<span class="quote">> Created <span class=""><a href="attachment.cgi?id=96977" name="attach_96977" title="execute clamp_brightness only for non LED devices">attachment 96977</a> <a href="attachment.cgi?id=96977&action=edit" title="execute clamp_brightness only for non LED devices">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=77092&attachment=96977'>[review]</a> [review]
> execute clamp_brightness only for non LED devices

> Related my prior comment with a patch.. i've created one, hope he is fine.</span >

This patch looks reasonable to me.

I do wonder, though, if it might make more sense to whitelist only the
subsystem used for display backlights, rather than blacklisting LED
"backlights".</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the QA Contact for the bug.</li>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>