<div dir="ltr">Hi all,<div><br></div><div>Gentle nudge for some feedback on this 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 Fri, 29 Jan 2021 at 15:11, 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 5 of this patch series addresses all the review comments in v4:<br>
- Switch DenoiseMode to an enum class.<br>
- Avoid double lookup in the DenoiseMode conversion table.<br>
- Add controls::draft::NoiseReductionMode to the available controls handled by the IPA.<br>
<br>
Regards,<br>
Naush<br>
<br>
<br>
Naushir Patuck (6):<br>
  pipeline: raspberrypi: Refactor stream configuration routine<br>
  pipeline: raspberrypi: Set the ISP Output1 to 1/4 resolution if unused<br>
  uapi: raspberrypi: Update the bcm2835-isp header definition<br>
  ipa: raspberrypi: Rename SdnStatus to DenoiseStatus<br>
  ipa: raspberrypi: Add a DenoiseAlgorithm class to the Controller<br>
  ipa: raspberrypi: Handle control::NoiseReductionMode in the controller<br>
<br>
 include/libcamera/ipa/raspberrypi.h           |  1 +<br>
 include/linux/bcm2835-isp.h                   | 32 ++++++-<br>
 .../controller/denoise_algorithm.hpp          | 23 +++++<br>
 .../{sdn_status.h => denoise_status.h}        |  7 +-<br>
 src/ipa/raspberrypi/controller/rpi/sdn.cpp    | 17 ++--<br>
 src/ipa/raspberrypi/controller/rpi/sdn.hpp    |  5 +-<br>
 src/ipa/raspberrypi/raspberrypi.cpp           | 59 +++++++++++--<br>
 .../pipeline/raspberrypi/raspberrypi.cpp      | 86 ++++++++++---------<br>
 8 files changed, 174 insertions(+), 56 deletions(-)<br>
 create mode 100644 src/ipa/raspberrypi/controller/denoise_algorithm.hpp<br>
 rename src/ipa/raspberrypi/controller/{sdn_status.h => denoise_status.h} (62%)<br>
<br>
-- <br>
2.25.1<br>
<br>
</blockquote></div>