[PATCH v6 14/26] dmr/i915/psr: Wake time is aux less wake time for Panel Replay

[Manna, Animesh]

> -----Original Message-----
> From: Hogander, Jouni <jouni.hogander at intel.com>
> Sent: Wednesday, June 5, 2024 3:56 PM
> To: intel-gfx at lists.freedesktop.org
> Cc: Manna, Animesh <animesh.manna at intel.com>; Kahola, Mika
> <mika.kahola at intel.com>; Hogander, Jouni <jouni.hogander at intel.com>
> Subject: [PATCH v6 14/26] dmr/i915/psr: Wake time is aux less wake time for
> Panel Replay
> 
> When checking vblank length used wake time is aux less wake time for eDP
> Panel Replay (vblank length is not checked for DP2.0 Panel Replay).
> 
> Signed-off-by: Jouni Högander <jouni.hogander at intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_psr.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_psr.c
> b/drivers/gpu/drm/i915/display/intel_psr.c
> index c9fcb25184ee..ccb7c022e364 100644
> --- a/drivers/gpu/drm/i915/display/intel_psr.c
> +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> @@ -1251,8 +1251,12 @@ static bool vblank_length_valid(struct intel_dp
> *intel_dp,
>  		crtc_state->hw.adjusted_mode.crtc_vblank_start;
>  	int wake_lines;
> 
> -	wake_lines = DISPLAY_VER(i915) < 20 ?
> psr2_block_count_lines(intel_dp) :
> -		intel_dp->alpm_parameters.io_wake_lines;
> +	if (crtc_state->has_panel_replay)
> +		wake_lines = intel_dp-
> >alpm_parameters.aux_less_wake_lines;

Existing code can take care for EDP panel replay as EDP panel replay is supported from LNL onwards.
Not sure why this change is needed.

Regards,
Animesh
  
> +	else
> +		wake_lines = DISPLAY_VER(i915) < 20 ?
> +			psr2_block_count_lines(intel_dp) :
> +			intel_dp->alpm_parameters.io_wake_lines;
> 
>  	if (crtc_state->req_psr2_sdp_prior_scanline)
>  		vblank -= 1;
> --
> 2.34.1