[libcamera-devel,v3,0/2] libcamera: Introduce V4L2Device base class

1417 mbox series

Just a few fixes spotted by Kieran during review:
- a trivial type (s/poiunter/pointer)
- remove the "ret = -errno" assignement in the ioctl error path in
  V4L2VideoDev and V4L2Subdevice classes, as V4L2Device::ioctl() already
  return -errno in case of failure.

Thanks
   j

Jacopo Mondi (2):
  libcamera: Rename V4L2Device to V4L2VideoDevice
  libcamera: Introduce V4L2Device base class

 include/libcamera/buffer.h                    |   2 +-
 src/libcamera/include/v4l2_device.h           | 166 +--
 src/libcamera/include/v4l2_subdevice.h        |   6 +-
 src/libcamera/include/v4l2_videodevice.h      | 183 ++++
 src/libcamera/meson.build                     |   2 +
 src/libcamera/pipeline/ipu3/ipu3.cpp          |  24 +-
 src/libcamera/pipeline/rkisp1/rkisp1.cpp      |   6 +-
 src/libcamera/pipeline/uvcvideo.cpp           |   6 +-
 src/libcamera/pipeline/vimc.cpp               |   6 +-
 src/libcamera/v4l2_device.cpp                 | 980 ++----------------
 src/libcamera/v4l2_subdevice.cpp              |  71 +-
 src/libcamera/v4l2_videodevice.cpp            | 977 +++++++++++++++++
 test/meson.build                              |   2 +-
 .../buffer_sharing.cpp                        |  18 +-
 .../capture_async.cpp                         |   6 +-
 .../double_open.cpp                           |   8 +-
 .../formats.cpp                               |   8 +-
 .../meson.build                               |   8 +-
 .../request_buffers.cpp                       |   6 +-
 .../stream_on_off.cpp                         |   6 +-
 .../v4l2_videodevice_test.cpp}                |   8 +-
 .../v4l2_videodevice_test.h}                  |   8 +-
 22 files changed, 1307 insertions(+), 1200 deletions(-)
 create mode 100644 src/libcamera/include/v4l2_videodevice.h
 create mode 100644 src/libcamera/v4l2_videodevice.cpp
 rename test/{v4l2_device => v4l2_videodevice}/buffer_sharing.cpp (90%)
 rename test/{v4l2_device => v4l2_videodevice}/capture_async.cpp (91%)
 rename test/{v4l2_device => v4l2_videodevice}/double_open.cpp (75%)
 rename test/{v4l2_device => v4l2_videodevice}/formats.cpp (85%)
 rename test/{v4l2_device => v4l2_videodevice}/meson.build (74%)
 rename test/{v4l2_device => v4l2_videodevice}/request_buffers.cpp (77%)
 rename test/{v4l2_device => v4l2_videodevice}/stream_on_off.cpp (78%)
 rename test/{v4l2_device/v4l2_device_test.cpp => v4l2_videodevice/v4l2_videodevice_test.cpp} (90%)
 rename test/{v4l2_device/v4l2_device_test.h => v4l2_videodevice/v4l2_videodevice_test.h} (81%)

--
2.21.0