On Mon, Jun 10, 2013 at 09:10:51AM +0100, Chris Wilson wrote: > On Mon, Jun 10, 2013 at 09:47:58AM +0200, Daniel Vetter wrote: > > In > > > > commit 53d3b4d7778daf15900867336c85d3f8dd70600c > > Author: Egbert Eich <eich at suse.de> > > Date: Tue Jun 4 17:13:21 2013 +0200 > > > > drm/i915/sdvo: Use &intel_sdvo->ddc instead of intel_sdvo->i2c for DDC > > > > Egbert Eich fixed a long-standing bug where we simply used a > > non-working i2c controller to read the EDID for SDVO-LVDS panels. > > Unfortunately some machines seem to not be able to cope with the mode > > provided in the EDID (specifically they seem to not be able to cope > > with a 4x pixel mutliplier instead of a 2x one). > > > > Since it took forever to notice the breakage it's fairly safe to > > assume that at least for SDVO-LVDS panels the VBT contains fairly sane > > data. So just switch around the order and use VBT modes first. > > > > v2: Also add EDID modes just in case, and spell Egbert correctly. > > > > Cc: Egbert Eich <eich at suse.de> > > Cc: Chris Wilson <chris at chris-wilson.co.uk> > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=65524 > > Cc: stable at vger.kernel.org > > Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > Tested-by: Chris Wilson <chris at chris-wilson.co.uk> Picked up for -fixes, thanks for testing. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch
