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>
