> +/* 10 bpp Red */
> +#define DRM_FORMAT_R10               fourcc_code('R', '1', '0', ' ') /* 
> [15:0] x:R 6:10 little endian */
> +
> +/* 12 bpp Red */
> +#define DRM_FORMAT_R12               fourcc_code('R', '1', '2', ' ') /* 
> [15:0] x:R 4:12 little endian */

Are these codes arbitrary, or are they taken from somewhere else?

If they are arbitrary, maybe it'd be better to pick XR10 and XR12 instead of
R10 and R12, to allow a future patch to add other format codes with the padding
at the end if we ever need that.

Reply via email to