On Mon, 8 Aug 2011 09:30:10 -0700, Jesse Barnes wrote: > Yep, looks fine. The only think we might want to sprinkle about are > checks for panel off so we can avoid visible corruption if we whack > timing or fb stuff while the panel is on. Yeah, could do. Would be nice to somehow get the LVDS code ripped out of the middle of intel_display.c; everything in intel_lvds.c is nicely bracketed by prepare/commit, which turn the panel off and back on. -- keith.packard@intel.com