drm/i915: Enable the PCH PLL for all generations after link training
authorChris Wilson <chris@chris-wilson.co.uk>
Sun, 13 May 2012 08:54:09 +0000 (09:54 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Sat, 19 May 2012 21:10:01 +0000 (23:10 +0200)
Hidden away within one chipset specific path was the necessary logic to
turn on the PLL. This needs to be done everywhere in order for us to
drive any display! As such as soon as we tested on a non-CougarPoint
chipset, we failed to bring up any DisplayPorts and generated a nice set
of assertion failures in the process. At least one part of our logic is
working, the part that assumes that we have no idea what we are doing.

Reported-by: guang.a.yang@intel.com
References: https://bugs.freedesktop.org/show_bug.cgi?id=49712
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c

index 9dc42bf557bed9f80b0d6adaff59a7ac6001c9d5..f3d39f4f6909d9783afec04e224c344cb099fd9c 100644 (file)
@@ -2796,14 +2796,14 @@ static void ironlake_pch_enable(struct drm_crtc *crtc)
        /* For PCH output, training FDI link */
        dev_priv->display.fdi_link_train(crtc);
 
+       intel_enable_pch_pll(intel_crtc);
+
        if (HAS_PCH_LPT(dev)) {
                DRM_DEBUG_KMS("LPT detected: programming iCLKIP\n");
                lpt_program_iclkip(crtc);
        } else if (HAS_PCH_CPT(dev)) {
                u32 sel;
 
-               intel_enable_pch_pll(intel_crtc);
-
                temp = I915_READ(PCH_DPLL_SEL);
                switch (pipe) {
                default: