<div dir="ltr">Another gentle ping for the patches below.<br><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 8 Oct 2024 at 12:30, 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 all,<br>
<br>
Patches 1/7 and 7/7 in this series need one more R-B tag then would be<br>
ready to merge.  Would somebody be able to have a look at them please?<br>
<br>
Thanks!<br>
Naush<br>
<br>
<br>
On Thu, 3 Oct 2024 at 08:53, Naushir Patuck <<a href="mailto:naush@raspberrypi.com" target="_blank">naush@raspberrypi.com</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> Version 3 of this work is functionally identical to v2, but with the following<br>
> minor changes:<br>
><br>
> - Update the control documentation to reflect the priority of rpi::ScalerCrops<br>
>   over the ScalerCrop control.<br>
> - Remove the default constructor of CropParams, requiring replacing cropParams_[n]<br>
>   accessor with cropsParams_.at(n) in the pipeline handler code.<br>
><br>
> Thanks,<br>
> Naush<br>
><br>
> Naushir Patuck (7):<br>
>   controls: rpi: Add a vendor rpi::ScalerCrops control<br>
>   pipeline: rpi: Remove CameraData::scalerCrop_<br>
>   pipeline: rpi: Pass crop rectangle as a parameter to<br>
>     platformSetIspCrop()<br>
>   pipeline: rpi: Introduce CameraData::CropParams<br>
>   pipeline: rpi: Track which ISP output is configured for a stream<br>
>   pipeline: rpi: Pass ISP output index into platformSetIspCrop()<br>
>   pipeline: rpi: Handler controls::rpi::ScalerCrops<br>
><br>
>  src/ipa/rpi/common/ipa_base.cpp               | 14 +++<br>
>  src/libcamera/control_ids_rpi.yaml            | 25 +++++<br>
>  .../pipeline/rpi/common/pipeline_base.cpp     | 94 +++++++++++++------<br>
>  .../pipeline/rpi/common/pipeline_base.h       | 22 ++++-<br>
>  src/libcamera/pipeline/rpi/vc4/vc4.cpp        | 17 +++-<br>
>  5 files changed, 136 insertions(+), 36 deletions(-)<br>
><br>
> --<br>
> 2.34.1<br>
><br>
</blockquote></div>