[Intel-gfx] [PATCH] drm/i915: prefer VBT modes for SVDO-LVDS over EDID

[Daniel Vetter]

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