<div dir="ltr"><div dir="ltr">Hi,<div><br></div><div>Sorry to be nagging on this, but could I get another review for this series please?</div><div>This fix needs to be in our up-coming libcamera release.</div><div><br></div><div>The changes to the forked version of DelayedControls are identical to what was</div><div>in version 5.</div><div><br></div><div>Many thanks,</div><div>Naush</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 21 Nov 2022 at 13:56, 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"><div dir="ltr">Hi,<br><div><br></div><div>Would I be able to get another review on this series please?</div><div><br></div><div>Thanks,</div><div>Naush</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 15 Nov 2022 at 09:07, Naushir Patuck <<a href="mailto:naush@raspberrypi.com" target="_blank">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 6 of this series forks DelayedControls into the Raspberry Pi pipeline<br>
handler namespace, patches 1 and 2 cover this.  Patches 3 and 4 update the<br>
forked DelayedControls class to add the user cookie with the same set of changes <br>
as were in DelayedControls in version 5 of this series.  I've removed all the<br>
existing review tags from these two patches to give folks another chance to<br>
validate the changes.<br>
<br>
Thanks,<br>
Naush<br>
<br>
Naushir Patuck (8):<br>
  pipeline: raspberrypi: Fork DelayedControls<br>
  pipeline: raspberrypi: Switch to RPi::DelayedControls<br>
  pipeline: raspberrypi: delayed_controls: Template the<br>
    ControlRingBuffer class<br>
  pipeline: raspberrypi: delayed_controls: Add user cookie to<br>
    DelayedControls<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/ipa/raspberrypi.mojom       |   6 +-<br>
 src/ipa/raspberrypi/controller/metadata.h     |  10 +<br>
 src/ipa/raspberrypi/controller/rpi/agc.cpp    |  10 +-<br>
 src/ipa/raspberrypi/raspberrypi.cpp           | 104 ++++---<br>
 .../pipeline/raspberrypi/delayed_controls.cpp | 293 ++++++++++++++++++<br>
 .../pipeline/raspberrypi/delayed_controls.h   |  87 ++++++<br>
 .../pipeline/raspberrypi/meson.build          |   1 +<br>
 .../pipeline/raspberrypi/raspberrypi.cpp      |  26 +-<br>
 8 files changed, 481 insertions(+), 56 deletions(-)<br>
 create mode 100644 src/libcamera/pipeline/raspberrypi/delayed_controls.cpp<br>
 create mode 100644 src/libcamera/pipeline/raspberrypi/delayed_controls.h<br>
<br>
-- <br>
2.25.1<br>
<br>
</blockquote></div>
</blockquote></div></div>