[PATCH 05/18] drm/i915: Move framestart/etc. state dump to a better spot

[Nautiyal, Ankit K]

On 12/11/2024 2:39 AM, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> Try to dump all the important stuff relating to the display timings
> in one spot.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

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


> ---
>   drivers/gpu/drm/i915/display/intel_crtc_state_dump.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> 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 1faef60be472..9d930ff862cf 100644
> --- a/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> +++ b/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> @@ -250,9 +250,6 @@ void intel_crtc_state_dump(const struct intel_crtc_state *pipe_config,
>   			   str_enabled_disabled(pipe_config->enable_psr2_sel_fetch));
>   	}
>   
> -	drm_printf(&p, "framestart delay: %d, MSA timing delay: %d\n",
> -		   pipe_config->framestart_delay, pipe_config->msa_timing_delay);
> -
>   	drm_printf(&p, "audio: %i, infoframes: %i, infoframes enabled: 0x%x\n",
>   		   pipe_config->has_audio, pipe_config->has_infoframe,
>   		   pipe_config->infoframes.enable);
> @@ -286,6 +283,9 @@ void intel_crtc_state_dump(const struct intel_crtc_state *pipe_config,
>   		drm_print_hex_dump(&p, "ELD: ", pipe_config->eld,
>   				   drm_eld_size(pipe_config->eld));
>   
> +	drm_printf(&p, "framestart delay: %d, MSA timing delay: %d\n",
> +		   pipe_config->framestart_delay, pipe_config->msa_timing_delay);
> +
>   	drm_printf(&p, "vrr: %s, vmin: %d, vmax: %d, pipeline full: %d, guardband: %d flipline: %d, vmin vblank: %d, vmax vblank: %d\n",
>   		   str_yes_no(pipe_config->vrr.enable),
>   		   pipe_config->vrr.vmin, pipe_config->vrr.vmax,