[MAVERICK][PATCH] Revert "UBUNTU: SAUCE: drm/i915: don't change DRM configuration when releasing load detect pipe"

Chase Douglas chase.douglas at canonical.com
Fri May 21 13:41:13 UTC 2010

This reverts commit 7e432ba9aae503aae4cbb7a7ed4d53e457c651e4.

This patch was originally applied for Lucid to fix some lid open/close
backlight issues. Upstream did not pull this in because they thought
there was a better solution.

I am not sure what the upstream solution is, but I had someone affected
by the bug test out the 2.6.34 mainline build. They could not reproduce
the issue, so we should revert this change to be in line with upstream.

Signed-off-by: Chase Douglas <chase.douglas at canonical.com>
 drivers/gpu/drm/i915/intel_display.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 139de44..c7502b6 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -3758,6 +3758,7 @@ struct drm_crtc *intel_get_load_detect_pipe(struct intel_encoder *intel_encoder,
 void intel_release_load_detect_pipe(struct intel_encoder *intel_encoder, int dpms_mode)
 	struct drm_encoder *encoder = &intel_encoder->enc;
+	struct drm_device *dev = encoder->dev;
 	struct drm_crtc *crtc = encoder->crtc;
 	struct drm_encoder_helper_funcs *encoder_funcs = encoder->helper_private;
 	struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
@@ -3767,6 +3768,7 @@ void intel_release_load_detect_pipe(struct intel_encoder *intel_encoder, int dpm
 		intel_encoder->base.encoder = NULL;
 		intel_encoder->load_detect_temp = false;
 		crtc->enabled = drm_helper_crtc_in_use(crtc);
+		drm_helper_disable_unused_functions(dev);
 	/* Switch crtc and encoder back off if necessary */

More information about the kernel-team mailing list