[libcamera-devel] [PATCH v7 0/5] v4l2_(sub)dev: improvements and tests

[Jacopo Mondi]

Hi Laurent,

On Fri, Mar 01, 2019 at 08:01:28PM +0200, Laurent Pinchart wrote:
> Hi Jacopo,
>
> Thank you for the patches.
>
> On Fri, Mar 01, 2019 at 04:40:58PM +0100, Jacopo Mondi wrote:
> > Hello,
> >   few minor changes to address Laurent's comments.
> >
> > 1) propagate error on ENUM_FRAME_SIZE (all errors != from EINVAL and ENOTTY,
> >    I wonder if subdevices can implement ENUM_MBUS but not ENUM_FRAME_SIZE
> > 2) remove method reordering in v4l2_device cosmetic patch
> > 3) use std::vector.empty() instead of (std::vector.size() == 0) in list
> >    formats test.
> >
> > Hopefully this version might be pushed.
>
> For the wholes series,
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Thanks, series now pushed to master

>
> > Jacopo Mondi (5):
> >   libcamera: v4l2_subdevice: Implement ENUM_FRAME_SIZES
> >   test: v4l2_subdevice: Add ListFormat test
> >   libcamera: v4l2_subdevice: Cosmetic update
> >   libcamera: v4l2_device: Add support for META_CAPTURE devices
> >   libcamera: v4l2_device: Cosmetic update
> >
> >  src/libcamera/geometry.cpp             |  34 ++++++++
> >  src/libcamera/include/geometry.h       |  12 +++
> >  src/libcamera/include/v4l2_device.h    |  26 +++++-
> >  src/libcamera/include/v4l2_subdevice.h |  11 ++-
> >  src/libcamera/v4l2_device.cpp          |  92 ++++++++++++++-------
> >  src/libcamera/v4l2_subdevice.cpp       | 107 ++++++++++++++++++++++---
> >  test/v4l2_subdevice/list_formats.cpp   |  81 +++++++++++++++++++
> >  test/v4l2_subdevice/meson.build        |   1 +
> >  8 files changed, 321 insertions(+), 43 deletions(-)
> >  create mode 100644 test/v4l2_subdevice/list_formats.cpp
>
> --
> Regards,
>
> Laurent Pinchart
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.libcamera.org/pipermail/libcamera-devel/attachments/20190302/cd343f99/attachment.sig>