> -----Original Message-----
> From: Intel-gfx <intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Swati
> Sharma
> Sent: Wednesday, July 6, 2022 3:53 PM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Subject: [PATCH] drm/i915/display: Add debug print for scaler filter
>
> Add debug print statement to print scaler filter property value. Since property can be
> set as either default or integer scaler; its good if we can get debug print for the same
> in dmesg log.
Looks Good to me.
Reviewed-by: Uma Shankar <uma.shankar@xxxxxxxxx>
> Cc: Juha-Pekka Heikkila <juhapekka.heikkila@xxxxxxxxx>
> Signed-off-by: Swati Sharma <swati2.sharma@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/display/intel_crtc_state_dump.c | 9 +++++----
> drivers/gpu/drm/i915/display/intel_display_debugfs.c | 5 +++--
> 2 files changed, 8 insertions(+), 6 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 4ca6e9493ff2..e9212f69c360 100644
> --- a/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> +++ b/drivers/gpu/drm/i915/display/intel_crtc_state_dump.c
> @@ -134,8 +134,8 @@ static void intel_dump_plane_state(const struct
> intel_plane_state *plane_state)
> plane->base.base.id, plane->base.name,
> fb->base.id, fb->width, fb->height, &fb->format->format,
> fb->modifier, str_yes_no(plane_state->uapi.visible));
> - drm_dbg_kms(&i915->drm, "\trotation: 0x%x, scaler: %d\n",
> - plane_state->hw.rotation, plane_state->scaler_id);
> + drm_dbg_kms(&i915->drm, "\trotation: 0x%x, scaler: %d, scaling_filter:
> %d\n",
> + plane_state->hw.rotation, plane_state->scaler_id,
> +plane_state->hw.scaling_filter);
> if (plane_state->uapi.visible)
> drm_dbg_kms(&i915->drm,
> "\tsrc: " DRM_RECT_FP_FMT " dst: " DRM_RECT_FMT
> "\n", @@ -262,10 +262,11 @@ void intel_crtc_state_dump(const struct
> intel_crtc_state *pipe_config,
>
> if (DISPLAY_VER(i915) >= 9)
> drm_dbg_kms(&i915->drm,
> - "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d\n",
> + "num_scalers: %d, scaler_users: 0x%x, scaler_id: %d,
> +scaling_filter: %d\n",
> crtc->num_scalers,
> pipe_config->scaler_state.scaler_users,
> - pipe_config->scaler_state.scaler_id);
> + pipe_config->scaler_state.scaler_id,
> + pipe_config->hw.scaling_filter);
>
> if (HAS_GMCH(i915))
> drm_dbg_kms(&i915->drm,
> diff --git a/drivers/gpu/drm/i915/display/intel_display_debugfs.c
> b/drivers/gpu/drm/i915/display/intel_display_debugfs.c
> index 6c3954479047..225b6bfc783c 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_debugfs.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_debugfs.c
> @@ -722,10 +722,11 @@ static void intel_scaler_info(struct seq_file *m, struct
> intel_crtc *crtc)
>
> /* Not all platformas have a scaler */
> if (num_scalers) {
> - seq_printf(m, "\tnum_scalers=%d, scaler_users=%x scaler_id=%d",
> + seq_printf(m, "\tnum_scalers=%d, scaler_users=%x scaler_id=%d
> +scaling_filter=%d",
> num_scalers,
> crtc_state->scaler_state.scaler_users,
> - crtc_state->scaler_state.scaler_id);
> + crtc_state->scaler_state.scaler_id,
> + crtc_state->hw.scaling_filter);
>
> for (i = 0; i < num_scalers; i++) {
> const struct intel_scaler *sc =
> --
> 2.25.1
