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 42c83ad..dc0eb31 100644 (file)
@@ -4439,6 +4439,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
         * - other detailed modes from base block
@@ -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;