[libcamera-devel] [PATCH v3 0/3] Correct ControlInfoMap from the IPA

[Naushir Patuck]

Hi,

In v3 of this series, I've made the following changes:
- Added the Bug tag in patch 1/3.
- Fix typo in commit message in patch 2/3.
- Patch 2/3 also keeps some default values for ExposureTime, AnalogeGain and
FrameDuration controls during the ipa::init() as requested.
- Use the frame duration to calculate the correct exposure time limit in patch 2/3.

I've left the code using the ControlInfoMap::Map mechanism to add values to a
ControlInfoMap container.  At a later date, I may attempt to change this by
adding a ControlInfoMap::add() member function. But if this API is to be replaced
by something else, it may not be worth it...?

Regards,
Naush

Naushir Patuck (3):
  pipeline: ipa: raspberrypi: Move ControlInfoMap to the IPA
  pipeline: ipa: raspberrypi: Correctly report available control limits
  pipeline: raspberrypi: Advertise ScalerCrop from the pipeline handler

 include/libcamera/ipa/raspberrypi.h           | 55 ----------------
 include/libcamera/ipa/raspberrypi.mojom       |  8 ++-
 src/ipa/raspberrypi/raspberrypi.cpp           | 66 +++++++++++++++++--
 .../pipeline/raspberrypi/raspberrypi.cpp      | 41 +++++++-----
 .../pipeline/raspberrypi/rpi_stream.h         |  1 -
 5 files changed, 92 insertions(+), 79 deletions(-)
 delete mode 100644 include/libcamera/ipa/raspberrypi.h

-- 
2.25.1