Commit 260cf8a2 authored Mar 10, 2009 by Pantelis Koukousoulas Committed by Dave Airlie Mar 11, 2009

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>

parent 16b71fdf

include/drm/drm_edid.h

+1 −1

Original line number	Diff line number	Diff line
		@@ -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;

Admin message