drm/i915: No need to put forcewake after a reset
authorMika Kuoppala <mika.kuoppala@linux.intel.com>
Wed, 5 Mar 2014 16:08:18 +0000 (18:08 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 5 Mar 2014 20:30:41 +0000 (21:30 +0100)
As we now have intel_uncore_forcewake_reset() no need
to do explicit put after reset.

v2: rebase

Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
Reviewed-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_uncore.c

index 6ca24acd8b3ea2015f5ceaa25d05304b80b76ff1..00320fd10322d728edf0e679b43c78e9eff526d2 100644 (file)
@@ -952,6 +952,7 @@ static int gen6_do_reset(struct drm_device *dev)
        struct drm_i915_private *dev_priv = dev->dev_private;
        int     ret;
        unsigned long irqflags;
+       u32 fw_engine = 0;
 
        /* Hold uncore.lock across reset to prevent any register access
         * with forcewake not set correctly
@@ -971,25 +972,21 @@ static int gen6_do_reset(struct drm_device *dev)
 
        intel_uncore_forcewake_reset(dev);
 
-       /* If reset with a user forcewake, try to restore, otherwise turn it off */
+       /* If reset with a user forcewake, try to restore */
        if (IS_VALLEYVIEW(dev)) {
                if (dev_priv->uncore.fw_rendercount)
-                       dev_priv->uncore.funcs.force_wake_get(dev_priv, FORCEWAKE_RENDER);
-               else
-                       dev_priv->uncore.funcs.force_wake_put(dev_priv, FORCEWAKE_RENDER);
+                       fw_engine |= FORCEWAKE_RENDER;
 
                if (dev_priv->uncore.fw_mediacount)
-                       dev_priv->uncore.funcs.force_wake_get(dev_priv, FORCEWAKE_MEDIA);
-               else
-                       dev_priv->uncore.funcs.force_wake_put(dev_priv, FORCEWAKE_MEDIA);
+                       fw_engine |= FORCEWAKE_MEDIA;
        } else {
                if (dev_priv->uncore.forcewake_count)
-                       dev_priv->uncore.funcs.force_wake_get(dev_priv, FORCEWAKE_ALL);
-               else
-                       dev_priv->uncore.funcs.force_wake_put(dev_priv, FORCEWAKE_ALL);
+                       fw_engine = FORCEWAKE_ALL;
        }
 
-       /* Restore fifo count */
+       if (fw_engine)
+               dev_priv->uncore.funcs.force_wake_get(dev_priv, fw_engine);
+
        if (IS_GEN6(dev) || IS_GEN7(dev))
                dev_priv->uncore.fifo_count =
                        __raw_i915_read32(dev_priv, GTFIFOCTL) &