From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935913AbdADKUO (ORCPT ); Wed, 4 Jan 2017 05:20:14 -0500 Received: from mailapp01.imgtec.com ([195.59.15.196]:52179 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751356AbdADKUH (ORCPT ); Wed, 4 Jan 2017 05:20:07 -0500 Date: Wed, 4 Jan 2017 10:19:18 +0000 From: Eric Engestrom To: Rainer Hochecker CC: Daniel Vetter , Jani Nikula , , , , , Subject: Re: [Intel-gfx] [PATCH v2] drm: add fourcc codes for 16bit R and GR Message-ID: <20170104101918.GE2136@imgtec.com> References: <20170103190207.4569-1-fernetmenta@kodi.tv> <20170104085201.av3shack7hd74ubs@phenom.ffwll.local> <87y3yrgpoe.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline In-Reply-To: <87y3yrgpoe.fsf@intel.com> User-Agent: Mutt/1.7.2 (2016-11-26) X-Originating-IP: [10.60.4.28] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday, 2017-01-04 11:06:09 +0200, Jani Nikula wrote: > On Wed, 04 Jan 2017, Daniel Vetter wrote: > > On Tue, Jan 03, 2017 at 08:02:07PM +0100, Rainer Hochecker wrote: > >> From: Rainer Hochecker > >> > >> Now sent with git send-email: > >> > >> Signed-off-by: Rainer Hochecker > >> --- > >> include/uapi/drm/drm_fourcc.h | 7 +++++++ > >> 1 file changed, 7 insertions(+) > >> > >> diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h > >> index a5890bf..f1ef9cb 100644 > >> --- a/include/uapi/drm/drm_fourcc.h > >> +++ b/include/uapi/drm/drm_fourcc.h > >> @@ -41,10 +41,17 @@ extern "C" { > >> /* 8 bpp Red */ > >> #define DRM_FORMAT_R8 fourcc_code('R', '8', ' ', ' ') /* [7:0] R */ > >> > >> +/* 16 bpp Red */ > >> +#define DRM_FORMAT_R16 fourcc_code('R', '1', '6', ' ') /* [15: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 */ > >> > >> +/* 32 bpp GR */ > >> +#define DRM_FORMAT_RG32 fourcc_code('R', 'G', '3', '2') /* [31:0] G:R 16:16 little endian */ Also, s/G:R/R:G/ in the comment above :) > >> +#define DRM_FORMAT_GR32 fourcc_code('G', 'R', '3', '2') /* [31:0] G:R 16:16 little endian */ > > > > Now the define's name is inconsistent, since that would suggest a 5 bpp > > format with 3 bits for R and 2 bits for G. I think what we want here for > > consistency is _RG16_16 and _GR16_16, along the lines of what Ville > > suggested. > > > > Sorry that this is such a bikeshed heaven ;-) > > If there's going to be another version, please fix the commit message > while at it. "Now sent with git send-email" is useless for posterity. > > BR, > Jani. > > > > -Daniel > > > >> + > >> /* 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 */ > >> -- > >> 2.9.3 > >> > > -- > Jani Nikula, Intel Open Source Technology Center