Thank you so much Jouni. Not sure if I am considered authorized to provide RB.
But if can count, 
Reviewed-by: Vidya Srinivas <[email protected]>

Regards
Vidya

> -----Original Message-----
> From: Hogander, Jouni <[email protected]>
> Sent: 12 May 2026 13:30
> To: [email protected]; [email protected]
> Cc: Hogander, Jouni <[email protected]>; Srinivas, Vidya
> <[email protected]>
> Subject: [PATCH] drm/i915/display: Handle odd position for planar formats in
> selective fetch
> 
> Since Lunarlake there is no restriction planar planes has to be even 
> positions.
> Due to this we may end up having odd offset for UV-plane in selective fetch
> configuration. Add handling for this case into selective fetch configuration.
> 
> Bspec: 68927
> Suggested-by: Vidya Srinivas <[email protected]>
> Signed-off-by: Jouni Högander <[email protected]>
> ---
>  drivers/gpu/drm/i915/display/skl_universal_plane.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c
> b/drivers/gpu/drm/i915/display/skl_universal_plane.c
> index ef431dd32e74..ad4bfff6903d 100644
> --- a/drivers/gpu/drm/i915/display/skl_universal_plane.c
> +++ b/drivers/gpu/drm/i915/display/skl_universal_plane.c
> @@ -1532,7 +1532,7 @@ static void
> icl_plane_update_sel_fetch_noarm(struct intel_dsb *dsb,
>       if (!color_plane)
>               y = plane_state->view.color_plane[color_plane].y + clip->y1;
>       else
> -             y = plane_state->view.color_plane[color_plane].y + clip->y1 /
> 2;
> +             y = plane_state->view.color_plane[color_plane].y +
> +DIV_ROUND_UP(clip->y1, 2);
> 
>       val = y << 16 | x;
> 
> --
> 2.43.0

Reply via email to