On Fri, Sep 20, 2013 at 03:21:03PM -0300, Paulo Zanoni wrote: > 2013/9/20 Paulo Zanoni <[email protected]>: > > 2013/9/20 Ville Syrjälä <[email protected]>: > >> On Thu, Sep 19, 2013 at 07:24:19PM -0300, Paulo Zanoni wrote: > >>> 2013/9/16 <[email protected]>: > >>> > From: Ville Syrjälä <[email protected]> > >>> > > >>> > We already restore planes during the modeset operation, so no need to do > >>> > another loop over the planes and try to restore them again. > >>> > >>> What about the call from intel_lid_notify()? It helps if you explain > >>> on the commit message why/how we already restore things. > >> > >> Sorry, I figured it's more or less obvious: > >> > >> for each crtc > >> -> __intel_set_mode > >> -> .crtc_enable > >> -> intel_enable_planes > >> -> for each plane on crtc > >> -> intel_plane_restore > > > > But that still doesn't explain the case where intel_lid_notify calls > > setup_hw_state directly. > > Perhaps maybe I'm just confused. > > Yeah, I was totally confused. > > Now that you resent patch 6, for patches 6-11: Reviewed-by: Paulo > Zanoni <[email protected]>
Merged them all, thanks for patches&review. > (Daniel will have to rebase some things) wiggle took care of everything ;-) Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
