Re: [PATCH v2 19/19] media: v4l2: Add 10-, 12- and 16-bpc 4:2:0 and 4:2:2 semi-planar YUV formats

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Hans,

On Mon, Nov 16, 2020 at 01:17:10PM +0100, Hans Verkuil wrote:
> On 02/11/2020 23:41, Laurent Pinchart wrote:
> > Add semi-planar YUV formats with 10, 12 and 16 bits per components, in
> > both 4:2:0 (NV12) and 4:2:2 (NV16) subsampling, and with both chroma
> > orderings. They are used by the Xilinx Video Frame Buffer Read/Write IP
> > cores.
> > 
> > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > ---
> >  .../media/v4l/pixfmt-yuv-planar.rst           | 564 +++++++++++++++++-
> >  include/uapi/linux/videodev2.h                |  12 +
> >  2 files changed, 575 insertions(+), 1 deletion(-)
> > 
> > diff --git a/Documentation/userspace-api/media/v4l/pixfmt-yuv-planar.rst b/Documentation/userspace-api/media/v4l/pixfmt-yuv-planar.rst
> > index 07081ab05419..7f0fe2e4b668 100644
> > --- a/Documentation/userspace-api/media/v4l/pixfmt-yuv-planar.rst
> > +++ b/Documentation/userspace-api/media/v4l/pixfmt-yuv-planar.rst
> 
> <snip>
> 
> >  Fully Planar YUV Formats
> >  ========================
> >  
> 
> I found a mistake in the "Overview of Fully Planar YUV Formats" table:
> 
> For formats V4L2_PIX_FMT_YVU420M and V4L2_PIX_FMT_YVU444M it gives "Y, Cb, Cr" as the
> planes order, this should be "Y, Cr, Cb". Can you fix that in a new patch?

Thanks for spotting this. I'll fix it in v3 of "[PATCH v2 14/19] media:
doc: pixfmt-yuv: Move all planar YUV formats to common file", as that's
where the error was introduced.

-- 
Regards,

Laurent Pinchart