[PATCH v7 0/4] libcamera: rkisp1: Plumb the ConverterDW100 converter
Umang Jain
umang.jain at ideasonboard.com
Fri Sep 6 08:34:40 CEST 2024
This series intends to support i.MX8MP DW100 dewarper in rkisp1
pipeline handler. It is modelled as a V4L2M2MConverter and has a
helper to support scaler crop control. Extensions to the base converter
interface has also been done initially in the series.
Patch 1/4 add ability to add features to the Converter interface.
Each Converter can dictate its own features it can support
Patch 2/4 adds crop feature enumeration and virtual functions and
gets implemented in the V4L2 M2M converter.
Patch 3/4 prepares the rkisp1 pipeline handler to register controls
for the camera, outside of the IPA. The dw100 converter will let us
support scaler crop - which will be registered as one of the camera
controls.
Patch 4/4 plumbs the ConverterDW100 in the rkisp1 pipeline handler
Series is based on top of
[PATCH v3 0/2] libcamera: Simplify error handling with ScopeExitActions class
Testing:
--------
The series is tested on i.MX8MP and cam utility, by setting scaler crop
control via capture script with IMX283 attached to isp0.
Changes in v7:
- Documentation improvements for input crop (1/4)
- Crop feature now discovered dynamically at construction (2/4)
- Handling of scaler crop control is deferred. Now scaler
crop is applied at dewarper after buffer from ISP is ready (4/4)
- Add explanatory comments about dw100/i.MX8MP platform (4/4)
- Drop dw100 Converter sub-classing from v6
- lower indentation (4/4)
- report scaler crop value for all frames now. (4/4)
- Use ScopeExitActions on dewarp cleanup paths (4/4)
- copy updateControls() documentation from IPU3 (3/4)
Changes in v6:
- clarify crop expectations - set on input side
- Rename setCrop() to setInputCrop
- Rename getFeatures() to features()
- Rename getCropBounds() to inputCropBounds()
- rename Features::Crop to Features::InputCrop
- Document crop functionality properly
- Improve error paths
- exportsBuffers() between RkISP1Path<>dewarper instead of
allocateBuffers.
Changes in v5:
- Return -ENOTSUP for base implementation for virtual setCrop() in 2/5
- Return configured size for base implemention for virtual getCropBounds
in 2/5.
- Fix a bug in getting crop bounds when streams are not configured
- drop getSelection() V4L2VideoDevices additions from v4. we can get
bounds by setSelection by setting 1x1 and UINT_MAX x UINT_MAX
rectangles.
changes in v4:
- rework Converter Interface to expose Features
- Add Crop Feature interface in Converter setCrop and setCropBound
- Implement cropping in V4L2M2MConverter for V4L2 M2M based converters
- Introduce useDewarper_ to denote when to use dewarper (simplied code
reading).
- Trivial typo fixing.
Changes in v3:
- Base on top of [2]
- Model dewarper class as converter, so can be used with Simple
pipeline-handler/ISI
Changes in v2:
- Fix raw capture configuration, disable dewarper in that case
Umang Jain (4):
libcamera: converter: Add interface for feature flags
libcamera: converter: Add interface to support cropping capability
libcamera: rkisp1: Prepare for additional camera controls
libcamera: rkisp1: Plumb the dw100 dewarper as V4L2M2M converter
include/libcamera/internal/converter.h | 19 +-
.../internal/converter/converter_v4l2_m2m.h | 13 +-
src/libcamera/converter.cpp | 65 ++++-
.../converter/converter_v4l2_m2m.cpp | 110 ++++++++-
src/libcamera/pipeline/rkisp1/rkisp1.cpp | 225 +++++++++++++++++-
5 files changed, 416 insertions(+), 16 deletions(-)
--
2.45.0
More information about the libcamera-devel
mailing list