[libcamera-devel] [PATCH 04/13] libcamera: pipeline: Add initialization hook for CameraData

[Niklas Söderlund]

Add a hook so CameraData can be initialized before a camera is exposed
to an application but after all resources and possibly an IPA have been
found.

Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
---
 src/libcamera/include/pipeline_handler.h |  2 ++
 src/libcamera/pipeline_handler.cpp       | 12 ++++++++++++
 2 files changed, 14 insertions(+)

diff --git a/src/libcamera/include/pipeline_handler.h b/src/libcamera/include/pipeline_handler.h
index 91d40ef40a465c4e..45f9457879c64363 100644
--- a/src/libcamera/include/pipeline_handler.h
+++ b/src/libcamera/include/pipeline_handler.h
@@ -43,6 +43,8 @@ public:
 	}
 	virtual ~CameraData() {}
 
+	virtual int initCameraData() { return 0; };
+
 	Camera *camera_;
 	PipelineHandler *pipe_;
 	std::list<Request *> queuedRequests_;
diff --git a/src/libcamera/pipeline_handler.cpp b/src/libcamera/pipeline_handler.cpp
index 766fd496306ece9c..6b3ee0ed8f39676a 100644
--- a/src/libcamera/pipeline_handler.cpp
+++ b/src/libcamera/pipeline_handler.cpp
@@ -66,6 +66,12 @@ LOG_DEFINE_CATEGORY(Pipeline)
  * is needed for the camera both parameters should be set to 0.
  */
 
+/**
+ * \fn CameraData::initCameraData()
+ * \brief Hook to initialize the camera data
+ * \return 0 on success or a negative error code otherwise
+ */
+
 /**
  * \var CameraData::camera_
  * \brief The camera related to this CameraData instance
@@ -462,6 +468,12 @@ void PipelineHandler::registerCamera(std::shared_ptr<Camera> camera,
 	}
 
 	data->camera_ = camera.get();
+	if (data->initCameraData()) {
+		LOG(Pipeline, Warning) << "Skipping " << camera->name()
+				       << " initialization camera data failed";
+		return;
+	}
+
 	cameraData_[camera.get()] = std::move(data);
 	cameras_.push_back(camera);
 	manager_->addCamera(std::move(camera));
-- 
2.22.1