Re: [PATCH v5 1/6] drm/i915/vrr: Add crtc_state dump for vrr.vsync params

["Nautiyal, Ankit K" <ankit.k.nautiyal@intel.com>]

On 12/2/2024 8:15 AM, Mitul Golani wrote:
> Add crtc_state dump for vrr.vsync_{start/end} params to track the
> state correctly.
>
> --v2:
> - remove vrr_ pretext and use space instead of underscore (Jani).
>
> Signed-off-by: Mitul Golani <mitulkumar.ajitkumar.golani@intel.com>


Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>


> ---
>   drivers/gpu/drm/i915/display/intel_crtc_state_dump.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c b/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> index 705ec5ad385c..221be4f44104 100644
> --- a/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> +++ b/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> @@ -296,11 +296,13 @@ void intel_crtc_state_dump(const struct intel_crtc_state *pipe_config,
>   		intel_dump_buffer("ELD: ", pipe_config->eld,
>   				  drm_eld_size(pipe_config->eld));
>   
> -	drm_printf(&p, "vrr: %s, vmin: %d, vmax: %d, pipeline full: %d, guardband: %d flipline: %d, vmin vblank: %d, vmax vblank: %d\n",
> +	drm_printf(&p, "vrr: %s, vmin: %d, vmax: %d, pipeline full: %d, guardband: %d flipline: %d, vsync start: %d, vsync end: %d, vmin vblank: %d, vmax vblank: %d\n",
>   		   str_yes_no(pipe_config->vrr.enable),
>   		   pipe_config->vrr.vmin, pipe_config->vrr.vmax,
>   		   pipe_config->vrr.pipeline_full, pipe_config->vrr.guardband,
>   		   pipe_config->vrr.flipline,
> +		   pipe_config->vrr.vsync_start,
> +		   pipe_config->vrr.vsync_end,
>   		   intel_vrr_vmin_vblank_start(pipe_config),
>   		   intel_vrr_vmax_vblank_start(pipe_config));
>