Hi Andrea, Thank you for the patch. On Friday, 28 September 2018 10:32:13 EEST Andrea Merello wrote: > In formats table the entry for CFA pattern "rggb" has GRBG fourcc. > This patch fixes it. > > Cc: linux-media@xxxxxxxxxxxxxxx > Signed-off-by: Mirco Di Salvo <mirco.disalvo@xxxxxx> > Signed-off-by: Andrea Merello <andrea.merello@xxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> Michal, should I take the patch in my tree ? > --- > drivers/media/platform/xilinx/xilinx-vip.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/platform/xilinx/xilinx-vip.c > b/drivers/media/platform/xilinx/xilinx-vip.c index > 311259129504..e9567cdfb89b 100644 > --- a/drivers/media/platform/xilinx/xilinx-vip.c > +++ b/drivers/media/platform/xilinx/xilinx-vip.c > @@ -36,7 +36,7 @@ static const struct xvip_video_format xvip_video_formats[] > = { { XVIP_VF_MONO_SENSOR, 8, "mono", MEDIA_BUS_FMT_Y8_1X8, > 1, V4L2_PIX_FMT_GREY, "Greyscale 8-bit" }, > { XVIP_VF_MONO_SENSOR, 8, "rggb", MEDIA_BUS_FMT_SRGGB8_1X8, > - 1, V4L2_PIX_FMT_SGRBG8, "Bayer 8-bit RGGB" }, > + 1, V4L2_PIX_FMT_SRGGB8, "Bayer 8-bit RGGB" }, > { XVIP_VF_MONO_SENSOR, 8, "grbg", MEDIA_BUS_FMT_SGRBG8_1X8, > 1, V4L2_PIX_FMT_SGRBG8, "Bayer 8-bit GRBG" }, > { XVIP_VF_MONO_SENSOR, 8, "gbrg", MEDIA_BUS_FMT_SGBRG8_1X8, -- Regards, Laurent Pinchart