summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2024-02-29 02:32:34 +0200
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2024-04-24 20:44:25 +0300
commit74e86ad15ebad1396854d475159e75894c02286d (patch)
tree4b3cd6bd1b5bf569375ae5fa25bc89ea0aa2602b
parent507dada7eb08e669deed345ba877291e9a43f364 (diff)
pipeline: raspberrypi: common: Configure sensor embedded data
Now that the CameraSensor class implements support for embedded data, use the corresponding API to enable or disable the embedded data stream. This changes requires sensor kernel drivers that implement the upstream embedded data API based on V4L2 streams. As the API is under development and not merged in the upstream kernel yet, this breaks compatibility with the downstream Raspberry Pi kernel. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-rw-r--r--src/libcamera/pipeline/rpi/common/pipeline_base.cpp12
1 files changed, 12 insertions, 0 deletions
diff --git a/src/libcamera/pipeline/rpi/common/pipeline_base.cpp b/src/libcamera/pipeline/rpi/common/pipeline_base.cpp
index d662c8f1..9a2b8b90 100644
--- a/src/libcamera/pipeline/rpi/common/pipeline_base.cpp
+++ b/src/libcamera/pipeline/rpi/common/pipeline_base.cpp
@@ -528,6 +528,18 @@ int PipelineHandlerBase::configure(Camera *camera, CameraConfiguration *config)
return ret;
/*
+ * Configure embedded data on the sensor. Only check for errors when
+ * enabling embedded data, as some sensors don't support disabling it,
+ * and Unicam will simply drop the embedded data packets if we don't
+ * capture them.
+ */
+ ret = data->sensor_->setEmbeddedDataEnabled(data->sensorMetadata_);
+ if (ret && data->sensorMetadata_) {
+ LOG(RPI, Error) << "Unable to enable embedded data: " << ret;
+ return ret;
+ }
+
+ /*
* Platform specific internal stream configuration. This also assigns
* external streams which get configured below.
*/