[libcamera-devel] [PATCH 1/2] include: drm_fourcc: Add 16-bit Bayer FourCC

[Laurent Pinchart]

Hi Niklas,

Thank you for the patch.

On Fri, Jul 24, 2020 at 10:55:49AM +0200, Niklas Söderlund wrote:
> The patch trying to upstream Bayer formats to the DRM FourCC header file
> in Linux left out the 16-bit formats, add them. This addition will be
> included in the next version of the DRM Bayer patch sent out.
> 
> Intention is to merge this in libcamera and update the header file once
> the upstream patch is picked up.
> 
> Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
> ---
>  include/linux/drm_fourcc.h | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/include/linux/drm_fourcc.h b/include/linux/drm_fourcc.h
> index 69240dfe1df0df28..baa3faecdf1575c7 100644
> --- a/include/linux/drm_fourcc.h
> +++ b/include/linux/drm_fourcc.h
> @@ -344,6 +344,12 @@ extern "C" {
>  #define DRM_FORMAT_SGBRG14	fourcc_code('G', 'B', '1', '4')
>  #define DRM_FORMAT_SBGGR14	fourcc_code('B', 'G', '1', '4')
>  
> +/* 16-bit Bayer formats */
> +#define DRM_FORMAT_SRGGB16	fourcc_code('R', 'G', '1', '6')

This conflicts with DRM_FORMAT_RGB565.

> +#define DRM_FORMAT_SGRBG16	fourcc_code('G', 'R', '1', '6')
> +#define DRM_FORMAT_SGBRG16	fourcc_code('G', 'B', '1', '6')
> +#define DRM_FORMAT_SBGGR16	fourcc_code('B', 'Y', 'R', '2')
> +
>  /*
>   * Format Modifiers:
>   *

-- 
Regards,

Laurent Pinchart