Hi Hans, On Thu, Jun 16, 2022 at 09:36:52PM +0300, Laurent Pinchart wrote: > Extend vivid to support the newly added YUVA and YUVX pixel formats > through the TPG. While the ISI driver itself can't be merged yet due to the dependency on the streams series, I was wondering if we could merge patches 1/7 to 3/7 already. They add two new pixel formats needed by the driver, and having them upstream would help merging their support in libcamera. I'll keep working on the ISI driver (and the streams series) to get those in too. If you agree with that, could you review patches 2/7 and 3/7 (as well as 1/7 if you desire) ? They are small. > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > .../media/test-drivers/vivid/vivid-vid-common.c | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/drivers/media/test-drivers/vivid/vivid-vid-common.c b/drivers/media/test-drivers/vivid/vivid-vid-common.c > index 19701fe72030..38d788b5cf19 100644 > --- a/drivers/media/test-drivers/vivid/vivid-vid-common.c > +++ b/drivers/media/test-drivers/vivid/vivid-vid-common.c > @@ -198,6 +198,21 @@ struct vivid_fmt vivid_formats[] = { > .planes = 1, > .buffers = 1, > }, > + { > + .fourcc = V4L2_PIX_FMT_YUVA32, > + .vdownsampling = { 1 }, > + .bit_depth = { 32 }, > + .planes = 1, > + .buffers = 1, > + .alpha_mask = 0xff000000, > + }, > + { > + .fourcc = V4L2_PIX_FMT_YUVX32, > + .vdownsampling = { 1 }, > + .bit_depth = { 32 }, > + .planes = 1, > + .buffers = 1, > + }, > { > .fourcc = V4L2_PIX_FMT_GREY, > .vdownsampling = { 1 }, -- Regards, Laurent Pinchart