<div dir="ltr">Hi,<br><div><br></div><div>All patches now have their review tags in this series.  Unless there are no other objections, it would be great if this could be merged.</div><div><br></div><div>Regards,</div><div>Naush</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 8 Nov 2022 at 09:39, 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"><div dir="ltr">Hi all,<div><br></div><div>Any chance we could progress this one please.  Patches 3/4/6 need a second</div><div>review.</div><div><br></div><div>Many thanks,</div><div>Naush</div><div><br></div><div>PS: <a href="http://patchwork.libcamera.org" target="_blank">patchwork.libcamera.org</a> seems to be a bit unhappy for a few days now.</div><div>Might need a reboot!</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 31 Oct 2022 at 11:45, 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>
In version 7:<br>
<br>
- For patch 2/7, the cookie must be provided in push() and reset(). Updated the<br>
rkisp1 and ipu3 pipeline handlers to provide frame numbers for the cookie value.<br>
- For patch 3/4, add a test for skipped/dropped frames and cookie handling.<br>
- Updated patch 6/7 to use the request sequence number for the context index<br>
instead of using a separte sequence counter.<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 tests<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       |   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>
 src/libcamera/delayed_controls.cpp            |  22 ++--<br>
 src/libcamera/pipeline/ipu3/ipu3.cpp          |   9 +-<br>
 .../pipeline/raspberrypi/raspberrypi.cpp      |  18 ++-<br>
 src/libcamera/pipeline/rkisp1/rkisp1.cpp      |   7 +-<br>
 test/delayed_controls.cpp                     | 121 ++++++++++++++++--<br>
 10 files changed, 240 insertions(+), 88 deletions(-)<br>
<br>
-- <br>
2.25.1<br>
<br>
</blockquote></div></div>
</blockquote></div>