diff mbox

[2/2] drm/i915: Remove the WaDisableBackToBackFlipFix w/a for Haswell

Message ID 1349372964-11889-2-git-send-email-damien.lespiau@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Damien Lespiau Oct. 4, 2012, 5:49 p.m. UTC
From: Damien Lespiau <damien.lespiau@intel.com>

This workaround is only valid for IVB and VLV and the write triggers an
error on HSW.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
---
 drivers/gpu/drm/i915/intel_pm.c |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)

Comments

Paulo Zanoni Oct. 4, 2012, 6:02 p.m. UTC | #1
2012/10/4 Damien Lespiau <damien.lespiau@gmail.com>:
> From: Damien Lespiau <damien.lespiau@intel.com>
>
> This workaround is only valid for IVB and VLV and the write triggers an
> error on HSW.

In other words: the register does not exist anymore.

It would be nice if we could volunteer someone to check the other
workarounds on the same function. The registers may exist but the WAs
may not be needed anymore.

Reviewed-by: Paulo Zanoni <paulo.r.zanonI@intel.com>

>
> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
> Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> ---
>  drivers/gpu/drm/i915/intel_pm.c |    4 ----
>  1 files changed, 0 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
> index 3d219ec..d0403e8 100644
> --- a/drivers/gpu/drm/i915/intel_pm.c
> +++ b/drivers/gpu/drm/i915/intel_pm.c
> @@ -3479,10 +3479,6 @@ static void haswell_init_clock_gating(struct drm_device *dev)
>         I915_WRITE(_3D_CHICKEN3,
>                    _MASKED_BIT_ENABLE(_3D_CHICKEN_SF_DISABLE_OBJEND_CULL));
>
> -       I915_WRITE(IVB_CHICKEN3,
> -                  CHICKEN3_DGMG_REQ_OUT_FIX_DISABLE |
> -                  CHICKEN3_DGMG_DONE_FIX_DISABLE);
> -
>         /* Apply the WaDisableRHWOOptimizationForRenderHang workaround. */
>         I915_WRITE(GEN7_COMMON_SLICE_CHICKEN1,
>                    GEN7_CSC1_RHWO_OPT_DISABLE_IN_RCC);
> --
> 1.7.7.5
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Lespiau, Damien Oct. 5, 2012, 1:28 p.m. UTC | #2
On Thu, Oct 4, 2012 at 7:02 PM, Paulo Zanoni <przanoni@gmail.com> wrote:
> It would be nice if we could volunteer someone to check the other
> workarounds on the same function. The registers may exist but the WAs
> may not be needed anymore.

I've done a pass this morning and with the new patch posted list, I
think we're good.
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 3d219ec..d0403e8 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -3479,10 +3479,6 @@  static void haswell_init_clock_gating(struct drm_device *dev)
 	I915_WRITE(_3D_CHICKEN3,
 		   _MASKED_BIT_ENABLE(_3D_CHICKEN_SF_DISABLE_OBJEND_CULL));
 
-	I915_WRITE(IVB_CHICKEN3,
-		   CHICKEN3_DGMG_REQ_OUT_FIX_DISABLE |
-		   CHICKEN3_DGMG_DONE_FIX_DISABLE);
-
 	/* Apply the WaDisableRHWOOptimizationForRenderHang workaround. */
 	I915_WRITE(GEN7_COMMON_SLICE_CHICKEN1,
 		   GEN7_CSC1_RHWO_OPT_DISABLE_IN_RCC);