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ä <[email protected]>
Signed-off-by: Imre Deak <[email protected]>
---
 drivers/gpu/drm/i915/display/intel_ddi.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c 
b/drivers/gpu/drm/i915/display/intel_ddi.c
index 3d6d27409ebe8..8805676cc3a65 100644
--- 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;
-- 
2.37.1

Reply via email to