On Tue, Oct 2, 2012 at 9:51 PM, Paulo Zanoni <przanoni at gmail.com> wrote: > From: Paulo Zanoni <paulo.r.zanoni at intel.com> > > And the right time is exactly after/before changing PIPE_CONF. See the > documentation about the mode set sequence. > > This code is not inside any encoder-specific callback because > DDI_FUNC_CTL is part of the pipe, so it is used by all encoders. Apart from the reordering of this patch with [PATCH 07/47] drm/i915: add haswell_crtc_mode_set, Reviewed-by: Damien Lespiau <damien.lespiau at intel.com> -- Damien