<div dir="ltr">Hi all,<div><br></div><div>Any other takers for a review on this series?  The patches need one more R-B tag.</div><div><br></div><div>Regards,</div><div>Naush</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 22 Jun 2022 at 11:20, Naushir Patuck <<a href="mailto:naush@raspberrypi.com">naush@raspberrypi.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
In v3 of this series, I've made the following changes:<br>
- Added the Bug tag in patch 1/3.<br>
- Fix typo in commit message in patch 2/3.<br>
- Patch 2/3 also keeps some default values for ExposureTime, AnalogeGain and<br>
FrameDuration controls during the ipa::init() as requested.<br>
- Use the frame duration to calculate the correct exposure time limit in patch 2/3.<br>
<br>
I've left the code using the ControlInfoMap::Map mechanism to add values to a<br>
ControlInfoMap container.  At a later date, I may attempt to change this by<br>
adding a ControlInfoMap::add() member function. But if this API is to be replaced<br>
by something else, it may not be worth it...?<br>
<br>
Regards,<br>
Naush<br>
<br>
Naushir Patuck (3):<br>
  pipeline: ipa: raspberrypi: Move ControlInfoMap to the IPA<br>
  pipeline: ipa: raspberrypi: Correctly report available control limits<br>
  pipeline: raspberrypi: Advertise ScalerCrop from the pipeline handler<br>
<br>
 include/libcamera/ipa/raspberrypi.h           | 55 ----------------<br>
 include/libcamera/ipa/raspberrypi.mojom       |  8 ++-<br>
 src/ipa/raspberrypi/raspberrypi.cpp           | 66 +++++++++++++++++--<br>
 .../pipeline/raspberrypi/raspberrypi.cpp      | 41 +++++++-----<br>
 .../pipeline/raspberrypi/rpi_stream.h         |  1 -<br>
 5 files changed, 92 insertions(+), 79 deletions(-)<br>
 delete mode 100644 include/libcamera/ipa/raspberrypi.h<br>
<br>
-- <br>
2.25.1<br>
<br>
</blockquote></div>