[PATCH] DRM: ignore invalid EDID extensions

Adam Jackson ajax at redhat.com
Wed Sep 22 13:59:09 PDT 2010


On Tue, 2010-09-21 at 23:02 +0100, Sam Tygier wrote:
> Currently an invalid EDID extension will cause the whole EDID to be
> considered invalid. Instead just drop the extension, and return the
> valid base block. The base block is modified to not claim to have
> extensions, and update the checksum.

This does not appear to be what your patch does.  I mean, yes, if
there's only one extension block, that's what it does, but if there's
more than one...

> For my EIZO S2242W the base block is fine, but the extension block is
> all zeros. Without this patch I get no X and no VTs.

I suspect what's actually happening there is that we're failing to get
the extension block, not that it's being returned as zeros.

Also, we shouldn't be converting "checksum failure" to "display
disconnected".  That's the real bug here.

- ajax
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20100922/fdf0bb28/attachment.pgp>


More information about the dri-devel mailing list