[PATCH v6 5/5] pipeline: simple: Reset delayedCtrls at start.
Kieran Bingham
kieran.bingham at ideasonboard.com
Mon Mar 31 18:53:40 CEST 2025
Quoting Stanislaw Gruszka (2025-03-05 13:52:56)
> Similar like in other pipelines (IPU3, rpi) avoid using stale
> values of DelayedControls class when the same camera is started
> second time.
>
> Co-developed-by: Hans de Goede <hdegoede at redhat.com>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> Signed-off-by: Stanislaw Gruszka <stanislaw.gruszka at linux.intel.com>
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> ---
> src/libcamera/pipeline/simple/simple.cpp | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/libcamera/pipeline/simple/simple.cpp b/src/libcamera/pipeline/simple/simple.cpp
> index 58aa3dba..659b1123 100644
> --- a/src/libcamera/pipeline/simple/simple.cpp
> +++ b/src/libcamera/pipeline/simple/simple.cpp
> @@ -1376,6 +1376,7 @@ int SimplePipelineHandler::start(Camera *camera, [[maybe_unused]] const ControlL
>
> video->bufferReady.connect(data, &SimpleCameraData::imageBufferReady);
>
> + data->delayedCtrls_->reset();
> if (frameStartEmitter) {
> ret = frameStartEmitter->setFrameStartEnabled(true);
> if (ret) {
> --
> 2.43.0
>
More information about the libcamera-devel
mailing list