[libcamera-devel] [PATCH v3 08/22] libcamera: pipeline: rkisp1: Set the crop rectangle
Jacopo Mondi
jacopo at jmondi.org
Fri Sep 25 16:28:14 CEST 2020
Hi Niklas,
On Fri, Sep 25, 2020 at 03:41:53AM +0200, Niklas Söderlund wrote:
> Changing resolutions back and forth can provoke the crop rectangle to go
> out of sync, set it as part of format configuration.
Is there any dependency on the kernel version ?
Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>
Thanks
j
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
> * Changes since v2
> - Fix s/port/part/ in commit message.
> ---
> src/libcamera/pipeline/rkisp1/rkisp1.cpp | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/src/libcamera/pipeline/rkisp1/rkisp1.cpp b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> index 27191b18c989aa5e..2f70bf2220fa5c84 100644
> --- a/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> +++ b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> @@ -629,6 +629,11 @@ int PipelineHandlerRkISP1::configure(Camera *camera, CameraConfiguration *c)
> if (ret < 0)
> return ret;
>
> + Rectangle rect(0, 0, format.size);
> + ret = isp_->setSelection(0, V4L2_SEL_TGT_CROP, &rect);
> + if (ret < 0)
> + return ret;
> +
> LOG(RkISP1, Debug) << "ISP input pad configured with " << format.toString();
>
> /* YUYV8_2X8 is required on the ISP source path pad for YUV output. */
> --
> 2.28.0
>
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel
More information about the libcamera-devel
mailing list