diff options
| author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2024-03-01 18:45:29 +0200 |
|---|---|---|
| committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2024-06-25 14:38:49 +0300 |
| commit | 369e5ed8fd2ee68ca856c593524aa47d0da66812 (patch) | |
| tree | 4691071b3673def6e6b17a6c355d0be1a0bf218f | |
| parent | 9363158f5830790a523772b374755dd5e92b2bcd (diff) | |
pipeline: raspberrypi: vc4: Fix configuration of the embedded data node
The Unicam embedded data video device suffers from two issues with the
upstream driver:
- The driver uses the generic metadata V4L2_META_FMT_GENERIC_* formats,
instead of the downstream V4L2_META_FMT_SENSOR_DATA.
- The driver requires the width and height of the embedded data stream
to be configured.
Fix them both.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
| -rw-r--r-- | src/libcamera/pipeline/rpi/vc4/vc4.cpp | 19 |
1 files changed, 17 insertions, 2 deletions
diff --git a/src/libcamera/pipeline/rpi/vc4/vc4.cpp b/src/libcamera/pipeline/rpi/vc4/vc4.cpp index e5f6accc..e8445f53 100644 --- a/src/libcamera/pipeline/rpi/vc4/vc4.cpp +++ b/src/libcamera/pipeline/rpi/vc4/vc4.cpp @@ -623,10 +623,25 @@ int Vc4CameraData::platformConfigure(const RPi::RPiCameraConfiguration *rpiConfi * supports it. */ if (sensorMetadata_) { + static const std::map<uint32_t, V4L2PixelFormat> metaFormats{ + { MEDIA_BUS_FMT_META_8, V4L2PixelFormat(V4L2_META_FMT_GENERIC_8) }, + { MEDIA_BUS_FMT_META_10, V4L2PixelFormat(V4L2_META_FMT_GENERIC_CSI2_10) }, + { MEDIA_BUS_FMT_META_12, V4L2PixelFormat(V4L2_META_FMT_GENERIC_CSI2_12) }, + { MEDIA_BUS_FMT_META_14, V4L2PixelFormat(V4L2_META_FMT_GENERIC_CSI2_14) }, + }; + V4L2SubdeviceFormat embeddedFormat = sensor_->embeddedDataFormat(); + const auto metaFormat = metaFormats.find(embeddedFormat.code); + if (metaFormat == metaFormats.end()) { + LOG(RPI, Error) + << "Unsupported metadata format " + << utils::hex(embeddedFormat.code, 4); + return -EINVAL; + } + V4L2DeviceFormat format{}; - format.fourcc = V4L2PixelFormat(V4L2_META_FMT_SENSOR_DATA); - format.planes[0].size = embeddedFormat.size.width * embeddedFormat.size.height; + format.fourcc = metaFormat->second; + format.size = embeddedFormat.size; LOG(RPI, Debug) << "Setting embedded data format " << format; ret = unicam_[Unicam::Embedded].dev()->setFormat(&format); |