diff options
author | Lyude <cpaul@redhat.com> | 2016-03-11 10:57:01 -0500 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-03-30 02:23:41 -0400 |
commit | a16b7658f4e0d4aec9bc3e75a5f0cc3f7a3a0422 (patch) | |
tree | 0a9f827ee533a35c90904718b414a5ac8bebae56 | |
parent | a7442b93cf32c1e1ddb721a26cd1f92302e2a222 (diff) |
drm/i915: Call intel_dp_mst_resume() before resuming displays
Since we need MST devices ready before we try to resume displays,
calling this after intel_display_resume() can result in some issues with
various laptop docks where the monitor won't turn back on after
suspending the system.
This order was originally changed in
commit e7d6f7d70829 ("drm/i915: resume MST after reading back hw state")
In order to fix some unclaimed register errors, however the actual cause
of those has since been fixed.
CC: stable@vger.kernel.org
Signed-off-by: Lyude <cpaul@redhat.com>
[danvet: Resolve conflicts with locking changes.]
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
-rw-r--r-- | drivers/gpu/drm/i915/i915_drv.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c index 20f8dbe7b21c..f73b4f7b2d39 100644 --- a/drivers/gpu/drm/i915/i915_drv.c +++ b/drivers/gpu/drm/i915/i915_drv.c | |||
@@ -770,10 +770,10 @@ static int i915_drm_resume(struct drm_device *dev) | |||
770 | dev_priv->display.hpd_irq_setup(dev); | 770 | dev_priv->display.hpd_irq_setup(dev); |
771 | spin_unlock_irq(&dev_priv->irq_lock); | 771 | spin_unlock_irq(&dev_priv->irq_lock); |
772 | 772 | ||
773 | intel_display_resume(dev); | ||
774 | |||
775 | intel_dp_mst_resume(dev); | 773 | intel_dp_mst_resume(dev); |
776 | 774 | ||
775 | intel_display_resume(dev); | ||
776 | |||
777 | /* | 777 | /* |
778 | * ... but also need to make sure that hotplug processing | 778 | * ... but also need to make sure that hotplug processing |
779 | * doesn't cause havoc. Like in the driver load code we don't | 779 | * doesn't cause havoc. Like in the driver load code we don't |