<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - Atombios stuck in a look during resume" href="https://bugs.freedesktop.org/show_bug.cgi?id=67043#c7">Comment # 7</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - Atombios stuck in a look during resume" href="https://bugs.freedesktop.org/show_bug.cgi?id=67043">bug 67043</a> from <a class="email" href="mailto:parag.warudkar@gmail.com" title="Parag <parag.warudkar@gmail.com>"> Parag</a> <pre>After a not-so-clean git bisect (compilation failures in vgacon.c, some TI driver I didn't need) this was the result - at least it is related to drm. git bisect good 372835a8527f85b3eff20a18c2c339e827dfd4e4 is the first bad commit commit 372835a8527f85b3eff20a18c2c339e827dfd4e4 Author: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>> Date: Sat Jun 15 00:13:13 2013 +0200 drm/crtc-helper: explicit DPMS on after modeset Atm the crtc helper implementation of set_config has really inconsisten semantics: If just an fb update is good enough, dpms state will be left as-is, but if we do a full modeset we force everything to dpms on. This change has already been applied to the i915 modeset code in commit e3de42b68478a8c95dd27520e9adead2af9477a5 Author: Imre Deak <<a href="mailto:imre.deak@intel.com">imre.deak@intel.com</a>> Date: Fri May 3 19:44:07 2013 +0200 drm/i915: force full modeset if the connector is in DPMS OFF mode which according to Greg KH seems to aim for a new record in most Bugzilla: links in a commit message. The history of this dpms forcing is pretty interesting. This patch here is an almost-revert of commit 811aaa55ba21ab37407018cfc01770d6b037d3fb Author: Keith Packard <<a href="mailto:keithp@keithp.com">keithp@keithp.com</a>> Date: Thu Feb 3 16:57:28 2011 -0800 drm: Only set DPMS ON when actually configuring a mode which fixed the bug of trying to dpms on disabled outputs, but introduced the new discrepancy between an fb update only and full modesets. The actual introduction of this goes back to commit bf9dc102e284a5aa78c73fc9d72e11d5ccd8669f Author: Keith Packard <<a href="mailto:keithp@keithp.com">keithp@keithp.com</a>> Date: Fri Nov 26 10:45:58 2010 -0800 drm: Set connector DPMS status to ON in drm_crtc_helper_set_config And if you'd dig around in the i915 driver code there's even more fun around forcing dpms on and losing our heads and temper of the resulting inconsistencies. Especially the DP re-training code had tons of funny stuff in it. Signed-off-by: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>> Signed-off-by: Dave Airlie <<a href="mailto:airlied@redhat.com">airlied@redhat.com</a>></pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>