Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] drm/i915/ltphy: Remove state verification for LT PHY fields
@ 2025-12-31  5:23 Suraj Kandpal
  2025-12-31  5:23 ` [PATCH 2/3] drm/i915/ltphy: Compare only certain fields in state verify function Suraj Kandpal
                   ` (6 more replies)
  0 siblings, 7 replies; 12+ messages in thread
From: Suraj Kandpal @ 2025-12-31  5:23 UTC (permalink / raw)
  To: intel-xe, intel-gfx; +Cc: ankit.k.nautiyal, Suraj Kandpal

Remove LT PHY State verification for all VDR register fields other
than VDR0_CONFIG and VDR2_CONFIG since those are the only reliable
shadow register that hold on to its value in case there is PSR/PR
which can cause power gating internally of the PHY.

Signed-off-by: Suraj Kandpal <suraj.kandpal@intel.com>
---
 drivers/gpu/drm/i915/display/intel_lt_phy.c | 30 +++++----------------
 1 file changed, 7 insertions(+), 23 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_lt_phy.c b/drivers/gpu/drm/i915/display/intel_lt_phy.c
index 939c8975fd4c..9501ac861712 100644
--- a/drivers/gpu/drm/i915/display/intel_lt_phy.c
+++ b/drivers/gpu/drm/i915/display/intel_lt_phy.c
@@ -2259,8 +2259,6 @@ void intel_lt_phy_pll_state_verify(struct intel_atomic_state *state,
 	struct intel_encoder *encoder;
 	struct intel_lt_phy_pll_state pll_hw_state = {};
 	const struct intel_lt_phy_pll_state *pll_sw_state = &new_crtc_state->dpll_hw_state.ltpll;
-	int clock;
-	int i, j;
 
 	if (DISPLAY_VER(display) < 35)
 		return;
@@ -2275,33 +2273,19 @@ void intel_lt_phy_pll_state_verify(struct intel_atomic_state *state,
 
 	encoder = intel_get_crtc_new_encoder(state, new_crtc_state);
 	intel_lt_phy_pll_readout_hw_state(encoder, new_crtc_state, &pll_hw_state);
-	clock = intel_lt_phy_calc_port_clock(encoder, new_crtc_state);
 
 	dig_port = enc_to_dig_port(encoder);
 	if (intel_tc_port_in_tbt_alt_mode(dig_port))
 		return;
 
-	INTEL_DISPLAY_STATE_WARN(display, pll_hw_state.clock != clock,
-				 "[CRTC:%d:%s] mismatch in LT PHY: Register CLOCK (expected %d, found %d)",
+	INTEL_DISPLAY_STATE_WARN(display, pll_hw_state.config[0] != pll_sw_state->config[0],
+				 "[CRTC:%d:%s] mismatch in LT PHY PLL CONFIG 0: (expected 0x%04x, found 0x%04x)",
 				 crtc->base.base.id, crtc->base.name,
-				 pll_sw_state->clock, pll_hw_state.clock);
-
-	for (i = 0; i < 3; i++) {
-		INTEL_DISPLAY_STATE_WARN(display, pll_hw_state.config[i] != pll_sw_state->config[i],
-					 "[CRTC:%d:%s] mismatch in LT PHY PLL CONFIG%d: (expected 0x%04x, found 0x%04x)",
-					 crtc->base.base.id, crtc->base.name, i,
-					 pll_sw_state->config[i], pll_hw_state.config[i]);
-	}
-
-	for (i = 0; i <= 12; i++) {
-		for (j = 3; j >= 0; j--)
-			INTEL_DISPLAY_STATE_WARN(display,
-						 pll_hw_state.data[i][j] !=
-						 pll_sw_state->data[i][j],
-						 "[CRTC:%d:%s] mismatch in LT PHY PLL DATA[%d][%d]: (expected 0x%04x, found 0x%04x)",
-						 crtc->base.base.id, crtc->base.name, i, j,
-						 pll_sw_state->data[i][j], pll_hw_state.data[i][j]);
-	}
+				 pll_sw_state->config[0], pll_hw_state.config[0]);
+	INTEL_DISPLAY_STATE_WARN(display, pll_hw_state.config[2] != pll_sw_state->config[2],
+				 "[CRTC:%d:%s] mismatch in LT PHY PLL CONFIG 2: (expected 0x%04x, found 0x%04x)",
+				 crtc->base.base.id, crtc->base.name,
+				 pll_sw_state->config[2], pll_hw_state.config[2]);
 }
 
 void intel_xe3plpd_pll_enable(struct intel_encoder *encoder,
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2026-01-05  5:57 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-31  5:23 [PATCH 1/3] drm/i915/ltphy: Remove state verification for LT PHY fields Suraj Kandpal
2025-12-31  5:23 ` [PATCH 2/3] drm/i915/ltphy: Compare only certain fields in state verify function Suraj Kandpal
2026-01-05  5:29   ` Nautiyal, Ankit K
2025-12-31  5:23 ` [PATCH 3/3] drm/i915/ltphy: Provide protection against unsupported modes Suraj Kandpal
2026-01-05  5:30   ` Nautiyal, Ankit K
2025-12-31  5:30 ` ✓ CI.KUnit: success for series starting with [1/3] drm/i915/ltphy: Remove state verification for LT PHY fields Patchwork
2025-12-31  6:10 ` ✓ Xe.CI.BAT: " Patchwork
2025-12-31  7:10 ` ✓ Xe.CI.Full: " Patchwork
2026-01-05  5:27 ` [PATCH 1/3] " Nautiyal, Ankit K
2026-01-05  5:57 ` [PATCH v2 " Suraj Kandpal
2026-01-05  5:57   ` [PATCH v2 2/3] drm/i915/ltphy: Compare only certain fields in state verify function Suraj Kandpal
2026-01-05  5:57   ` [PATCH v2 3/3] drm/i915/ltphy: Provide protection against unsupported modes Suraj Kandpal

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox