[libcamera-devel] [PATCH] libcamera: v4l2_device: Map YCbCrEncoding::None to V4L2

[paul.elder at ideasonboard.com]

On Sun, Sep 04, 2022 at 02:06:43AM +0300, Laurent Pinchart via libcamera-devel wrote:
> The ycbcrEncodingToV4l2 map is missing the YCbCrEncoding::None encoding,
> which results in a failure of V4L2Device::fromColorSpace() to convert
> color spaces from libcamera to V4L2 for RGB formats. Fix it by adding
> the missing encoding. As V4L2 has no such encoding, use
> V4L2_YCBCR_ENC_DEFAULT as the value doesn't matter.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>

> ---
>  src/libcamera/v4l2_device.cpp | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/libcamera/v4l2_device.cpp b/src/libcamera/v4l2_device.cpp
> index 1d899657bf7d..7f251bd820a0 100644
> --- a/src/libcamera/v4l2_device.cpp
> +++ b/src/libcamera/v4l2_device.cpp
> @@ -796,6 +796,8 @@ static const std::map<ColorSpace::TransferFunction, v4l2_xfer_func> transferFunc
>  };
>  
>  static const std::map<ColorSpace::YcbcrEncoding, v4l2_ycbcr_encoding> ycbcrEncodingToV4l2 = {
> +	/* V4L2 has no "none" encoding. */
> +	{ ColorSpace::YcbcrEncoding::None, V4L2_YCBCR_ENC_DEFAULT },
>  	{ ColorSpace::YcbcrEncoding::Rec601, V4L2_YCBCR_ENC_601 },
>  	{ ColorSpace::YcbcrEncoding::Rec709, V4L2_YCBCR_ENC_709 },
>  	{ ColorSpace::YcbcrEncoding::Rec2020, V4L2_YCBCR_ENC_BT2020 },