[PATCH 08/18] drm/i915: Include the scanline offset in the state dump

[Nautiyal, Ankit K]

On 12/11/2024 2:39 AM, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> When looking at raw hardware scanline numbers it's helpful to
> remember what the offset between the hardware values and our
> more human readable numbers should be. Include that in the state dump.
>
> 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 | 4 ++++
>   1 file changed, 4 insertions(+)
>
> 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 97e3cdccda01..1fbaa67e2fea 100644
> --- a/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> +++ b/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> @@ -10,6 +10,7 @@
>   #include "intel_crtc_state_dump.h"
>   #include "intel_display_types.h"
>   #include "intel_hdmi.h"
> +#include "intel_vblank.h"
>   #include "intel_vdsc.h"
>   #include "intel_vrr.h"
>   
> @@ -283,6 +284,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, "scanline offset: %d\n",
> +		   intel_crtc_scanline_offset(pipe_config));
> +
>   	drm_printf(&p, "vblank delay: %d, framestart delay: %d, MSA timing delay: %d\n",
>   		   pipe_config->hw.adjusted_mode.crtc_vblank_start -
>   		   pipe_config->hw.adjusted_mode.crtc_vdisplay,