drm/i915/tc: Assume a TC port is legacy if VBT says the port has HDMI

Since an HDMI output can only be enabled in legacy mode on TC ports,
assume that VBT is wrong and the port is legacy if VBT says the port is
non-legacy and has HDMI. If VBT says to enable DP as well leave the
non-legacy flag enabled, relying on the flag getting fixed up based on
the HPD status during sink detection.

v2: Fix the legacy port flag only if DP is not enabled.

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230321220101.983366-1-imre.deak@intel.com

diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index 3d6d274..8805676 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -4497,6 +4497,16 @@ void intel_ddi_init(struct drm_i915_private *dev_priv, enum port port)
                        !intel_bios_encoder_supports_typec_usb(devdata) &&
                        !intel_bios_encoder_supports_tbt(devdata);
 
+               if (!is_legacy && init_hdmi) {
+                       is_legacy = !init_dp;
+
+                       drm_dbg_kms(&dev_priv->drm,
+                                   "VBT says port %c is non-legacy TC and has HDMI (with DP: %s), assume it's %s\n",
+                                   port_name(port),
+                                   str_yes_no(init_dp),
+                                   is_legacy ? "legacy" : "non-legacy");
+               }
+
                intel_tc_port_init(dig_port, is_legacy);
 
                encoder->update_prepare = intel_ddi_update_prepare;