diff mbox

libdrm/fourcc: Add formats R8, RG88, GR88, NV24, NV42

Message ID 1461254561-31238-1-git-send-email-dongseong.hwang@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Hwang, Dongseong April 21, 2016, 4:02 p.m. UTC
Follow-up of kernel patch: https://lists.freedesktop.org/archives/dri-devel/2015-July/086041.html

Generate it using `make headers_install`

ChromeOS will use new format to optimize video decoding.

CC: Stéphane Marchesin <marcheu@chromium.org>
CC: Daniele Castagna <dcastagna@chromium.org>
Cc: Rainer Hochecker <rainer.hochecker@onlinehome.de>
Cc: Benjamin Widawsky <benjamin.widawsky@intel.com>
CC: Chad Versace <chad.versace@intel.com>
Signed-off-by: Dongseong Hwang <dongseong.hwang@intel.com>
---
 include/drm/drm_fourcc.h | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

Comments

Hwang, Dongseong April 21, 2016, 4:06 p.m. UTC | #1
Hi Stéphane and Daniele,

Could you give me lgtm?
Daniel wants someone from client side to ack this change in order to land
it.

Kind Regards,
Dongseong

On Thu, Apr 21, 2016 at 7:02 PM, Dongseong Hwang <dongseong.hwang@intel.com>
wrote:

> Follow-up of kernel patch:
> https://lists.freedesktop.org/archives/dri-devel/2015-July/086041.html
>
> Generate it using `make headers_install`
>
> ChromeOS will use new format to optimize video decoding.
>
> CC: Stéphane Marchesin <marcheu@chromium.org>
> CC: Daniele Castagna <dcastagna@chromium.org>
> Cc: Rainer Hochecker <rainer.hochecker@onlinehome.de>
> Cc: Benjamin Widawsky <benjamin.widawsky@intel.com>
> CC: Chad Versace <chad.versace@intel.com>
> Signed-off-by: Dongseong Hwang <dongseong.hwang@intel.com>
> ---
>  include/drm/drm_fourcc.h | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/include/drm/drm_fourcc.h b/include/drm/drm_fourcc.h
> index e741b09..bf68099 100644
> --- a/include/drm/drm_fourcc.h
> +++ b/include/drm/drm_fourcc.h
> @@ -34,6 +34,13 @@
>  /* color index */
>  #define DRM_FORMAT_C8          fourcc_code('C', '8', ' ', ' ') /* [7:0] C
> */
>
> +/* 8 bpp Red */
> +#define DRM_FORMAT_R8          fourcc_code('R', '8', ' ', ' ') /* [7:0] R
> */
> +
> +/* 16 bpp RG */
> +#define DRM_FORMAT_RG88                fourcc_code('R', 'G', '8', '8') /*
> [15:0] R:G 8:8 little endian */
> +#define DRM_FORMAT_GR88                fourcc_code('G', 'R', '8', '8') /*
> [15:0] G:R 8:8 little endian */
> +
>  /* 8 bpp RGB */
>  #define DRM_FORMAT_RGB332      fourcc_code('R', 'G', 'B', '8') /* [7:0]
> R:G:B 3:3:2 */
>  #define DRM_FORMAT_BGR233      fourcc_code('B', 'G', 'R', '8') /* [7:0]
> B:G:R 2:3:3 */
> @@ -106,6 +113,8 @@
>  #define DRM_FORMAT_NV21                fourcc_code('N', 'V', '2', '1') /*
> 2x2 subsampled Cb:Cr plane */
>  #define DRM_FORMAT_NV16                fourcc_code('N', 'V', '1', '6') /*
> 2x1 subsampled Cr:Cb plane */
>  #define DRM_FORMAT_NV61                fourcc_code('N', 'V', '6', '1') /*
> 2x1 subsampled Cb:Cr plane */
> +#define DRM_FORMAT_NV24                fourcc_code('N', 'V', '2', '4') /*
> non-subsampled Cr:Cb plane */
> +#define DRM_FORMAT_NV42                fourcc_code('N', 'V', '4', '2') /*
> non-subsampled Cb:Cr plane */
>
>  /*
>   * 3 plane YCbCr
> @@ -216,7 +225,7 @@
>   * - multiple of 128 pixels for the width
>   * - multiple of  32 pixels for the height
>   *
> - * For more information: see
> http://linuxtv.org/downloads/v4l-dvb-apis/re32.html
> + * For more information: see
> https://linuxtv.org/downloads/v4l-dvb-apis/re32.html
>   */
>  #define DRM_FORMAT_MOD_SAMSUNG_64_32_TILE      fourcc_mod_code(SAMSUNG, 1)
>
> --
> 2.5.0
>
>
Hwang, Dongseong April 21, 2016, 5:45 p.m. UTC | #2
As it's landed in kernel, it doesn't need ack from client users.
Sorry for noise.

In addition, I'll send new patch with tree and commit sha info.

- Dongseong

On Thu, Apr 21, 2016 at 7:06 PM, Hwang, Dongseong <dongseong.hwang@intel.com
> wrote:

> Hi Stéphane and Daniele,
>
> Could you give me lgtm?
> Daniel wants someone from client side to ack this change in order to land
> it.
>
> Kind Regards,
> Dongseong
>
> On Thu, Apr 21, 2016 at 7:02 PM, Dongseong Hwang <
> dongseong.hwang@intel.com> wrote:
>
>> Follow-up of kernel patch:
>> https://lists.freedesktop.org/archives/dri-devel/2015-July/086041.html
>>
>> Generate it using `make headers_install`
>>
>> ChromeOS will use new format to optimize video decoding.
>>
>> CC: Stéphane Marchesin <marcheu@chromium.org>
>> CC: Daniele Castagna <dcastagna@chromium.org>
>> Cc: Rainer Hochecker <rainer.hochecker@onlinehome.de>
>> Cc: Benjamin Widawsky <benjamin.widawsky@intel.com>
>> CC: Chad Versace <chad.versace@intel.com>
>> Signed-off-by: Dongseong Hwang <dongseong.hwang@intel.com>
>> ---
>>  include/drm/drm_fourcc.h | 11 ++++++++++-
>>  1 file changed, 10 insertions(+), 1 deletion(-)
>>
>> diff --git a/include/drm/drm_fourcc.h b/include/drm/drm_fourcc.h
>> index e741b09..bf68099 100644
>> --- a/include/drm/drm_fourcc.h
>> +++ b/include/drm/drm_fourcc.h
>> @@ -34,6 +34,13 @@
>>  /* color index */
>>  #define DRM_FORMAT_C8          fourcc_code('C', '8', ' ', ' ') /* [7:0]
>> C */
>>
>> +/* 8 bpp Red */
>> +#define DRM_FORMAT_R8          fourcc_code('R', '8', ' ', ' ') /* [7:0]
>> R */
>> +
>> +/* 16 bpp RG */
>> +#define DRM_FORMAT_RG88                fourcc_code('R', 'G', '8', '8')
>> /* [15:0] R:G 8:8 little endian */
>> +#define DRM_FORMAT_GR88                fourcc_code('G', 'R', '8', '8')
>> /* [15:0] G:R 8:8 little endian */
>> +
>>  /* 8 bpp RGB */
>>  #define DRM_FORMAT_RGB332      fourcc_code('R', 'G', 'B', '8') /* [7:0]
>> R:G:B 3:3:2 */
>>  #define DRM_FORMAT_BGR233      fourcc_code('B', 'G', 'R', '8') /* [7:0]
>> B:G:R 2:3:3 */
>> @@ -106,6 +113,8 @@
>>  #define DRM_FORMAT_NV21                fourcc_code('N', 'V', '2', '1')
>> /* 2x2 subsampled Cb:Cr plane */
>>  #define DRM_FORMAT_NV16                fourcc_code('N', 'V', '1', '6')
>> /* 2x1 subsampled Cr:Cb plane */
>>  #define DRM_FORMAT_NV61                fourcc_code('N', 'V', '6', '1')
>> /* 2x1 subsampled Cb:Cr plane */
>> +#define DRM_FORMAT_NV24                fourcc_code('N', 'V', '2', '4')
>> /* non-subsampled Cr:Cb plane */
>> +#define DRM_FORMAT_NV42                fourcc_code('N', 'V', '4', '2')
>> /* non-subsampled Cb:Cr plane */
>>
>>  /*
>>   * 3 plane YCbCr
>> @@ -216,7 +225,7 @@
>>   * - multiple of 128 pixels for the width
>>   * - multiple of  32 pixels for the height
>>   *
>> - * For more information: see
>> http://linuxtv.org/downloads/v4l-dvb-apis/re32.html
>> + * For more information: see
>> https://linuxtv.org/downloads/v4l-dvb-apis/re32.html
>>   */
>>  #define DRM_FORMAT_MOD_SAMSUNG_64_32_TILE      fourcc_mod_code(SAMSUNG,
>> 1)
>>
>> --
>> 2.5.0
>>
>>
>
diff mbox

Patch

diff --git a/include/drm/drm_fourcc.h b/include/drm/drm_fourcc.h
index e741b09..bf68099 100644
--- a/include/drm/drm_fourcc.h
+++ b/include/drm/drm_fourcc.h
@@ -34,6 +34,13 @@ 
 /* color index */
 #define DRM_FORMAT_C8		fourcc_code('C', '8', ' ', ' ') /* [7:0] C */
 
+/* 8 bpp Red */
+#define DRM_FORMAT_R8		fourcc_code('R', '8', ' ', ' ') /* [7:0] R */
+
+/* 16 bpp RG */
+#define DRM_FORMAT_RG88		fourcc_code('R', 'G', '8', '8') /* [15:0] R:G 8:8 little endian */
+#define DRM_FORMAT_GR88		fourcc_code('G', 'R', '8', '8') /* [15:0] G:R 8:8 little endian */
+
 /* 8 bpp RGB */
 #define DRM_FORMAT_RGB332	fourcc_code('R', 'G', 'B', '8') /* [7:0] R:G:B 3:3:2 */
 #define DRM_FORMAT_BGR233	fourcc_code('B', 'G', 'R', '8') /* [7:0] B:G:R 2:3:3 */
@@ -106,6 +113,8 @@ 
 #define DRM_FORMAT_NV21		fourcc_code('N', 'V', '2', '1') /* 2x2 subsampled Cb:Cr plane */
 #define DRM_FORMAT_NV16		fourcc_code('N', 'V', '1', '6') /* 2x1 subsampled Cr:Cb plane */
 #define DRM_FORMAT_NV61		fourcc_code('N', 'V', '6', '1') /* 2x1 subsampled Cb:Cr plane */
+#define DRM_FORMAT_NV24		fourcc_code('N', 'V', '2', '4') /* non-subsampled Cr:Cb plane */
+#define DRM_FORMAT_NV42		fourcc_code('N', 'V', '4', '2') /* non-subsampled Cb:Cr plane */
 
 /*
  * 3 plane YCbCr
@@ -216,7 +225,7 @@ 
  * - multiple of 128 pixels for the width
  * - multiple of  32 pixels for the height
  *
- * For more information: see http://linuxtv.org/downloads/v4l-dvb-apis/re32.html
+ * For more information: see https://linuxtv.org/downloads/v4l-dvb-apis/re32.html
  */
 #define DRM_FORMAT_MOD_SAMSUNG_64_32_TILE	fourcc_mod_code(SAMSUNG, 1)