[Intel-gfx] [PATCH v2 17/17] drm/i915/dp_mst: Verify the HW state of MST encoders
Imre Deak
imre.deak at intel.com
Tue Jan 31 15:05:48 UTC 2023
Read out and verify an MST encoder's HW state after any of the
MST connectors driven by the encoder is modeset.
Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
Signed-off-by: Imre Deak <imre.deak at intel.com>
---
drivers/gpu/drm/i915/display/intel_ddi.c | 91 +++++++++++++++++--
drivers/gpu/drm/i915/display/intel_display.c | 2 +-
.../drm/i915/display/intel_display_types.h | 18 ++++
drivers/gpu/drm/i915/display/intel_dp_mst.c | 68 +++++++++++++-
drivers/gpu/drm/i915/display/intel_dp_mst.h | 2 +-
.../drm/i915/display/intel_modeset_verify.c | 2 +-
drivers/gpu/drm/i915/i915_reg.h | 6 +-
7 files changed, 173 insertions(+), 16 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index 254559abedfba..6acda4d357af3 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -661,13 +661,23 @@ int intel_ddi_toggle_hdcp_bits(struct intel_encoder *intel_encoder,
return ret;
}
+static enum transcoder get_transcoder_for_pipe(const struct intel_encoder *encoder,
+ enum pipe pipe)
+{
+ struct drm_i915_private *i915 = to_i915(encoder->base.dev);
+
+ if (HAS_TRANSCODER(i915, TRANSCODER_EDP) && encoder->port == PORT_A)
+ return TRANSCODER_EDP;
+ else
+ return (enum transcoder) pipe;
+}
+
bool intel_ddi_connector_get_hw_state(struct intel_connector *intel_connector)
{
struct drm_device *dev = intel_connector->base.dev;
struct drm_i915_private *dev_priv = to_i915(dev);
struct intel_encoder *encoder = intel_attached_encoder(intel_connector);
int type = intel_connector->base.connector_type;
- enum port port = encoder->port;
enum transcoder cpu_transcoder;
intel_wakeref_t wakeref;
enum pipe pipe = 0;
@@ -684,10 +694,7 @@ bool intel_ddi_connector_get_hw_state(struct intel_connector *intel_connector)
goto out;
}
- if (HAS_TRANSCODER(dev_priv, TRANSCODER_EDP) && port == PORT_A)
- cpu_transcoder = TRANSCODER_EDP;
- else
- cpu_transcoder = (enum transcoder) pipe;
+ cpu_transcoder = get_transcoder_for_pipe(encoder, pipe);
tmp = intel_de_read(dev_priv, TRANS_DDI_FUNC_CTL(cpu_transcoder));
@@ -2155,17 +2162,29 @@ i915_reg_t dp_tp_ctl_reg(struct intel_encoder *encoder,
return DP_TP_CTL(encoder->port);
}
-i915_reg_t dp_tp_status_reg(struct intel_encoder *encoder,
- const struct intel_crtc_state *crtc_state)
+static i915_reg_t __dp_tp_status_reg(struct intel_encoder *encoder,
+ enum transcoder transcoder)
{
struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
if (DISPLAY_VER(dev_priv) >= 12)
- return TGL_DP_TP_STATUS(tgl_dp_tp_transcoder(crtc_state));
+ return TGL_DP_TP_STATUS(transcoder);
else
return DP_TP_STATUS(encoder->port);
}
+i915_reg_t dp_tp_status_reg(struct intel_encoder *encoder,
+ const struct intel_crtc_state *crtc_state)
+{
+ struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
+ enum transcoder transcoder = INVALID_TRANSCODER;
+
+ if (DISPLAY_VER(dev_priv) >= 12)
+ transcoder = tgl_dp_tp_transcoder(crtc_state);
+
+ return __dp_tp_status_reg(encoder, transcoder);
+}
+
static void intel_dp_sink_set_msa_timing_par_ignore_state(struct intel_dp *intel_dp,
const struct intel_crtc_state *crtc_state,
bool enable)
@@ -3500,6 +3519,61 @@ static void intel_ddi_get_config(struct intel_encoder *encoder,
intel_audio_codec_get_config(encoder, pipe_config);
}
+static bool intel_ddi_get_hw_mst_state(struct intel_encoder *encoder,
+ struct intel_encoder_mst_state *state)
+{
+ struct drm_i915_private *i915 = to_i915(encoder->base.dev);
+ enum transcoder transcoder;
+ u8 vc_pipe_mask = 0;
+ u8 pipe_mask;
+ bool is_mst;
+ u32 dp_status;
+ bool in_mst_mode;
+ int vc;
+
+ intel_ddi_get_encoder_pipes(encoder, &pipe_mask, &is_mst);
+ if (!is_mst || !pipe_mask)
+ return false;
+
+ transcoder = get_transcoder_for_pipe(encoder, ffs(pipe_mask) - 1);
+ dp_status = intel_de_read(i915, __dp_tp_status_reg(encoder, transcoder));
+
+ in_mst_mode = REG_FIELD_GET(DP_TP_STATUS_MODE_STATUS_MST, dp_status);
+ if (drm_WARN_ON(&i915->drm, !in_mst_mode))
+ return false;
+
+ state->num_mst_streams = REG_FIELD_GET(DP_TP_STATUS_NUM_STREAMS_ENABLED, dp_status);
+ drm_WARN_ON(&i915->drm, state->num_mst_streams == 0 ||
+ state->num_mst_streams > ARRAY_SIZE(state->mst_stream));
+
+ for (vc = 0; vc < ARRAY_SIZE(state->mst_stream); vc++) {
+ struct intel_crtc *crtc;
+ struct intel_link_m_n m_n;
+
+ if (vc >= state->num_mst_streams) {
+ state->mst_stream[vc].pipe = INVALID_PIPE;
+ state->mst_stream[vc].tu = 0;
+ continue;
+ }
+
+ state->mst_stream[vc].pipe = (dp_status & DP_TP_STATUS_PAYLOAD_MAPPING_MASK(vc)) >>
+ DP_TP_STATUS_PAYLOAD_MAPPING_SHIFT(vc);
+
+ drm_WARN_ON(&i915->drm, vc_pipe_mask & BIT(state->mst_stream[vc].pipe));
+ vc_pipe_mask |= BIT(state->mst_stream[vc].pipe);
+
+ transcoder = get_transcoder_for_pipe(encoder, state->mst_stream[vc].pipe);
+ crtc = intel_crtc_for_pipe(i915, state->mst_stream[vc].pipe);
+
+ intel_cpu_transcoder_get_m1_n1(crtc, transcoder, &m_n);
+ state->mst_stream[vc].tu = m_n.tu;
+ }
+
+ drm_WARN_ON(&i915->drm, vc_pipe_mask != pipe_mask);
+
+ return true;
+}
+
void intel_ddi_get_clock(struct intel_encoder *encoder,
struct intel_crtc_state *crtc_state,
struct intel_shared_dpll *pll)
@@ -4384,6 +4458,7 @@ void intel_ddi_init(struct drm_i915_private *dev_priv, enum port port)
encoder->post_disable = intel_ddi_post_disable;
encoder->update_pipe = intel_ddi_update_pipe;
encoder->get_hw_state = intel_ddi_get_hw_state;
+ encoder->get_hw_mst_state = intel_ddi_get_hw_mst_state;
encoder->sync_state = intel_ddi_sync_state;
encoder->initial_fastset_check = intel_ddi_initial_fastset_check;
encoder->suspend = intel_ddi_encoder_suspend;
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 7976658771ab3..8b2b7a30b1b01 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -7541,7 +7541,7 @@ static void intel_atomic_commit_tail(struct intel_atomic_state *state)
if (state->modeset) {
intel_verify_planes(state);
- intel_dp_mst_verify_state(state);
+ intel_dp_mst_verify_state(state, true);
}
intel_sagv_post_plane_update(state);
diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/gpu/drm/i915/display/intel_display_types.h
index 06d51d2b5e0d6..066a1b956b0ba 100644
--- a/drivers/gpu/drm/i915/display/intel_display_types.h
+++ b/drivers/gpu/drm/i915/display/intel_display_types.h
@@ -151,6 +151,18 @@ enum intel_hotplug_state {
INTEL_HOTPLUG_RETRY,
};
+struct intel_encoder_mst_state {
+ /**
+ * TODO: move the encoder state from intel_crtc_state to here, and
+ * make this a generic encoder state.
+ */
+ u8 num_mst_streams;
+ struct {
+ u8 pipe;
+ u8 tu;
+ } mst_stream[I915_MAX_PIPES];
+};
+
struct intel_encoder {
struct drm_encoder base;
@@ -213,6 +225,12 @@ struct intel_encoder {
* be set correctly before calling this function. */
void (*get_config)(struct intel_encoder *,
struct intel_crtc_state *pipe_config);
+ /**
+ * Determine if the encoder is active in MST mode and if so
+ * read out the corresponding HW state.
+ */
+ bool (*get_hw_mst_state)(struct intel_encoder *encoder,
+ struct intel_encoder_mst_state *mst_state);
/*
* Optional hook called during init/resume to sync any state
diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c
index a9bb339e41987..5bc18450b09c6 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_mst.c
+++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c
@@ -1322,23 +1322,55 @@ int intel_dp_mst_add_topology_state_for_crtc(struct intel_atomic_state *state,
/**
* intel_dp_mst_verify_state - Verify the MST state for all connectors in the atomic state
* @state: atomic state
+ * @connectors_enabled: %true if called after the modeset's enabling
+ * %false if called after the disabling sequence
*
* Verify the MST SW and sink state for all modesetted MST connectors in @state.
*/
-void intel_dp_mst_verify_state(struct intel_atomic_state *state)
+void intel_dp_mst_verify_state(struct intel_atomic_state *state, bool connectors_enabled)
{
+ struct drm_i915_private *i915 = to_i915(state->base.dev);
struct drm_dp_mst_topology_state *mst_state;
struct drm_dp_mst_topology_mgr *mgr;
int i;
for_each_new_mst_mgr_in_state(&state->base, mgr, mst_state, i) {
struct intel_dp *intel_dp = container_of(mgr, struct intel_dp, mst_mgr);
+ struct intel_encoder *encoder = &dp_to_dig_port(intel_dp)->base;
+ struct intel_encoder_mst_state hw_state;
struct drm_connector *_connector;
struct drm_connector_state *_conn_state;
bool mst_needs_modeset = false;
+ bool has_alloc_errors = drm_dp_mst_has_payload_alloc_errors(mst_state);
+ int i;
+
+ if (!encoder->get_hw_mst_state(encoder, &hw_state)) {
+ drm_WARN_ON(&i915->drm, drm_dp_mst_payload_count(mst_state));
+ continue;
+ }
+
+ /*
+ * The streams for which the payload allocation has failed got
+ * still enabled in the HW. In that case check only for an
+ * upper bound of mgr->payload_count here and skip the MST payload
+ * vs. HW state check later, since those will not match.
+ */
+ if (!has_alloc_errors)
+ drm_WARN_ON(&i915->drm, hw_state.num_mst_streams !=
+ drm_dp_mst_payload_count(mst_state));
+ else
+ drm_WARN_ON(&i915->drm, hw_state.num_mst_streams <
+ drm_dp_mst_payload_count(mst_state));
for_each_new_connector_in_state(&state->base, _connector, _conn_state, i) {
struct intel_connector *connector = to_intel_connector(_connector);
+ struct intel_digital_connector_state *conn_state;
+ struct intel_crtc *crtc;
+ struct intel_crtc_state *crtc_state = NULL;
+ struct drm_dp_mst_atomic_payload *payload;
+ enum pipe pipe;
+ int sw_vc;
+ int tu;
if (!connector->mst_port ||
!intel_connector_needs_modeset(state, &connector->base))
@@ -1346,7 +1378,39 @@ void intel_dp_mst_verify_state(struct intel_atomic_state *state)
mst_needs_modeset = true;
- break;
+ if (has_alloc_errors)
+ break;
+
+ payload = drm_atomic_get_mst_payload_state(mst_state, connector->port);
+ if (drm_WARN_ON(&i915->drm, !payload))
+ continue;
+
+ conn_state = to_intel_digital_connector_state(_conn_state);
+ crtc = conn_state->base.crtc ? to_intel_crtc(conn_state->base.crtc) : NULL;
+ if (crtc)
+ crtc_state = intel_atomic_get_new_crtc_state(state, crtc);
+
+ if (connectors_enabled && crtc_state && crtc_state->hw.active) {
+ pipe = crtc->pipe;
+ tu = drm_dp_mst_payload_time_slots(payload);
+ drm_WARN_ON(&i915->drm, crtc_state->dp_m_n.tu != tu);
+ } else {
+ pipe = INVALID_PIPE;
+ tu = 0;
+ }
+
+ sw_vc = drm_dp_mst_payload_vchannel(mst_state, payload);
+ if (sw_vc < 0) {
+ drm_WARN_ON(&i915->drm, pipe != INVALID_PIPE);
+ drm_WARN_ON(&i915->drm, tu != 0);
+ } else {
+ if (drm_WARN_ON(&i915->drm,
+ sw_vc >= ARRAY_SIZE(hw_state.mst_stream)))
+ continue;
+
+ drm_WARN_ON(&i915->drm, hw_state.mst_stream[sw_vc].pipe != pipe);
+ drm_WARN_ON(&i915->drm, hw_state.mst_stream[sw_vc].tu != tu);
+ }
}
if (!mst_needs_modeset)
diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.h b/drivers/gpu/drm/i915/display/intel_dp_mst.h
index 74633390c280c..53367b6a1ba39 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_mst.h
+++ b/drivers/gpu/drm/i915/display/intel_dp_mst.h
@@ -26,6 +26,6 @@ int intel_dp_mst_add_topology_state_for_crtc(struct intel_atomic_state *state,
int intel_dp_mst_add_topology_state_for_connector(struct intel_atomic_state *state,
struct intel_connector *connector,
struct intel_crtc *crtc);
-void intel_dp_mst_verify_state(struct intel_atomic_state *state);
+void intel_dp_mst_verify_state(struct intel_atomic_state *state, bool connectors_enabled);
#endif /* __INTEL_DP_MST_H__ */
diff --git a/drivers/gpu/drm/i915/display/intel_modeset_verify.c b/drivers/gpu/drm/i915/display/intel_modeset_verify.c
index 45f0d9789ef8e..caf541b562e54 100644
--- a/drivers/gpu/drm/i915/display/intel_modeset_verify.c
+++ b/drivers/gpu/drm/i915/display/intel_modeset_verify.c
@@ -245,5 +245,5 @@ void intel_modeset_verify_disabled(struct drm_i915_private *dev_priv,
verify_encoder_state(dev_priv, state);
verify_connector_state(state, NULL);
intel_shared_dpll_verify_disabled(dev_priv);
- intel_dp_mst_verify_state(state);
+ intel_dp_mst_verify_state(state, false);
}
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 596efc940ee70..6a406c9daa042 100644
--- a/drivers/gpu/drm/i915/i915_reg.h
+++ b/drivers/gpu/drm/i915/i915_reg.h
@@ -6818,10 +6818,10 @@ enum skl_power_gate {
#define DP_TP_STATUS_IDLE_DONE (1 << 25)
#define DP_TP_STATUS_ACT_SENT (1 << 24)
#define DP_TP_STATUS_MODE_STATUS_MST (1 << 23)
+#define DP_TP_STATUS_NUM_STREAMS_ENABLED (7 << 16)
#define DP_TP_STATUS_AUTOTRAIN_DONE (1 << 12)
-#define DP_TP_STATUS_PAYLOAD_MAPPING_VC2 (3 << 8)
-#define DP_TP_STATUS_PAYLOAD_MAPPING_VC1 (3 << 4)
-#define DP_TP_STATUS_PAYLOAD_MAPPING_VC0 (3 << 0)
+#define DP_TP_STATUS_PAYLOAD_MAPPING_SHIFT(vc) ((vc) * 4)
+#define DP_TP_STATUS_PAYLOAD_MAPPING_MASK(vc) (3 << DP_TP_STATUS_PAYLOAD_MAPPING_SHIFT(vc))
/* DDI Buffer Control */
#define _DDI_BUF_CTL_A 0x64000
--
2.37.1
More information about the Intel-gfx
mailing list