[PATCH v9 12/12] drm/i915/psr: Add panel replay sel update support to debugfs interface

[Manna, Animesh]

> -----Original Message-----
> From: Hogander, Jouni <jouni.hogander at intel.com>
> Sent: Friday, May 3, 2024 12:04 PM
> To: intel-gfx at lists.freedesktop.org
> Cc: Manna, Animesh <animesh.manna at intel.com>; Hogander, Jouni
> <jouni.hogander at intel.com>; Joshi, Kunal1 <kunal1.joshi at intel.com>
> Subject: [PATCH v9 12/12] drm/i915/psr: Add panel replay sel update
> support to debugfs interface
> 
> Add panel replay selective update support to debugfs status interface. In
> case of sink supporting panel replay we will print out:
> 
> Sink support: PSR = no, Panel Replay = yes, Panel Replay Selective Update =
> yes
> 
> and PSR mode will look like this if printing out enabled panel replay selective
> update:
> 
> PSR mode: Panel Replay Selective Update Enabled
> 
> Current PSR and panel replay printouts remain same.
> 
> Cc: Kunal Joshi <kunal1.joshi at intel.com>
> 
> Signed-off-by: Jouni Högander <jouni.hogander at intel.com>

Reviewed-by: Animesh Manna <animesh.manna at intel.com>

> ---
>  drivers/gpu/drm/i915/display/intel_psr.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_psr.c
> b/drivers/gpu/drm/i915/display/intel_psr.c
> index 2198448fdb27..2ad7964998e0 100644
> --- a/drivers/gpu/drm/i915/display/intel_psr.c
> +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> @@ -3575,7 +3575,9 @@ static int intel_psr_status(struct seq_file *m, struct
> intel_dp *intel_dp)
> 
>  	if (psr->sink_support)
>  		seq_printf(m, " [0x%02x]", intel_dp->psr_dpcd[0]);
> -	seq_printf(m, ", Panel Replay = %s\n", str_yes_no(psr-
> >sink_panel_replay_support));
> +	seq_printf(m, ", Panel Replay = %s", str_yes_no(psr-
> >sink_panel_replay_support));
> +	seq_printf(m, ", Panel Replay Selective Update = %s\n",
> +		   str_yes_no(psr->sink_panel_replay_su_support));
> 
>  	if (!(psr->sink_support || psr->sink_panel_replay_support))
>  		return 0;
> @@ -3584,9 +3586,10 @@ static int intel_psr_status(struct seq_file *m,
> struct intel_dp *intel_dp)
>  	mutex_lock(&psr->lock);
> 
>  	if (psr->panel_replay_enabled)
> -		status = "Panel Replay Enabled";
> +		status = psr->sel_update_enabled ? "Panel Replay Selective
> Update Enabled" :
> +			"Panel Replay Enabled";
>  	else if (psr->enabled)
> -		status = psr->sel_update_enabled ? "PSR2 enabled" : "PSR1
> enabled";
> +		status = psr->sel_update_enabled ? "PSR2" : "PSR1";
>  	else
>  		status = "disabled";
>  	seq_printf(m, "PSR mode: %s\n", status);
> --
> 2.34.1