diff options
| author | Hans de Goede | 2017-11-14 14:55:18 +0100 |
|---|---|---|
| committer | Hans de Goede | 2017-11-15 09:55:12 +0100 |
| commit | 01c799c995bb8a87878b0d64edfd33014968da8e (patch) | |
| tree | 2b29f095cea1cbc33fb843be5d311819afbf27b9 /drivers/gpu/drm/i915/i915_drv.c | |
| parent | drm/i915: Re-register PMIC bus access notifier on runtime resume (diff) | |
| download | kernel-qcow2-linux-01c799c995bb8a87878b0d64edfd33014968da8e.tar.gz kernel-qcow2-linux-01c799c995bb8a87878b0d64edfd33014968da8e.tar.xz kernel-qcow2-linux-01c799c995bb8a87878b0d64edfd33014968da8e.zip | |
drm/i915: Call uncore_suspend before platform suspend handlers
Quoting Ville: "the forcewake timer might still be active until the uncore
suspend, and having active forcewakes while we've already told the GT wake
stuff to stop acting normally doesn't seem quite right to me."
Reported-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Suggested-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20171114135518.15981-3-hdegoede@redhat.com
Diffstat (limited to 'drivers/gpu/drm/i915/i915_drv.c')
| -rw-r--r-- | drivers/gpu/drm/i915/i915_drv.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c index f4588add15db..3423d873123a 100644 --- a/drivers/gpu/drm/i915/i915_drv.c +++ b/drivers/gpu/drm/i915/i915_drv.c @@ -2526,6 +2526,8 @@ static int intel_runtime_suspend(struct device *kdev) intel_runtime_pm_disable_interrupts(dev_priv); + intel_uncore_suspend(dev_priv); + ret = 0; if (IS_GEN9_LP(dev_priv)) { bxt_display_core_uninit(dev_priv); @@ -2538,6 +2540,8 @@ static int intel_runtime_suspend(struct device *kdev) if (ret) { DRM_ERROR("Runtime suspend failed, disabling it (%d)\n", ret); + intel_uncore_runtime_resume(dev_priv); + intel_runtime_pm_enable_interrupts(dev_priv); enable_rpm_wakeref_asserts(dev_priv); @@ -2545,8 +2549,6 @@ static int intel_runtime_suspend(struct device *kdev) return ret; } - intel_uncore_suspend(dev_priv); - enable_rpm_wakeref_asserts(dev_priv); WARN_ON_ONCE(atomic_read(&dev_priv->runtime_pm.wakeref_count)); |