drm/gma500: Check connector status before restoring sdvo
authorPatrik Jakobsson <patrik.r.jakobsson@gmail.com>
Mon, 15 Apr 2013 20:02:07 +0000 (22:02 +0200)
committerPatrik Jakobsson <patrik.r.jakobsson@gmail.com>
Mon, 15 Apr 2013 21:38:06 +0000 (23:38 +0200)
This patch fixes a bug introduced by:

commit 749387dc8d8270b279f27a0a794cdf4f4a4aa774
Author: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
Date:   Sun Apr 7 16:35:50 2013 +0200
drm/gma500: Fix hibernation problems on sdvo encoders

The bug is triggered when we do a mode set on a sdvo encoder with all
connectors in the disconnected state. A crtc is considered enabled by
drm even though all of its connectors are disconnected. Work around
this by adding a check in our sdvo restore function.

Also remove the unneeded dpms on. Prepare and Commit will take care of
that.

Signed-off-by: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
drivers/gpu/drm/gma500/psb_intel_sdvo.c

index cd1b40722eddc07c75f702d161e068504c0e7d50..19e36603b23b64e2c500cb425d3bf0246b2fdd8f 100644 (file)
@@ -1854,12 +1854,11 @@ static void psb_intel_sdvo_restore(struct drm_connector *connector)
 
        REG_WRITE(sdvo->sdvo_reg, sdvo->saveSDVO);
 
-       /* Force dpms on upon resume */
-       psb_intel_sdvo_dpms(encoder, DRM_MODE_DPMS_ON);
-
        /* Force a full mode set on the crtc. We're supposed to have the
           mode_config lock already. */
-       drm_crtc_helper_set_mode(crtc, &crtc->mode, crtc->x, crtc->y, NULL);
+       if (connector->status == connector_status_connected)
+               drm_crtc_helper_set_mode(crtc, &crtc->mode, crtc->x, crtc->y,
+                                        NULL);
 }
 
 static const struct drm_encoder_helper_funcs psb_intel_sdvo_helper_funcs = {