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@xxxxxxxxxxxxxxxx> > > --- > > .../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
