On Fri, 30 Sep 2011 18:27:28 +0200, Daniel Vetter <daniel@xxxxxxxx> wrote: > Can you elaborate a bit why this is no longer needed? Jesse seems to have > introduced this spefically for edp panels in d209848d61794968. If this > becomes rendundant due to your panel power sequencing fixes, maybe move it > down the series? The only reason for this delay is to avoid some potential noise on the screen during mode setting, all of the power sequencing delays are already correctly managed by the driver. As far as I can tell, we're trading off the small potential for noise on the screen with an extra 300ms delay at boot time. In the eDP spec, there's a delay listed (T8) between the start of video data and turning on the backlight. There are no values presented and the instructions are simply that the source is supposed to wait until the display is stable. I'm not sure how we'd ever know that the display is stable. -- keith.packard@xxxxxxxxx
Attachment:
pgpWdB9j9wRid.pgp
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
