[PATCH v6 01/26] drm/i915/alpm: Do not use fast_wake_lines for aux less wake time

[Manna, Animesh]

> -----Original Message-----
> From: Hogander, Jouni <jouni.hogander at intel.com>
> Sent: Wednesday, June 5, 2024 3:55 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 01/26] drm/i915/alpm: Do not use fast_wake_lines for
> aux less wake time
> 
> We want to have own variables for fast wake lines and aux less wake time. It
> might be needed to choose if we can enable Panel Replay Selective Update or
> PSR2.
> 
> Also currently aux less wake time is overwritten by calculated fast wake time.
> 
> v2:use less wake time in intel_alpm_lobf_compute_config

Typo aux_less_*. With this fixed,
Reviewed-by: Animesh Manna <animesh.manna at intel.com>

Regards,
Animesh
> 
> Fixes: da6a9836ac09 ("drm/i915/psr: Calculate aux less wake time")
> Signed-off-by: Jouni Högander <jouni.hogander at intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_alpm.c          | 4 ++--
>  drivers/gpu/drm/i915/display/intel_display_types.h | 1 +
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_alpm.c
> b/drivers/gpu/drm/i915/display/intel_alpm.c
> index a26716c14aa3..52a573367976 100644
> --- a/drivers/gpu/drm/i915/display/intel_alpm.c
> +++ b/drivers/gpu/drm/i915/display/intel_alpm.c
> @@ -159,7 +159,7 @@ static int
> _lnl_compute_aux_less_alpm_params(struct intel_dp *intel_dp,
>  	if (i915->display.params.psr_safest_params)
>  		aux_less_wake_lines =
> ALPM_CTL_AUX_LESS_WAKE_TIME_MASK;
> 
> -	intel_dp->alpm_parameters.fast_wake_lines = aux_less_wake_lines;
> +	intel_dp->alpm_parameters.aux_less_wake_lines =
> aux_less_wake_lines;
>  	intel_dp->alpm_parameters.silence_period_sym_clocks =
> silence_period;
>  	intel_dp->alpm_parameters.lfps_half_cycle_num_of_syms =
> lfps_half_cycle;
> 
> @@ -298,7 +298,7 @@ void intel_alpm_lobf_compute_config(struct intel_dp
> *intel_dp,
>  	if (intel_alpm_aux_less_wake_supported(intel_dp))
>  		waketime_in_lines = intel_dp-
> >alpm_parameters.io_wake_lines;
>  	else
> -		waketime_in_lines = intel_dp-
> >alpm_parameters.fast_wake_lines;
> +		waketime_in_lines = intel_dp-
> >alpm_parameters.aux_less_wake_lines;
> 
>  	crtc_state->has_lobf = (context_latency + guardband) >
>  		(first_sdp_position + waketime_in_lines); diff --git
> a/drivers/gpu/drm/i915/display/intel_display_types.h
> b/drivers/gpu/drm/i915/display/intel_display_types.h
> index 77609656317b..df29ec37ef28 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_types.h
> +++ b/drivers/gpu/drm/i915/display/intel_display_types.h
> @@ -1847,6 +1847,7 @@ struct intel_dp {
> 
>  		/* LNL and beyond */
>  		u8 check_entry_lines;
> +		u8 aux_less_wake_lines;
>  		u8 silence_period_sym_clocks;
>  		u8 lfps_half_cycle_num_of_syms;
>  	} alpm_parameters;
> --
> 2.34.1