[PATCH 2/2] gstreamer: Map 10/12/14/16 bayer formats supports

[Laurent Pinchart]

Hi Nicolas,

Thank you for the patch.

On Fri, Feb 02, 2024 at 09:58:19AM +0100, Nicolas Dufresne wrote:
> From: Nicolas Dufresne <nicolas.dufresne at collabora.com>
> 
> These formats are now defined in upstream GStreamer main branch, so it is now
> safe to use their names. Note that libcamera only supports little endian
> variants of these formats.
> 
> Signed-off-by: Nicolas Dufresne <nicolas.dufresne at collabora.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  src/gstreamer/gstlibcamera-utils.cpp | 48 ++++++++++++++++++++++++++++
>  1 file changed, 48 insertions(+)
> 
> diff --git a/src/gstreamer/gstlibcamera-utils.cpp b/src/gstreamer/gstlibcamera-utils.cpp
> index 2a0b77f4..469ac810 100644
> --- a/src/gstreamer/gstlibcamera-utils.cpp
> +++ b/src/gstreamer/gstlibcamera-utils.cpp
> @@ -25,6 +25,22 @@ static struct {
>  	{ GST_VIDEO_FORMAT_ENCODED, formats::SGBRG8 },
>  	{ GST_VIDEO_FORMAT_ENCODED, formats::SGRBG8 },
>  	{ GST_VIDEO_FORMAT_ENCODED, formats::SRGGB8 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SBGGR10 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SGBRG10 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SGRBG10 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SRGGB10 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SBGGR12 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SGBRG12 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SGRBG12 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SRGGB12 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SBGGR14 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SGBRG14 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SGRBG14 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SRGGB14 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SBGGR16 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SGBRG16 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SGRBG16 },
> +	{ GST_VIDEO_FORMAT_ENCODED, formats::SRGGB16 },
>  
>  	/* Monochrome */
>  	{ GST_VIDEO_FORMAT_GRAY8, formats::R8 },
> @@ -250,6 +266,38 @@ bayer_format_to_string(int format)
>  		return "grbg";
>  	case formats::SRGGB8:
>  		return "rggb";
> +	case formats::SBGGR10:
> +		return "bggr10le";
> +	case formats::SGBRG10:
> +		return "gbrg10le";
> +	case formats::SGRBG10:
> +		return "grbg10le";
> +	case formats::SRGGB10:
> +		return "rggb10le";
> +	case formats::SBGGR12:
> +		return "bggr12le";
> +	case formats::SGBRG12:
> +		return "gbrg12le";
> +	case formats::SGRBG12:
> +		return "grbg12le";
> +	case formats::SRGGB12:
> +		return "rggb12le";
> +	case formats::SBGGR14:
> +		return "bggr14le";
> +	case formats::SGBRG14:
> +		return "gbrg14le";
> +	case formats::SGRBG14:
> +		return "grbg14le";
> +	case formats::SRGGB14:
> +		return "rggb14le";
> +	case formats::SBGGR16:
> +		return "bggr16le";
> +	case formats::SGBRG16:
> +		return "gbrg16le";
> +	case formats::SGRBG16:
> +		return "grbg16le";
> +	case formats::SRGGB16:
> +		return "rggb16le";
>  	}
>  	return NULL;
>  }

-- 
Regards,

Laurent Pinchart