Re: [PATCH 0/3] media: uapi: videodev2: Add 20-, 24- and 28-bit bayer pixel formats

[Niklas Söderlund]

Hi Jacopo,

Thanks for your review.

On 2026-11-12 11:16:09 +0100, Jacopo Mondi wrote:
> Hi Niklas,
> 
> On Thu, Sep 10, 2026 at 07:22:30PM +0200, Niklas Söderlund wrote:
> > Hello,
> >
> > This series adds the pixel formats and documentation for the 20-, 24-
> > and 28-bit Bayer formats. This is done in preparation of work on the
> > Renesas R-Car video capture pipeline which will support these pixel
> > formats in multiple drivers. For this reason it is posted as a separate
> > series before the driver work.
> >
> > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> > ---
> > Niklas Söderlund (3):
> >       media: uapi: videodev2: Add 20-bit Bayer formats
> >       media: uapi: videodev2: Add 24-bit Bayer formats
> >       media: uapi: videodev2: Add 28-bit Bayer formats
> >
> >  .../userspace-api/media/v4l/pixfmt-bayer.rst       |   3 +
> >  .../userspace-api/media/v4l/pixfmt-srggb20.rst     | 102 +++++++++++++++++++++
> >  .../userspace-api/media/v4l/pixfmt-srggb24.rst     | 102 +++++++++++++++++++++
> >  .../userspace-api/media/v4l/pixfmt-srggb28.rst     | 102 +++++++++++++++++++++
> >  drivers/media/v4l2-core/v4l2-ioctl.c               |  12 +++
> >  include/uapi/linux/videodev2.h                     |  12 +++
> 
> Have you forgot to add the new formats to the table in
> v4l2_format_info() in drivers/media/v4l2-core/v4l2-common.c ?

Yes I have, thanks for catching this. Will fix.

> 
> 
> 
> >  6 files changed, 333 insertions(+)
> > ---
> > base-commit: f9536a8065d9eed0d9114878c56ffc4e5a18bdfc
> > change-id: 20260908-v4l2-add-raw-pix-50ca1182ccd2
> >
> > Best regards,
> > --
> > Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> >
> >

-- 
Kind Regards,
Niklas Söderlund