From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Subject: Re: [PATCH v2] [media] Add RGB444_1X12 and RGB565_1X16 media bus formats Date: Mon, 29 Dec 2014 13:45:22 +0100 Message-ID: <20141229134522.07ffd4a5@bbrezillon> References: <1416126278-17708-1-git-send-email-boris.brezillon@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1416126278-17708-1-git-send-email-boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Mauro Carvalho Chehab Cc: Boris Brezillon , Hans Verkuil , Laurent Pinchart , linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Philipp Zabel , Sakari Ailus List-Id: linux-api@vger.kernel.org Hello Mauro, Last week I received a notification informing me that this patch was "Not Applicable". Could you give more details on why you think this should not go through the media-tree (or am I misunderstanding the meaning of "Not Applicable") ? I really need this patch for the atmel HLCDC DRM driver, moreover this patch from Philip [1] depends on mine. Regards, Boris [1]http://comments.gmane.org/gmane.linux.drivers.video-input-infrastructure/85952 On Sun, 16 Nov 2014 09:24:38 +0100 Boris Brezillon wrote: > Add RGB444_1X12 and RGB565_1X16 format definitions and update the > documentation. > > Signed-off-by: Boris Brezillon > Acked-by: Mauro Carvalho Chehab > --- > Changes since v1: > - keep BPP and bits per sample ordering > > Documentation/DocBook/media/v4l/subdev-formats.xml | 40 ++++++++++++++++++++++ > include/uapi/linux/media-bus-format.h | 4 ++- > 2 files changed, 43 insertions(+), 1 deletion(-) > > diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml > index 18730b9..0d6f731 100644 > --- a/Documentation/DocBook/media/v4l/subdev-formats.xml > +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml > @@ -176,6 +176,24 @@ > > > > + > + MEDIA_BUS_FMT_RGB444_1X12 > + 0x100d > + > + &dash-ent-20; > + r3 > + r2 > + r1 > + r0 > + g3 > + g2 > + g1 > + g0 > + b3 > + b2 > + b1 > + b0 > + > > MEDIA_BUS_FMT_RGB444_2X8_PADHI_BE > 0x1001 > @@ -288,6 +306,28 @@ > g4 > g3 > > + > + MEDIA_BUS_FMT_RGB565_1X16 > + 0x100d > + > + &dash-ent-16; > + r4 > + r3 > + r2 > + r1 > + r0 > + g5 > + g4 > + g3 > + g2 > + g1 > + g0 > + b4 > + b3 > + b2 > + b1 > + b0 > + > > MEDIA_BUS_FMT_BGR565_2X8_BE > 0x1005 > diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h > index 23b4090..37091c6 100644 > --- a/include/uapi/linux/media-bus-format.h > +++ b/include/uapi/linux/media-bus-format.h > @@ -33,11 +33,13 @@ > > #define MEDIA_BUS_FMT_FIXED 0x0001 > > -/* RGB - next is 0x100e */ > +/* RGB - next is 0x1010 */ > +#define MEDIA_BUS_FMT_RGB444_1X12 0x100e > #define MEDIA_BUS_FMT_RGB444_2X8_PADHI_BE 0x1001 > #define MEDIA_BUS_FMT_RGB444_2X8_PADHI_LE 0x1002 > #define MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE 0x1003 > #define MEDIA_BUS_FMT_RGB555_2X8_PADHI_LE 0x1004 > +#define MEDIA_BUS_FMT_RGB565_1X16 0x100f > #define MEDIA_BUS_FMT_BGR565_2X8_BE 0x1005 > #define MEDIA_BUS_FMT_BGR565_2X8_LE 0x1006 > #define MEDIA_BUS_FMT_RGB565_2X8_BE 0x1007 -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com