Re: [PATCH 7/9] drm/i915/wm: Verify 'ddb_y' as well as 'ddb'

Govindapillai, Vinod
2026-04-08T11:59:54Z

On Tue, 2026-03-24 at 15:48 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Verify the pre-icl NV12 Y color plane DDB entry. Thus far
> we've only verified the RGB/UV DDB entry.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
>  drivers/gpu/drm/i915/display/skl_watermark.c | 4 ++++
>  1 file changed, 4 insertions(+)
>

Reviewed-by: Vinod Govindapillai <vinod.govindapillai@intel.com>

> diff --git a/drivers/gpu/drm/i915/display/skl_watermark.c
> b/drivers/gpu/drm/i915/display/skl_watermark.c
> index 17faf090a154..1d932c37d768 100644
> --- a/drivers/gpu/drm/i915/display/skl_watermark.c
> +++ b/drivers/gpu/drm/i915/display/skl_watermark.c
> @@ -4022,6 +4022,10 @@ void intel_wm_state_verify(struct
> intel_atomic_state *state,
>   skl_ddb_entry_verify(plane, "DDB",
>        &hw->ddb[plane->id],
>        &new_crtc_state-
> >wm.skl.plane_ddb[plane->id]);
> +
> + skl_ddb_entry_verify(plane, "DDB Y",
> +      &hw->ddb_y[plane->id],
> +      &new_crtc_state-
> >wm.skl.plane_ddb_y[plane->id]);
>   }
>
>   kfree(hw);