[PATCH v2 3/6] libcamera: v4l2_subdevice: Add new metadata formats

Kieran Bingham kieran.bingham at ideasonboard.com
Tue Jan 7 14:33:51 CET 2025


Quoting Jacopo Mondi (2024-11-08 10:51:13)
> From: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> 
> Support the newly introduced V4L2 media bus formats for metadata. This
> includes generic metadata formats, and two sensor-specific embedded data
> formats.

Two sensor specific formats ? I only see MEDIA_BUS_FMT_OV2740_EMBEDDED,
or does this 'include' MEDIA_BUS_FMT_CCS_EMBEDDED?

Anyway, It doesn't worry me.


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

> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> Reviewed-by: Jacopo Mondi <jacopo.mondi at ideasonboard.com>
> Signed-off-by: Jacopo Mondi <jacopo.mondi at ideasonboard.com>
> ---
>  src/libcamera/v4l2_subdevice.cpp | 63 ++++++++++++++++++++++++++++++++
>  1 file changed, 63 insertions(+)
> 
> diff --git a/src/libcamera/v4l2_subdevice.cpp b/src/libcamera/v4l2_subdevice.cpp
> index 9f2ec4798407..f65dd6593c99 100644
> --- a/src/libcamera/v4l2_subdevice.cpp
> +++ b/src/libcamera/v4l2_subdevice.cpp
> @@ -700,6 +700,69 @@ const std::map<uint32_t, MediaBusFormatInfo> mediaBusFormatInfo{
>                 .bitsPerPixel = 0,
>                 .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
>         } },
> +       { MEDIA_BUS_FMT_META_8, {
> +               .name = "META_8",
> +               .code = MEDIA_BUS_FMT_META_8,
> +               .type = MediaBusFormatInfo::Type::Metadata,
> +               .bitsPerPixel = 8,
> +               .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
> +       } },
> +       { MEDIA_BUS_FMT_META_10, {
> +               .name = "META_10",
> +               .code = MEDIA_BUS_FMT_META_10,
> +               .type = MediaBusFormatInfo::Type::Metadata,
> +               .bitsPerPixel = 10,
> +               .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
> +       } },
> +       { MEDIA_BUS_FMT_META_12, {
> +               .name = "META_12",
> +               .code = MEDIA_BUS_FMT_META_12,
> +               .type = MediaBusFormatInfo::Type::Metadata,
> +               .bitsPerPixel = 12,
> +               .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
> +       } },
> +       { MEDIA_BUS_FMT_META_14, {
> +               .name = "META_14",
> +               .code = MEDIA_BUS_FMT_META_14,
> +               .type = MediaBusFormatInfo::Type::Metadata,
> +               .bitsPerPixel = 14,
> +               .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
> +       } },
> +       { MEDIA_BUS_FMT_META_16, {
> +               .name = "META_16",
> +               .code = MEDIA_BUS_FMT_META_16,
> +               .type = MediaBusFormatInfo::Type::Metadata,
> +               .bitsPerPixel = 16,
> +               .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
> +       } },
> +       { MEDIA_BUS_FMT_META_20, {
> +               .name = "META_20",
> +               .code = MEDIA_BUS_FMT_META_20,
> +               .type = MediaBusFormatInfo::Type::Metadata,
> +               .bitsPerPixel = 20,
> +               .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
> +       } },
> +       { MEDIA_BUS_FMT_META_24, {
> +               .name = "META_24",
> +               .code = MEDIA_BUS_FMT_META_24,
> +               .type = MediaBusFormatInfo::Type::Metadata,
> +               .bitsPerPixel = 24,
> +               .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
> +       } },
> +       { MEDIA_BUS_FMT_CCS_EMBEDDED, {
> +               .name = "CCS_EMBEDDED",
> +               .code = MEDIA_BUS_FMT_CCS_EMBEDDED,
> +               .type = MediaBusFormatInfo::Type::EmbeddedData,
> +               .bitsPerPixel = 0,
> +               .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
> +       } },
> +       { MEDIA_BUS_FMT_OV2740_EMBEDDED, {
> +               .name = "OV2740_EMBEDDED",
> +               .code = MEDIA_BUS_FMT_CCS_EMBEDDED,
> +               .type = MediaBusFormatInfo::Type::EmbeddedData,
> +               .bitsPerPixel = 0,
> +               .colourEncoding = PixelFormatInfo::ColourEncodingRAW,
> +       } },
>  };
>  
>  } /* namespace */
> -- 
> 2.47.0
>


More information about the libcamera-devel mailing list