drm/i915: disable GT power saving early during system suspend
authorImre Deak <imre.deak@intel.com>
Mon, 12 May 2014 15:35:05 +0000 (18:35 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 22 May 2014 19:53:27 +0000 (21:53 +0200)
Atm, we disable GT power saving during the end of the suspend sequence
in i915_save_state(). Doing the disabling at that point seems arbitrary.
One reason to disable it early though is to have a quiescent HW state
before we do anything else (for example save registers). So move the
disabling earlier, which also takes care canceling of the deferred RPS
enabling work done by intel_disable_gt_powersave().

Note that after the move we'll call intel_disable_gt_powersave() only
in case modeset is enabled, but that's anyway the only case where we
have it enabled in the first place.

Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Robert Beckett <robert.beckett@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_drv.c
drivers/gpu/drm/i915/i915_suspend.c

index acb93fad4df74964d1d86626b59caba8a2d00d41..4619c9e519076828b5449732fa39cea3c363224c 100644 (file)
@@ -524,10 +524,11 @@ static int i915_drm_freeze(struct drm_device *dev)
                        return error;
                }
 
-               cancel_delayed_work_sync(&dev_priv->rps.delayed_resume_work);
-
                drm_irq_uninstall(dev);
                dev_priv->enable_hotplug_processing = false;
+
+               intel_disable_gt_powersave(dev);
+
                /*
                 * Disable CRTCs directly since we want to preserve sw state
                 * for _thaw.
index 56785e8fb2eb5f7e209b8713f3815b4c5ecbcf22..043123c77a1f4e0f3d92143f3804eb614c94fd24 100644 (file)
@@ -328,8 +328,6 @@ int i915_save_state(struct drm_device *dev)
                }
        }
 
-       intel_disable_gt_powersave(dev);
-
        /* Cache mode state */
        if (INTEL_INFO(dev)->gen < 7)
                dev_priv->regfile.saveCACHE_MODE_0 = I915_READ(CACHE_MODE_0);