On Wed, Feb 24, 2016 at 07:57:43PM +0200, Imre Deak wrote: > During system suspend we need to first disable power wells then > unitialize the display core. In case power well support is disabled we > did this in the wrong order, so fix this up. > > Fixes: d314cd43 ("drm/i915: fix handling of the disable_power_well module option") > CC: stable@xxxxxxxxxxxxxxx > CC: Patrik Jakobsson <patrik.jakobsson@xxxxxxxxxxxxxxx> > Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx> Reviewed-by: Patrik Jakobsson <patrik.jakobsson@xxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_runtime_pm.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c b/drivers/gpu/drm/i915/intel_runtime_pm.c > index e232976..8276dc2 100644 > --- a/drivers/gpu/drm/i915/intel_runtime_pm.c > +++ b/drivers/gpu/drm/i915/intel_runtime_pm.c > @@ -2319,15 +2319,15 @@ void intel_power_domains_init_hw(struct drm_i915_private *dev_priv, bool resume) > */ > void intel_power_domains_suspend(struct drm_i915_private *dev_priv) > { > - if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) > - skl_display_core_uninit(dev_priv); > - > /* > * Even if power well support was disabled we still want to disable > * power wells while we are system suspended. > */ > if (!i915.disable_power_well) > intel_display_power_put(dev_priv, POWER_DOMAIN_INIT); > + > + if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) > + skl_display_core_uninit(dev_priv); > } > > /** > -- > 2.5.0 > -- --- Intel Sweden AB Registered Office: Knarrarnasgatan 15, 164 40 Kista, Stockholm, Sweden Registration Number: 556189-6027 -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html