<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Cannot control backlight intensity on Powerbook"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=31122#c16">Comment # 16</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Cannot control backlight intensity on Powerbook"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=31122">bug 31122</a>
              from <span class="vcard"><a class="email" href="mailto:imirkin@alum.mit.edu" title="Ilia Mirkin <imirkin@alum.mit.edu>"> <span class="fn">Ilia Mirkin</span></a>
</span></b>
        <pre>(In reply to A. Chaud from <a href="show_bug.cgi?id=31122#c15">comment #15</a>)
<span class="quote">> Regarding the 0x535.  It's a workaround for <a class="bz_bug_link 
          bz_status_REOPENED "
   title="REOPENED - Backlight off after suspend-to-disk on Powerbook"
   href="show_bug.cgi?id=31120">bug 31120</a>.  The lower half of
> the register gets set to 0x7fff upon waking from suspend which disables the
> backlight. This changes it back to a working value.</span >

There should be a way to save a value on suspend and restore that saved value
on resume.

BTW, I didn't check whether you're still doing this in the latest patch, but a
few things I remember noticing:

(a) foo | 0x80000000, not +
(b) use ARRAY_SIZE, although I think you got rid of that array anyways</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>