@@ -339,11 +339,14 @@ static u32 intel_ddi_buf_ctl_config_val(struct intel_encoder *encoder,
struct intel_display *display = to_intel_display(encoder);
struct drm_i915_private *i915 = to_i915(encoder->base.dev);
struct intel_digital_port *dig_port = enc_to_dig_port(encoder);
+ bool is_dp = intel_crtc_has_dp_encoder(crtc_state);
u32 val = 0;
/* DDI_BUF_CTL_ENABLE will be set by intel_ddi_prepare_link_retrain() later */
- val |= DDI_PORT_WIDTH(crtc_state->lane_count) |
- DDI_BUF_TRANS_SELECT(0);
+ if (is_dp || DISPLAY_VER(display) >= 14)
+ val |= DDI_PORT_WIDTH(crtc_state->lane_count);
+
+ val |= DDI_BUF_TRANS_SELECT(0);
if (dig_port->lane_reversal)
val |= DDI_BUF_PORT_REVERSAL;
@@ -351,14 +354,15 @@ static u32 intel_ddi_buf_ctl_config_val(struct intel_encoder *encoder,
val |= DDI_A_4_LANES;
if (DISPLAY_VER(i915) >= 14) {
- if (intel_dp_is_uhbr(crtc_state))
+ if (is_dp && intel_dp_is_uhbr(crtc_state))
val |= DDI_BUF_PORT_DATA_40BIT;
else
val |= DDI_BUF_PORT_DATA_10BIT;
}
if (IS_ALDERLAKE_P(i915) && intel_encoder_is_tc(encoder)) {
- val |= ddi_buf_phy_link_rate(crtc_state->port_clock);
+ if (is_dp)
+ val |= ddi_buf_phy_link_rate(crtc_state->port_clock);
/*
* TODO: remove the following once DDI_BUF_CTL is updated via
* an RMW everywhere.
@@ -367,7 +371,7 @@ static u32 intel_ddi_buf_ctl_config_val(struct intel_encoder *encoder,
val |= DDI_BUF_CTL_TC_PHY_OWNERSHIP;
}
- if (IS_DISPLAY_VER(display, 11, 13) && intel_encoder_is_tc(encoder)) {
+ if (is_dp && IS_DISPLAY_VER(display, 11, 13) && intel_encoder_is_tc(encoder)) {
int delay = dp_phy_lane_stagger_delay(crtc_state->port_clock);
val |= DDI_BUF_LANE_STAGGER_DELAY(delay);
@@ -3417,7 +3421,6 @@ static void intel_ddi_enable_hdmi(struct intel_atomic_state *state,
struct intel_digital_port *dig_port = enc_to_dig_port(encoder);
struct drm_connector *connector = conn_state->connector;
enum port port = encoder->port;
- u32 buf_ctl = 0;
if (!intel_hdmi_handle_sink_scrambling(encoder, connector,
crtc_state->hdmi_high_tmds_clock_ratio,
@@ -3482,11 +3485,6 @@ static void intel_ddi_enable_hdmi(struct intel_atomic_state *state,
* is filled with lane count, already set in the crtc_state.
* The same is required to be filled in PORT_BUF_CTL for C10/20 Phy.
*/
- if (dig_port->lane_reversal)
- buf_ctl |= DDI_BUF_PORT_REVERSAL;
- if (dig_port->ddi_a_4_lanes)
- buf_ctl |= DDI_A_4_LANES;
-
if (DISPLAY_VER(dev_priv) >= 14) {
u32 port_buf = 0;
@@ -3497,17 +3495,9 @@ static void intel_ddi_enable_hdmi(struct intel_atomic_state *state,
intel_de_rmw(dev_priv, XELPDP_PORT_BUF_CTL1(dev_priv, port),
XELPDP_PORT_WIDTH_MASK | XELPDP_PORT_REVERSAL, port_buf);
-
- buf_ctl |= DDI_PORT_WIDTH(crtc_state->lane_count);
-
- if (DISPLAY_VER(dev_priv) >= 20)
- buf_ctl |= XE2LPD_DDI_BUF_D2D_LINK_ENABLE;
- } else if (IS_ALDERLAKE_P(dev_priv) && intel_encoder_is_tc(encoder)) {
- drm_WARN_ON(&dev_priv->drm, !intel_tc_port_in_legacy_mode(dig_port));
- buf_ctl |= DDI_BUF_CTL_TC_PHY_OWNERSHIP;
}
- intel_enable_ddi_buf(encoder, buf_ctl);
+ intel_enable_ddi_buf(encoder, intel_ddi_buf_ctl_config_val(encoder, crtc_state));
}
static void intel_ddi_enable(struct intel_atomic_state *state,
Reuse the existing helper to compute the configuration value of the DDI_BUF_CTL register for HDMI outputs instead of open-coding this. Note that dropping the XE2LPD_DDI_BUF_D2D_LINK_ENABLE flag is ok, since an earlier mtl_ddi_enable_d2d() has set it already and intel_enable_ddi_buf()'s RMW will not update this flag. Signed-off-by: Imre Deak <imre.deak@intel.com> --- drivers/gpu/drm/i915/display/intel_ddi.c | 30 ++++++++---------------- 1 file changed, 10 insertions(+), 20 deletions(-)