From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from galahad.ideasonboard.com ([185.26.127.97]:33761 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752042AbaLCMyI (ORCPT ); Wed, 3 Dec 2014 07:54:08 -0500 From: Laurent Pinchart To: Philipp Zabel Cc: Mauro Carvalho Chehab , Hans Verkuil , Boris Brezillon , linux-media@vger.kernel.org, kernel@pengutronix.de, Emil Renner Berthing Subject: Re: [PATCH 3/3] Add RGB666_1X24_CPADHI media bus format Date: Wed, 03 Dec 2014 14:54:44 +0200 Message-ID: <2522773.vDCBinTE4t@avalon> In-Reply-To: <1417602500-29152-3-git-send-email-p.zabel@pengutronix.de> References: <1417602500-29152-1-git-send-email-p.zabel@pengutronix.de> <1417602500-29152-3-git-send-email-p.zabel@pengutronix.de> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-media-owner@vger.kernel.org List-ID: Hi Philipp, Thank you for the patch. On Wednesday 03 December 2014 11:28:20 Philipp Zabel wrote: > Commit 9e74d2926a28 ("staging: imx-drm: add LVDS666 support for parallel > display") describes a 24-bit bus format where three 6-bit components each > take the lower part of 8 bits with the two high bits zero padded. Add a > component-wise padded media bus format RGB666_1X24_CPADHI to support this > connection. > > Cc: Emil Renner Berthing > Signed-off-by: Philipp Zabel > --- > Documentation/DocBook/media/v4l/subdev-formats.xml | 30 +++++++++++++++++++ > include/uapi/linux/media-bus-format.h | 1 + > 2 files changed, 31 insertions(+) > > diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml > b/Documentation/DocBook/media/v4l/subdev-formats.xml index 9afb846..c259b9e > 100644 > --- a/Documentation/DocBook/media/v4l/subdev-formats.xml > +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml > @@ -469,6 +469,36 @@ > b1 > b0 > > + > + MEDIA_BUS_FMT_RGB666_1X24_CPADHI > + 0x1015 > + > + &dash-ent-8; > + 0 > + 0 > + r5 > + r4 > + r3 > + r2 > + r1 > + r0 > + 0 > + 0 > + g5 > + g4 > + g3 > + g2 > + g1 > + g0 > + 0 > + 0 > + b5 > + b4 > + b3 > + b2 > + b1 > + b0 > + > > MEDIA_BUS_FMT_RGB888_1X24 > 0x100a > diff --git a/include/uapi/linux/media-bus-format.h > b/include/uapi/linux/media-bus-format.h index 977316e..ec80fb8 100644 > --- a/include/uapi/linux/media-bus-format.h > +++ b/include/uapi/linux/media-bus-format.h > @@ -45,6 +45,7 @@ Could you please update the next value comment ? > #define MEDIA_BUS_FMT_RGB565_2X8_BE 0x1007 > #define MEDIA_BUS_FMT_RGB565_2X8_LE 0x1008 > #define MEDIA_BUS_FMT_RGB666_1X18 0x1009 > +#define MEDIA_BUS_FMT_RGB666_1X24_CPADHI 0x1015 > #define MEDIA_BUS_FMT_RGB666_LVDS_SPWG 0x1010 > #define MEDIA_BUS_FMT_RGB888_1X24 0x100a > #define MEDIA_BUS_FMT_BGR888_1X24 0x1013 -- Regards, Laurent Pinchart