<div dir="ltr">Hi,<div><br></div><div>Any chance of another review for this one please?</div><div><br></div><div>Thanks,</div><div>Naush</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 26 Sept 2022 at 10:57, 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 3 of this series addresses the comments from Umang and David - mostly<br>
minor typos, and a simplification to the new Metadata:mergeCopy() function.<br>
<br>
Thanks,<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       |  3 +-<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      | 12 +--<br>
 src/libcamera/pipeline/rkisp1/rkisp1.cpp      |  3 +-<br>
 test/delayed_controls.cpp                     | 52 +++++++++++-<br>
 10 files changed, 156 insertions(+), 60 deletions(-)<br>
<br>
-- <br>
2.25.1<br>
<br>
</blockquote></div>