<div dir="ltr">Hi all,<div><br></div><div>Any chance I could get somebody to have a review on this series please?</div><div><br></div><div>Would it possibly help move this further if I were to revert my changes to DelayedControls,</div><div>and add a Raspberry Pi specific implementation?</div><div><br></div><div>Regards,</div><div>Naush</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 19 Oct 2022 at 10:01, 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>
Version 4 of this series only has one change from the previous versions:<br>
<br>
In patch 6/7, the IPA cookie (index into the RPiMetadata array) is passed from<br>
the pipeline handler into the IPA rather than the other way around.  This helps<br>
simplify some of our future work on per-frame-control.<br>
<br>
There is still lots of discussion on this series to be had, particularly for<br>
patch 1/7 where we add a cookie to DelayedControls, so feel free to provide some<br>
feedback.<br>
<br>
Regards,<br>
Naush<br>
<br>
Naushir Patuck (7):<br>
  delayed_controls: Template the ControlRingBuffer class<br>
  delayed_controls: Add user cookie to DelayedControls<br>
  tests: delayed_controls: Add cookie value test<br>
  ipa: raspberrypi: Add RPiController::Metadata::mergeCopy<br>
  ipa: raspberrypi: Use an array of RPiController::Metadata objects<br>
  pipeline: ipa: raspberrypi: Use IPA cookies<br>
  ipa: raspberrypi: agc: Fix digital gain calculation for manual mode<br>
<br>
 include/libcamera/internal/delayed_controls.h | 21 +++--<br>
 include/libcamera/ipa/raspberrypi.mojom       |  4 +-<br>
 src/ipa/raspberrypi/controller/metadata.h     | 10 +++<br>
 src/ipa/raspberrypi/controller/rpi/agc.cpp    | 10 ++-<br>
 src/ipa/raspberrypi/raspberrypi.cpp           | 82 ++++++++++++-------<br>
 src/libcamera/delayed_controls.cpp            | 20 +++--<br>
 src/libcamera/pipeline/ipu3/ipu3.cpp          |  3 +-<br>
 .../pipeline/raspberrypi/raspberrypi.cpp      | 19 +++--<br>
 src/libcamera/pipeline/rkisp1/rkisp1.cpp      |  3 +-<br>
 test/delayed_controls.cpp                     | 52 +++++++++++-<br>
 10 files changed, 163 insertions(+), 61 deletions(-)<br>
<br>
-- <br>
2.25.1<br>
<br>
</blockquote></div>