drm/edid: Fix parsing of EDID 1.4 Established Timings III descriptor
authorPaul Parsons <lost.distance@yahoo.com>
Sat, 26 Mar 2016 13:18:38 +0000 (13:18 +0000)
committerJani Nikula <jani.nikula@intel.com>
Tue, 5 Apr 2016 08:15:44 +0000 (11:15 +0300)
The EDID 1.4 specification section 3.10.3.9 defines an Established Timings III
descriptor (tag #F7h). The parsing of this descriptor by drm_est3_modes() is
off by one byte: the offset of the first timing bitmap is 6, not 5.

Signed-off-by: Paul Parsons <lost.distance@yahoo.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20160328002258.E75DF6E35D@gabe.freedesktop.org
drivers/gpu/drm/drm_edid.c

index 94541187c62ac952dfeb0a2c778ba47ac90d6def..c33edb0f23b27c3b4a1f558367d4f103aae79c08 100644 (file)
@@ -2241,7 +2241,7 @@ drm_est3_modes(struct drm_connector *connector, struct detailed_timing *timing)
 {
        int i, j, m, modes = 0;
        struct drm_display_mode *mode;
-       u8 *est = ((u8 *)timing) + 5;
+       u8 *est = ((u8 *)timing) + 6;
 
        for (i = 0; i < 6; i++) {
                for (j = 7; j >= 0; j--) {
This page took 0.029548 seconds and 5 git commands to generate.