On 11/02/2015 18:21, Lespiau, Damien wrote:
I have no idea how that crept in, but we need to do the write from the
ring and this is a masked register. Two fixes in 1!

Cc: Nick Hoath <[email protected]>
Signed-off-by: Damien Lespiau <[email protected]>

Reviewed-by: Nick Hoath <[email protected]>

---
  drivers/gpu/drm/i915/intel_ringbuffer.c | 10 +++-------
  1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c 
b/drivers/gpu/drm/i915/intel_ringbuffer.c
index 8735e56..acc1669 100644
--- a/drivers/gpu/drm/i915/intel_ringbuffer.c
+++ b/drivers/gpu/drm/i915/intel_ringbuffer.c
@@ -959,13 +959,9 @@ static int gen9_init_workarounds(struct intel_engine_cs 
*ring)

        if (INTEL_REVID(dev) == SKL_REVID_A0 ||
            INTEL_REVID(dev) == SKL_REVID_B0) {
-               /*
-               * WaDisableDgMirrorFixInHalfSliceChicken5:skl
-               * This is a pre-production w/a.
-               */
-               I915_WRITE(GEN9_HALF_SLICE_CHICKEN5,
-                       I915_READ(GEN9_HALF_SLICE_CHICKEN5) &
-                       ~GEN9_DG_MIRROR_FIX_ENABLE);
+               /* WaDisableDgMirrorFixInHalfSliceChicken5:skl */
+               WA_CLR_BIT_MASKED(GEN9_HALF_SLICE_CHICKEN5,
+                                 GEN9_DG_MIRROR_FIX_ENABLE);
        }

        if (INTEL_REVID(dev) >= SKL_REVID_C0) {


_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to