On Fri, Jun 07, 2013 at 09:10:23PM +0200, Daniel Vetter wrote: > On Fri, Jun 07, 2013 at 01:47:41PM +0100, Chris Wilson wrote: > > + if (intel_crtc_set_config(&set)) > > + __intel_set_mode(crtc, &crtc->mode, > > + crtc->x, crtc->y, crtc->fb); > > Won't this horribly scream in the modeset state checker again if we've > only partially restored the output state (in case the bios was really > seriuos about killing our config)? I thought that was going to be a useful scream. It should mean that we have we missing bits in the pipe_config recovery, right? -Chris -- Chris Wilson, Intel Open Source Technology Centre