[Intel-gfx] [PATCH] LVDS: Fix output mode list construction.

[Adam Jackson]

On Wed, 2009-10-14 at 06:36 +1000, Dave Airlie wrote:
> On Wed, Oct 14, 2009 at 5:18 AM, Adam Jackson <ajax at redhat.com> wrote:
> > Construct a list of modes smaller than the LVDS panel size if either:
> >
> > - the panel does not have EDID or DisplayID
> > - it has EDID/dID but does not claim to be continuous-frequency (in
> >  which case the server would have done it for us already)
> >
> > The old method would attempt to do this by mangling the EDID block and
> > parsed ranges, which was unreasonably fragile, and would fail to do so
> > when EDID existed but claimed only a single mode.
> 
> This really should be done in the kernel from what I can see, so all KMS users
> get a consistent list of modes for the LVDS device.

Minor quibble for the case where the GPU doesn't actually have a scaler,
but, sure.

X's notion of default mode list is actually pretty crap.  I'm likely to
fold it down to be just DMT plus the extras from the various established
timing blocks.  The kernel should do the same.

- ajax
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20091013/abde0097/attachment.sig>