On Thu, 1 Mar 2012 14:18:58 -0800, Jesse Barnes <jbarnes at virtuousgeek.org> wrote: > @@ -2427,6 +2448,8 @@ intel_dp_init(struct drm_device *dev, int output_reg) > intel_connector_attach_encoder(intel_connector, intel_encoder); > drm_sysfs_connector_add(connector); > > + intel_encoder->base.crtc = intel_dp_get_crtc(&intel_encoder->base); intel_encoder->base.crtc = dev_priv->pipe_to_crtc_mapping[intel_dp_get_pipe(intel_dp)] As the pipe-to-crtc mapping is already established by this point, and indeed used in the LVDS chunk, you may as well use it here. -Chris -- Chris Wilson, Intel Open Source Technology Centre