<div dir="ltr"><div dir="ltr">Hi Laurent,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 4 Feb 2021 at 21:49, Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.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 Naush,<br>
<br>
On Fri, Jan 29, 2021 at 03:11:48PM +0000, Naushir Patuck wrote:<br>
> 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>
I've reviewed the whole series and only have minor comments. I've<br>
offered to fix the small issues when applying (if you agree with the<br>
comments of course) in the first few patches, but as there are a few<br>
additional questions later, it may be easier if you could send a v6.<br>
Other patches to the RPi IPA have also been merged in the meantime,<br>
generating a few minor conflicts, and I'd prefer if you could resolve<br>
them and test the result.<br></blockquote><div><br></div><div>Thanks for all the reviews.  I have incorporated all the suggested changes, and done a rebase.  I will post a new version 6 of the series after a bit more testing.</div><div><br></div><div>Regards,</div><div>Naush</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<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>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div></div>