FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.

» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Ubuntu Kernel Team

LinkBack Thread Tools
Old 03-01-2010, 03:37 PM
Chase Douglas
Default drm/i915: don't change DRM configuration when releasing load detect pipe

From: Jesse Barnes <jbarnes@virtuousgeek.org>

When we get a CRTC to use for load detection, we restore its DPMS state
if needed. We shouldn't, however, change the DRM configuration by
calling drm_helper_disable_unused_functions when we release the CRTC.
Doing so can cause problems with resume, since at suspend or lid close
time, X may choose to probe outputs. If it doesn't re-probe them at
open or resume time, LVDS won't be restored, since
drm_helper_disable_unused_functions will have turned it off, preventing
the mode set at lid open from restoring it.

Fixes kernel bz #14997.

BugLink: http://bugs.launchpad.net/bugs/488328

Cc: stable at kernel.org
Tested-by: Takashi Iwai <tiwai at suse.de>
Tested-by: Thorsten Leemhuis <fedora at leemhuis.info>
Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org>
Signed-off-by: Chase Douglas <chase.douglas@canonical.com>
drivers/gpu/drm/i915/intel_display.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index b00a1aa..0e4af08 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -3619,7 +3619,6 @@ struct drm_crtc *intel_get_load_detect_pipe(struct intel_output *intel_output,
void intel_release_load_detect_pipe(struct intel_output *intel_output, int dpms_mode)
struct drm_encoder *encoder = &intel_output->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;
@@ -3629,7 +3628,6 @@ void intel_release_load_detect_pipe(struct intel_output *intel_output, int dpms_
intel_output->base.encoder = NULL;
intel_output->load_detect_temp = false;
crtc->enabled = drm_helper_crtc_in_use(crtc);
- drm_helper_disable_unused_functions(dev);

/* Switch crtc and output back off if necessary */

kernel-team mailing list

Thread Tools

All times are GMT. The time now is 09:58 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org