Re: [PATCH] DRM: ignore invalid EDID extensions

2010-10-28 Thread Marius Gröger
On 28.10.2010 11:12, wrote sam tygier: Without this patch i get no display (X or VT) on myEIZO S2242W monitor when using KMS This was discussed back in september, this is the second version of the patch based on ajax's comments. i have opened a bug for the issue: https://bugs.freedesktop.org/sho

Re: [PATCH] DRM: ignore invalid EDID extensions

2010-10-01 Thread Sam Tygier
On 22/09/10 22:55, Adam Jackson wrote: Yeah, I hate to just drop extension blocks, but it's better than the alternative. They're optional for a reason I suppose. 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 suspe

Re: [PATCH] DRM: ignore invalid EDID extensions

2010-09-23 Thread Sam Tygier
On 22/09/10 22:55, Adam Jackson wrote: On Wed, 2010-09-22 at 22:42 +0100, Sam Tygier wrote: On 22/09/10 21:59, Adam Jackson wrote: 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

Re: [PATCH] DRM: ignore invalid EDID extensions

2010-09-22 Thread Sam Tygier
On 22/09/10 21:59, Adam Jackson wrote: 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 ex

Re: [PATCH] DRM: ignore invalid EDID extensions

2010-09-22 Thread Adam Jackson
On Wed, 2010-09-22 at 22:42 +0100, Sam Tygier wrote: > On 22/09/10 21:59, Adam Jackson wrote: > > 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 >

Re: [PATCH] DRM: ignore invalid EDID extensions

2010-09-22 Thread Adam Jackson
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.