[libcamera-devel] [PATCH v2 6/8] libcamera: pipeline: ipu3: Stop IPA before stopping devices
Jacopo Mondi
jacopo at jmondi.org
Sat Mar 13 10:37:27 CET 2021
Hi Kieran,
On Fri, Mar 12, 2021 at 05:47:25AM +0000, Kieran Bingham wrote:
> The IPA should be stopped before the hardware devices to ensure that
> all asynchronous actions have completed within the IPA before resources
> are removed and released.
>
> Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>
Thanks
j
> ---
> src/libcamera/pipeline/ipu3/ipu3.cpp | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp
> index 00da2bb26e41..7ab3a5bfdccb 100644
> --- a/src/libcamera/pipeline/ipu3/ipu3.cpp
> +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp
> @@ -764,13 +764,13 @@ void PipelineHandlerIPU3::stop(Camera *camera)
> IPU3CameraData *data = cameraData(camera);
> int ret = 0;
>
> + data->ipa_->stop();
> +
> ret |= data->imgu_->stop();
> ret |= data->cio2_.stop();
> if (ret)
> LOG(IPU3, Warning) << "Failed to stop camera " << camera->id();
>
> - data->ipa_->stop();
> -
> freeBuffers(camera);
> }
>
> --
> 2.25.1
>
> _______________________________________________
> 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