[Intel-gfx] [PATCH 36/37] drm/i915: add warning when using old bits on Haswell/LPT
Rodrigo Vivi
rodrigo.vivi at gmail.com
Mon Mar 26 19:49:15 CEST 2012
Reviewed-by: Rodrigo Vivi <rodrigo.vivi at gmail.com>
On Wed, Mar 21, 2012 at 10:10 PM, Eugeni Dodonov
<eugeni.dodonov at intel.com> wrote:
> Those have different functionality on Haswell architecture, so let's
> trigger a warning message when we are going through a path we should not
> go into on Haswell.
>
> This patch is here for make debugging and log tracing easier, it should
> go away in the future, we we'll stop hitting those code paths.
>
> Signed-off-by: Eugeni Dodonov <eugeni.dodonov at intel.com>
> ---
> drivers/gpu/drm/i915/intel_display.c | 59 +++++++++++++++++++++++++++-------
> 1 file changed, 47 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index a37e0b7..10b92e8 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -847,9 +847,16 @@ static void assert_fdi_tx(struct drm_i915_private *dev_priv,
> u32 val;
> bool cur_state;
>
> - reg = FDI_TX_CTL(pipe);
> - val = I915_READ(reg);
> - cur_state = !!(val & FDI_TX_ENABLE);
> + if (IS_HASWELL(dev_priv->dev)) {
> + DRM_ERROR("Attempting to check FDI_TX_CTL on Haswell, using DDI instead\n");
> + reg = DDI_FUNC_CTL(pipe);
> + val = I915_READ(reg);
> + cur_state = !!(val & PIPE_DDI_FUNC_ENABLE);
> + } else {
> + reg = FDI_TX_CTL(pipe);
> + val = I915_READ(reg);
> + cur_state = !!(val & FDI_TX_ENABLE);
> + }
> WARN(cur_state != state,
> "FDI TX state assertion failure (expected %s, current %s)\n",
> state_string(state), state_string(cur_state));
> @@ -864,9 +871,14 @@ static void assert_fdi_rx(struct drm_i915_private *dev_priv,
> u32 val;
> bool cur_state;
>
> - reg = FDI_RX_CTL(pipe);
> - val = I915_READ(reg);
> - cur_state = !!(val & FDI_RX_ENABLE);
> + if (IS_HASWELL(dev_priv->dev) && pipe > 0) {
> + DRM_ERROR("Attempting to enable FDI_RX on Haswell pipe > 0\n");
> + return;
> + } else {
> + reg = FDI_RX_CTL(pipe);
> + val = I915_READ(reg);
> + cur_state = !!(val & FDI_RX_ENABLE);
> + }
> WARN(cur_state != state,
> "FDI RX state assertion failure (expected %s, current %s)\n",
> state_string(state), state_string(cur_state));
> @@ -884,6 +896,11 @@ static void assert_fdi_tx_pll_enabled(struct drm_i915_private *dev_priv,
> if (dev_priv->info->gen == 5)
> return;
>
> + if (IS_HASWELL(dev_priv->dev)) {
> + DRM_ERROR("Attempting to check FDI_TX_PLL on Haswell, aborting\n");
> + return;
> + }
> +
> reg = FDI_TX_CTL(pipe);
> val = I915_READ(reg);
> WARN(!(val & FDI_TX_PLL_ENABLE), "FDI TX PLL assertion failure, should be active but is disabled\n");
> @@ -895,6 +912,10 @@ static void assert_fdi_rx_pll_enabled(struct drm_i915_private *dev_priv,
> int reg;
> u32 val;
>
> + if (IS_HASWELL(dev_priv->dev) && pipe > 0) {
> + DRM_ERROR("Attempting to enable FDI on Haswell with pipe > 0\n");
> + return;
> + }
> reg = FDI_RX_CTL(pipe);
> val = I915_READ(reg);
> WARN(!(val & FDI_RX_PLL_ENABLE), "FDI RX PLL assertion failure, should be active but is disabled\n");
> @@ -1000,6 +1021,11 @@ static void assert_pch_refclk_enabled(struct drm_i915_private *dev_priv)
> u32 val;
> bool enabled;
>
> + if (HAS_PCH_LPT(dev_priv->dev)) {
> + DRM_ERROR("LPT does not has PCH refclk, skipping check\n");
> + return;
> + }
> +
> val = I915_READ(PCH_DREF_CONTROL);
> enabled = !!(val & (DREF_SSC_SOURCE_MASK | DREF_NONSPREAD_SOURCE_MASK |
> DREF_SUPERSPREAD_SOURCE_MASK));
> @@ -1323,6 +1349,10 @@ static void intel_enable_transcoder(struct drm_i915_private *dev_priv,
> assert_fdi_tx_enabled(dev_priv, pipe);
> assert_fdi_rx_enabled(dev_priv, pipe);
>
> + if (IS_HASWELL(dev_priv->dev) && pipe > 0) {
> + DRM_ERROR("Attempting to enable FDI on Haswell with pipe > 0\n");
> + return;
> + }
> reg = TRANSCONF(pipe);
> val = I915_READ(reg);
> pipeconf_val = I915_READ(PIPECONF(pipe));
> @@ -3308,13 +3338,18 @@ static void ironlake_fdi_pll_enable(struct drm_crtc *crtc)
> udelay(200);
>
> /* Enable CPU FDI TX PLL, always on for Ironlake */
> - reg = FDI_TX_CTL(pipe);
> - temp = I915_READ(reg);
> - if ((temp & FDI_TX_PLL_ENABLE) == 0) {
> - I915_WRITE(reg, temp | FDI_TX_PLL_ENABLE);
> + if (IS_HASWELL(dev)) {
> + DRM_ERROR("Skipping enablement of FDI_TX_PLL on Haswell\n");
> + return;
> + } else {
> + reg = FDI_TX_CTL(pipe);
> + temp = I915_READ(reg);
> + if ((temp & FDI_TX_PLL_ENABLE) == 0) {
> + I915_WRITE(reg, temp | FDI_TX_PLL_ENABLE);
>
> - POSTING_READ(reg);
> - udelay(100);
> + POSTING_READ(reg);
> + udelay(100);
> + }
> }
> }
>
> --
> 1.7.9.2
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
GPG: 0x905BE242 @ wwwkeys.pgp.net
More information about the Intel-gfx
mailing list