drm/edid: parse sink information before CEA blocks
authorShashank Sharma <shashank.sharma@intel.com>
Thu, 13 Jul 2017 15:33:09 +0000 (21:03 +0530)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 14 Jul 2017 18:23:54 +0000 (21:23 +0300)
CEA-861-F adds ycbcr capability map block, for HDMI 2.0 sinks.
This block contains a map of indexes of CEA modes, which can
support YCBCR 420 output also. To avoid multiple parsing of same
CEA block, let's parse the sink information and get this map, before
parsing CEA modes.

This patch moves the call to drm_add_display_info function, before the
mode parsing block.

V4: Introduced new patch in the series
V5: Move this patch before 4:2:0 parsing patch (ville)
    Added r-b from Ville
V6: Rebase
V7: Rebase

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Shashank Sharma <shashank.sharma@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1499960000-9232-4-git-send-email-shashank.sharma@intel.com
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
drivers/gpu/drm/drm_edid.c

index 42c83add9d37de27935d2aa9f35580235df39077..dc0eb31355c90cd65b3419c2ad244ea5af4575ac 100644 (file)
@@ -4438,6 +4438,13 @@ int drm_add_edid_modes(struct drm_connector *connector, struct edid *edid)
 
        quirks = edid_get_quirks(edid);
 
+       /*
+        * CEA-861-F adds ycbcr capability map block, for HDMI 2.0 sinks.
+        * To avoid multiple parsing of same block, lets parse that map
+        * from sink info, before parsing CEA modes.
+        */
+       drm_add_display_info(connector, edid);
+
        /*
         * EDID spec says modes should be preferred in this order:
         * - preferred detailed mode
@@ -4465,8 +4472,6 @@ int drm_add_edid_modes(struct drm_connector *connector, struct edid *edid)
        if (quirks & (EDID_QUIRK_PREFER_LARGE_60 | EDID_QUIRK_PREFER_LARGE_75))
                edid_fixup_preferred(connector, quirks);
 
-       drm_add_display_info(connector, edid);
-
        if (quirks & EDID_QUIRK_FORCE_6BPC)
                connector->display_info.bpc = 6;