[PATCH 07/15] drm/i915/psr: Add function to compute max link-wake latency

Jani Nikula jani.nikula at linux.intel.com
Mon Aug 4 20:54:55 UTC 2025


On Mon, 04 Aug 2025, Ankit Nautiyal <ankit.k.nautiyal at intel.com> wrote:
> Introduce a helper to compute the max link wake latency when using
> Auxless/Aux wake mechanism for PSR/Panel Replay/LOBF features.
>
> This will be used to compute the minimum guardband so that the link wake
> latencies are accounted and these features work smoothly for higher
> refresh rate panels.
>
> Bspec: 70151, 71477
> Signed-off-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_psr.c | 64 ++++++++++++++++++++++++
>  drivers/gpu/drm/i915/display/intel_psr.h |  3 ++
>  2 files changed, 67 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i915/display/intel_psr.c
> index 6bd3454bb00e..6cdaff3ccc9f 100644
> --- a/drivers/gpu/drm/i915/display/intel_psr.c
> +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> @@ -33,6 +33,7 @@
>  #include "intel_atomic.h"
>  #include "intel_crtc.h"
>  #include "intel_cursor_regs.h"
> +#include "intel_cx0_phy.h"
>  #include "intel_ddi.h"
>  #include "intel_de.h"
>  #include "intel_display_irq.h"
> @@ -4249,3 +4250,66 @@ bool intel_psr_needs_alpm_aux_less(struct intel_dp *intel_dp,
>  {
>  	return intel_dp_is_edp(intel_dp) && crtc_state->has_panel_replay;
>  }
> +
> +static
> +int intel_psr_compute_aux_wake_latency(struct intel_dp *intel_dp,
> +				       struct intel_crtc_state *crtc_state)
> +{
> +#define TFW_EXIT_LATENCY_MS		20000
> +#define FAST_WAKE_LATENCY_MS		12000 /* Preamble: 8us; PHY wake: 4us */
> +	struct intel_encoder *encoder = &dp_to_dig_port(intel_dp)->base;
> +	int aux_wake_latency_us;
> +	int io_buffer_wake_ms;
> +
> +	io_buffer_wake_ms = intel_encoder_is_c10phy(encoder) ? 9790 : 14790;
> +
> +	aux_wake_latency_us =
> +		DIV_ROUND_UP(io_buffer_wake_ms + TFW_EXIT_LATENCY_MS + FAST_WAKE_LATENCY_MS, 1000);

See https://lore.kernel.org/r/eeda84457c813151a3459a46a91946b4fbbb9e44@intel.com

> +
> +	return aux_wake_latency_us;
> +}
> +
> +static
> +int intel_psr_compute_auxless_latency(struct intel_crtc_state *crtc_state)
> +{
> +#define PHY_ESTABLISHMENT_PERIOD_MS	50000
> +#define LFPS_PERIOD_MS			800
> +#define SILENCE_MAX_MS			180
> +	int linkrate_mhz = crtc_state->port_clock / 1000;
> +	int clock_data_switch_ms;
> +	int auxless_latency_us;
> +	int time_ml_phy_lock_ms;
> +	int num_ml_phy_lock;
> +	/*
> +	 * TPS4 length = 252
> +	 * tML_PHY_LOCK = TPS4 Length * ( 10 / (Link Rate in MHz) )
> +	 * Number ML_PHY_LOCK = ( 7 + CEILING(6.5us / tML_PHY_LOCK ) + 1)
> +	 * t2 = Number ML_PHY_LOCK * tML_PHY_LOCK
> +	 * tCDS term  = 2 * t2
> +	 * =>tCDS_term  = 2 * (7 * (252 * (10 /linkrate))+6.5)
> +	 */
> +	time_ml_phy_lock_ms = (1000 * 252 * 10) / linkrate_mhz;
> +	num_ml_phy_lock = 7 + DIV_ROUND_UP(6500 * 1000, time_ml_phy_lock_ms) / 1000 + 1;
> +	clock_data_switch_ms = 2 * time_ml_phy_lock_ms * num_ml_phy_lock;
> +
> +	auxless_latency_us = (LFPS_PERIOD_MS  + SILENCE_MAX_MS + PHY_ESTABLISHMENT_PERIOD_MS +
> +			      clock_data_switch_ms) / 1000;
> +
> +	return auxless_latency_us;
> +}
> +
> +int intel_psr_compute_max_link_wake_latency(struct intel_dp *intel_dp,
> +					    struct intel_crtc_state *crtc_state,
> +					    bool assume_all_enabled)
> +{
> +	int aux_wake_latency = 0;
> +	int auxless_latency = 0;
> +
> +	if (assume_all_enabled || crtc_state->has_sel_update)
> +		auxless_latency = intel_psr_compute_aux_wake_latency(intel_dp, crtc_state);
> +
> +	if (assume_all_enabled || crtc_state->has_panel_replay)
> +		aux_wake_latency = intel_psr_compute_auxless_latency(crtc_state);
> +
> +	return max(auxless_latency, aux_wake_latency);
> +}
> diff --git a/drivers/gpu/drm/i915/display/intel_psr.h b/drivers/gpu/drm/i915/display/intel_psr.h
> index 9b061a22361f..c58d29620b49 100644
> --- a/drivers/gpu/drm/i915/display/intel_psr.h
> +++ b/drivers/gpu/drm/i915/display/intel_psr.h
> @@ -81,5 +81,8 @@ void intel_psr_debugfs_register(struct intel_display *display);
>  bool intel_psr_needs_alpm(struct intel_dp *intel_dp, const struct intel_crtc_state *crtc_state);
>  bool intel_psr_needs_alpm_aux_less(struct intel_dp *intel_dp,
>  				   const struct intel_crtc_state *crtc_state);
> +int intel_psr_compute_max_link_wake_latency(struct intel_dp *intel_dp,
> +					    struct intel_crtc_state *crtc_state,
> +					    bool assume_all_enabled);
>  
>  #endif /* __INTEL_PSR_H__ */

-- 
Jani Nikula, Intel


More information about the Intel-xe mailing list