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