drm/i915: Clear GDSR after reset on ILK
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 19 May 2014 16:23:26 +0000 (19:23 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 20 May 2014 19:06:33 +0000 (21:06 +0200)
Clear the reset domain after a succesful GPU reset on ilk. We already
do that on gen4, so let's try to be a bit more consistent. And if
ether render or media reset fails, we might use the leftover value
in the register to pinpoint the culprit.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_uncore.c

index b542bf6302f68ba8313590f1176c21f34f4bbbae..4ec192bc27ba66d2c1731a1366cd1ea8c655e231 100644 (file)
@@ -1002,8 +1002,14 @@ static int ironlake_do_reset(struct drm_device *dev)
 
        I915_WRITE(MCHBAR_MIRROR_BASE + ILK_GDSR,
                   ILK_GRDOM_MEDIA | ILK_GRDOM_RESET_ENABLE);
-       return wait_for((I915_READ(MCHBAR_MIRROR_BASE + ILK_GDSR) &
-                        ILK_GRDOM_RESET_ENABLE) == 0, 500);
+       ret = wait_for((I915_READ(MCHBAR_MIRROR_BASE + ILK_GDSR) &
+                       ILK_GRDOM_RESET_ENABLE) == 0, 500);
+       if (ret)
+               return ret;
+
+       I915_WRITE(MCHBAR_MIRROR_BASE + ILK_GDSR, 0);
+
+       return 0;
 }
 
 static int gen6_do_reset(struct drm_device *dev)