On Tue, Oct 23, 2012 at 9:29 PM, Paulo Zanoni <przanoni at gmail.com> wrote: > static void haswell_crtc_off(struct drm_crtc *crtc) > { > + struct intel_crtc *intel_crtc = to_intel_crtc(crtc); > + > + intel_crtc->cpu_transcoder = intel_crtc->pipe; > intel_ddi_put_crtc_pll(crtc); > } > I can't find the reason why you would set the cpu_transcoder in the off() function, would you mind explaining why? (or maybe the clue is in a later patch, which might mean this hunk belongs to a later patch as well). -- Damien