<div>My change tries to detect the lack of initialization by A) finding no VBT, and B) finding 0 values in these registers.</div><div><br></div><div>But what if there is a VBIOS out there that really wants these values to be 0?  I provide for that case by checking for VBT.</div>
<div><br></div><div>Is this a reasonable case?  If not, I have no problem moving the check to init_vbt_default().</div><div><br></div><div>bryan.</div><div><br><div class="gmail_quote">On Thu, Oct 7, 2010 at 3:55 PM, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On Thu, 7 Oct 2010 15:48:14 -0700, Bryan Freed <<a href="mailto:bfreed@chromium.org">bfreed@chromium.org</a>> wrote:<br>

> The time between start of the pixel clock and backlight enable is a basic<br>
> panel timing constraint.  If no VBIOS Table is found, and the Panel Power<br>
> On/Off registers are found to be 0, assume we are booting without VBIOS<br>
> initialization and set these registers to something reasonable.<br>
<br>
</div>IIRC, the panel sequence registers are meant to be stored in the VBIOS. So<br>
if we add the parsing of those to the driver and add the defaults to<br>
init_vbt_default() then we can check whether PP_ON_DELAYS is valid upon<br>
device init (module load and resume) and fixup in case the BIOS does not.<br>
-Chris<br>
<font color="#888888"><br>
--<br>
Chris Wilson, Intel Open Source Technology Centre<br>
</font></blockquote></div><br></div>