drm/i915: Only poll DW3_A when init DDI PHY for ports B and C.
authorRodrigo Vivi <rodrigo.vivi@intel.com>
Thu, 17 Nov 2016 19:17:36 +0000 (11:17 -0800)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Fri, 2 Dec 2016 20:16:25 +0000 (12:16 -0800)
According to Bspec we need to
"Poll for PORT_REF_DW3_A grc_done == 1b"
only on ports B and C initialization sequence when
copying rcomp from port A.

So let's follow the spec and only poll for that case
and not on every port A initialization.

v2: Also remove the grc_done check from bxt_ddi_phy_is_enabled()
    otherwise it might believe it is disabled and force it to re program.

Cc: Imre Deak <imre.deak@intel.com>
Cc: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1479410256-25735-1-git-send-email-rodrigo.vivi@intel.com
drivers/gpu/drm/i915/intel_dpio_phy.c

index 46e38a05b3ac7040e3e2e9851f6051df8c8adbdf..09b6709297867d6751957dc12541812fe3f30344 100644 (file)
@@ -328,14 +328,6 @@ bool bxt_ddi_phy_is_enabled(struct drm_i915_private *dev_priv,
                return false;
        }
 
-       if (phy_info->rcomp_phy == -1 &&
-           !(I915_READ(BXT_PORT_REF_DW3(phy)) & GRC_DONE)) {
-               DRM_DEBUG_DRIVER("DDI PHY %d powered, but GRC isn't done\n",
-                                phy);
-
-               return false;
-       }
-
        if (!(I915_READ(BXT_PHY_CTL_FAMILY(phy)) & COMMON_RESET_DIS)) {
                DRM_DEBUG_DRIVER("DDI PHY %d powered, but still in reset\n",
                                 phy);
@@ -441,6 +433,9 @@ static void _bxt_ddi_phy_init(struct drm_i915_private *dev_priv,
 
        if (phy_info->rcomp_phy != -1) {
                uint32_t grc_code;
+
+               bxt_phy_wait_grc_done(dev_priv, phy_info->rcomp_phy);
+
                /*
                 * PHY0 isn't connected to an RCOMP resistor so copy over
                 * the corresponding calibrated value from PHY1, and disable
@@ -464,10 +459,6 @@ static void _bxt_ddi_phy_init(struct drm_i915_private *dev_priv,
        val = I915_READ(BXT_PHY_CTL_FAMILY(phy));
        val |= COMMON_RESET_DIS;
        I915_WRITE(BXT_PHY_CTL_FAMILY(phy), val);
-
-       if (phy_info->rcomp_phy == -1)
-               bxt_phy_wait_grc_done(dev_priv, phy);
-
 }
 
 void bxt_ddi_phy_uninit(struct drm_i915_private *dev_priv, enum dpio_phy phy)