Commit c9354c85c1c7bac788ce57d3c17f2016c1c45b1d

Avatar

Repository git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git master
by Linus.Torvalds (Using name ‘Linus Torvalds’) on 2009-11-02 17:29 (28 days ago)

i915: fix intel graphics suspend breakage due to resume/lid event confusion
In commit c1c7af60892070e4b82ad63bbfb95ae745056de0 ("drm/i915: force
mode set at lid open time") the intel graphics driver was taught to
restore the LVDS mode on lid open.

That caused problems with interaction with the suspend/resume code,
which commonly runs at the same time (suspend is often caused by the lid
close event, while lid open is commonly a resume event), which was
worked around with in commit 06891e27a9b5dba5268bb80e41a283f51335afe7
("drm/i915: fix suspend/resume breakage in lid notifier").

However, in the meantime the lid event code had also grown a user event
notifier (commit 06324194eee97a51b5f172270df49ec39192d6cc: "drm/i915:
generate a KMS uevent at lid open/close time"), and now _that_ causes
problems with suspend/resume and some versions of Xorg reacting to those
uevents by setting the mode.

So this effectively reverts that commit 06324194ee, and makes the lid
open protection logic against suspend/resume more explicit. This fixes
at least one laptop. See

http://bugzilla.kernel.org/show_bug.cgi?id=14484

for more details.

Acked-by: Jesse Barnes <jba...@virtuousgeek.org>
Cc: Riccardo Magliocchetti <riccardo.ma...@gmail.com>
Cc: Eric Anholt <er...@anholt.net>
Signed-off-by: Linus Torvalds <torv...@linux-foundation.org>

Language Totals

  Language Code Added Code Removed Comments Added Comments Removed Blanks Added Blanks Removed
  C 9 5 10 0 2 0
  C++ 1 1 0 0 0 0

Files Changed

[3 total ]
File Language Code Added Code Removed Comments Added Comments Removed Blanks Added Blanks Removed
drivers/gpu/drm/i915/i915_drv.c
  C 2 2 1 0 0 0
drivers/gpu/drm/i915/i915_drv.h
  C++ 1 1 0 0 0 0
drivers/gpu/drm/i915/intel_lvds.c
  C 7 3 9 0 2 0