[PATCH v2 6/6] drm/drm_edid: Avoid CEA VICs meant for HDMI2.1 in AVI Infoframes for older HDMI sinks.

Ankit Nautiyal ankit.k.nautiyal at intel.com
Wed Aug 11 12:35:35 UTC 2021


Modify the check for CEA modes, to avoid writing VICs added in
CEA-861-G (meant for HDMI2.1) in AVI infoframes for older HDMI sinks.

This patch also adds a function to determine if sink is HDMI2.1
and uses corrects the drm message to show HDMI2.1 sink detetction.

Signed-off-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>
---
 drivers/gpu/drm/drm_edid.c | 28 ++++++++++++++++++++++++++--
 1 file changed, 26 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 6325877c5fd6..0e7da78c842f 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -4915,10 +4915,12 @@ static void drm_parse_hdmi_forum_vsdb(struct drm_connector *connector,
 		u8 dsc_max_slices;
 		struct drm_hdmi_dsc_cap *hdmi_dsc = &hdmi->dsc_cap;
 
-		DRM_DEBUG_KMS("hdmi_21 sink detected. parsing edid\n");
 		max_frl_rate = (hf_vsdb[7] & DRM_EDID_MAX_FRL_RATE_MASK) >> 4;
 		drm_get_max_frl_rate(max_frl_rate, &hdmi->max_lanes,
 				     &hdmi->max_frl_rate_per_lane);
+		if (max_frl_rate)
+			DRM_DEBUG_KMS("hdmi_21 sink detected. parsing edid\n");
+
 		hdmi_dsc->v_1p2 = hf_vsdb[11] & DRM_EDID_DSC_1P2;
 
 		if (hdmi_dsc->v_1p2) {
@@ -5490,6 +5492,18 @@ void drm_set_preferred_mode(struct drm_connector *connector,
 }
 EXPORT_SYMBOL(drm_set_preferred_mode);
 
+static bool is_hdmi21_sink(const struct drm_connector *connector)
+{
+	/*
+	 * FIXME: sil-sii8620 doesn't have a connector around when
+	 * we need one, so we have to be prepared for a NULL connector.
+	 */
+	if (!connector)
+		return true;
+
+	return connector->display_info.hdmi.max_frl_rate_per_lane ? true : false;
+}
+
 static bool is_hdmi2_sink(const struct drm_connector *connector)
 {
 	/*
@@ -5611,8 +5625,18 @@ static u8 drm_mode_cea_vic(const struct drm_connector *connector,
 	 * HDMI 1.4 VIC range: 1 <= VIC <= 64 (CEA-861-D) but
 	 * HDMI 2.0 VIC range: 1 <= VIC <= 107 (CEA-861-F). So we
 	 * have to make sure we dont break HDMI 1.4 sinks.
+	 * Similarly,
+	 * HDMI 2.1 VIC range: 1 <= VIC <= 219 (CEA-861-G), we have
+	 * to make sure we dont break HDMI2.0 sinks.
 	 */
-	if (!is_hdmi2_sink(connector) && vic > 64)
+
+	if (!is_hdmi21_sink(connector) && !is_hdmi2_sink(connector) && vic > 64)
+		return 0;
+
+	if (!is_hdmi21_sink(connector) && vic > 107)
+		return 0;
+
+	if (vic > 219)
 		return 0;
 
 	return vic;
-- 
2.25.1



More information about the Intel-gfx-trybot mailing list