[PATCH] libcamera: camera: Log info message to report camera creation

[Kieran Bingham]

Quoting Laurent Pinchart (2025-02-26 00:55:47)
> Camera creation is one of the most important events generated by
> libcamera, but we are completely silent about it. The lack of a log
> message makes it more difficult to identify problems and provide
> support. Fix it by adding an Info message that reports the camera id and
> its pipeline handler when the camera is created.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  src/libcamera/camera.cpp | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/libcamera/camera.cpp b/src/libcamera/camera.cpp
> index 56c5851993c9..35097bfdafa2 100644
> --- a/src/libcamera/camera.cpp
> +++ b/src/libcamera/camera.cpp
> @@ -862,6 +862,10 @@ std::shared_ptr<Camera> Camera::create(std::unique_ptr<Private> d,
>                 }
>         };
> 
> +       LOG(Camera, Info)
> +               << "Creating camera '" << id << "' for pipeline handler "
> +               << d->pipe_->name();
> +

I'm fine with this indeed.

Later on I might send the patch I have that puts the pipeline handler
into the CameraID too ;-)


Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

>         Camera *camera = new Camera(std::move(d), id, streams);
> 
>         return std::shared_ptr<Camera>(camera, Deleter());
> 
> base-commit: 33ce463a46c44f874fdbc3e484bee730e7b251a3
> --
> Regards,
> 
> Laurent Pinchart
>