From: Pantelis Koukousoulas <pktoss@gmail.com>
Date: Tue, 10 Mar 2009 11:16:14 +0000 (+0200)
Subject: drm: fix EDID parser problem with positive/negative hsync/vsync
X-Git-Tag: upstream/snapshot3+hdmi~20024^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=260cf8a2cbe022131a4850ea6175ed716dc2663c;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git

drm: fix EDID parser problem with positive/negative hsync/vsync

Comparing the layouts of struct detail_pixel_timing with
x.org's struct detailed_timings and how those are handled,
it appears that the hsync_positive and vsync_positive
fields are backwards.

This patch fixes https://bugs.freedesktop.org/show_bug.cgi?id=20019
for me. It was tested on 2 monitors, LG FLATRON L225WS 22" and
a YAKUMO 17" for which more details are unknown.

Signed-off-by: Pantelis Koukousoulas <pktoss@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
---

diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h
index ff8d27a..a11cc9d 100644
--- a/include/drm/drm_edid.h
+++ b/include/drm/drm_edid.h
@@ -69,8 +69,8 @@ struct detailed_pixel_timing {
 	u8 hborder;
 	u8 vborder;
 	u8 unknown0:1;
-	u8 vsync_positive:1;
 	u8 hsync_positive:1;
+	u8 vsync_positive:1;
 	u8 separate_sync:2;
 	u8 stereo:1;
 	u8 unknown6:1;