Hi Geert, Thank you for the patch. On Wed, Jul 05, 2023 at 05:26:16PM +0200, Geert Uytterhoeven wrote: > Add support for drawing the SMPTE and tiles patterns in buffers using > semi-planar YUV formats with non-subsampled chroma planes. > > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > tests/util/pattern.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/tests/util/pattern.c b/tests/util/pattern.c > index 158c0b160a2ee870..f45a26ccec38f202 100644 > --- a/tests/util/pattern.c > +++ b/tests/util/pattern.c > @@ -698,6 +698,8 @@ static void fill_smpte(const struct util_format_info *info, void *planes[3], > case DRM_FORMAT_NV21: > case DRM_FORMAT_NV16: > case DRM_FORMAT_NV61: > + case DRM_FORMAT_NV24: > + case DRM_FORMAT_NV42: > u = info->yuv.order & YUV_YCbCr ? planes[1] : planes[1] + 1; > v = info->yuv.order & YUV_YCrCb ? planes[1] : planes[1] + 1; > return fill_smpte_yuv_planar(&info->yuv, planes[0], u, v, > @@ -1023,6 +1025,8 @@ static void fill_tiles(const struct util_format_info *info, void *planes[3], > case DRM_FORMAT_NV21: > case DRM_FORMAT_NV16: > case DRM_FORMAT_NV61: > + case DRM_FORMAT_NV24: > + case DRM_FORMAT_NV42: > u = info->yuv.order & YUV_YCbCr ? planes[1] : planes[1] + 1; > v = info->yuv.order & YUV_YCrCb ? planes[1] : planes[1] + 1; > return fill_tiles_yuv_planar(info, planes[0], u, v, -- Regards, Laurent Pinchart