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.

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.

Signed-off-by: Sam Tygier <samtyg...@yahoo.co.uk>
---
 drivers/gpu/drm/drm_edid.c |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index f87bf10..5ade343 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -285,6 +285,15 @@ carp:
        dev_warn(connector->dev->dev, "%s: EDID block %d invalid.\n",
                 drm_get_connector_name(connector), j);
+ /* Invalid extension, so set block[0x7e] to zero, and return
+        * the base block */
+       block[EDID_LENGTH-1] += block[0x7e];
+       block[0x7e] = 0;
+       new = krealloc(block, EDID_LENGTH, GFP_KERNEL);
+       if (!new)
+               goto out;
+       block = new;
+       return block;
 out:
        kfree(block);
        return NULL;
--
1.7.1
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to