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

Submitted by Hwang, Dongseong on April 21, 2016, 5:46 p.m.

Details

Message ID 1461260779-26455-1-git-send-email-dongseong.hwang@intel.com
State New
Headers show
Series "libdrm/fourcc: Add formats R8, RG88, GR88, NV24, NV42" ( rev: 1 ) in DRI devel

Not browsing as part of any series.

Commit Message

Hwang, Dongseong April 21, 2016, 5:46 p.m.
Produced from headers_install of 9dabb0053b63bc32ab6ad5d13209d1e43395313f
(drm-intel-nightly) in the kernel.

ChromeOS will use new format to optimize video decoding.

CC: St├ęphane Marchesin <marcheu@chromium.org>
CC: Daniele Castagna <dcastagna@chromium.org>
CC: Emil Velikov <emil.l.velikov@gmail.com>
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(-)

Patch hide | download patch | download mbox

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)
 

Comments

On 21 April 2016 at 18:46, Dongseong Hwang <dongseong.hwang@intel.com> wrote:
> Produced from headers_install of 9dabb0053b63bc32ab6ad5d13209d1e43395313f
> (drm-intel-nightly) in the kernel.
>
> ChromeOS will use new format to optimize video decoding.
>
Did you check before sending the patch out ?
As mentioned over IRC a few times - there's no need for the
update.danvet already sorted it out.

Also, I realise that CrOS development is quite high paced, although
some of us prefer quality over quantity. Meaning: don't send multiple
emails (spam?) if you have not covered (ideally) all the concerns
mentioned. Not doing so is one of the ways to alienate developers.

Pretty please ?

Thanks
Emil
I incorrectly interpreted your comments. I thought you want me to reply
every feedback.
It's first trial for me to contribute on linux. sorry for lots of noise.
I've learnt linux practice. next time it will be more smooth.
My undesirable behavior doesn't represent CrOS development. CrOS
development keeps in mind of product, so also prefers quality and
robustness.

I checked the change is already landed. Close this patch as WontFix.

Thanks you for good guidance.
Dongseong

On Thu, Apr 21, 2016 at 8:56 PM, Emil Velikov <emil.l.velikov@gmail.com>
wrote:

> On 21 April 2016 at 18:46, Dongseong Hwang <dongseong.hwang@intel.com>
> wrote:
> > Produced from headers_install of 9dabb0053b63bc32ab6ad5d13209d1e43395313f
> > (drm-intel-nightly) in the kernel.
> >
> > ChromeOS will use new format to optimize video decoding.
> >
> Did you check before sending the patch out ?
> As mentioned over IRC a few times - there's no need for the
> update.danvet already sorted it out.
>
> Also, I realise that CrOS development is quite high paced, although
> some of us prefer quality over quantity. Meaning: don't send multiple
> emails (spam?) if you have not covered (ideally) all the concerns
> mentioned. Not doing so is one of the ways to alienate developers.
>
> Pretty please ?
>
> Thanks
> Emil
>