Not sure where this check came from, but it's wrong according to a bug
report I can't find right now.  We need to parse the detailed blocks no
matter what, or lots of monitors won't be given a good set of default
modes.

Adam, can you check & ack this?

Thanks,
Jesse

Signed-off-by: Jesse Barnes <jbar...@virtuousgeek.org>

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index ca9c616..049b338 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -493,10 +493,6 @@ static int add_detailed_info(struct drm_connector 
*connector,
                struct detailed_non_pixel *data = &timing->data.other_data;
                struct drm_display_mode *newmode;
 
-               /* EDID up to and including 1.2 may put monitor info here */
-               if (edid->version == 1 && edid->revision < 3)
-                       continue;
-
                /* Detailed mode timing */
                if (timing->pixel_clock) {
                        newmode = drm_mode_detailed(dev, edid, timing, quirks);

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to