Re: [PATCH v2 1/3] Add BGR888_1X24 and GBR888_1X24 media bus formats

[Philipp Zabel <p.zabel@pengutronix.de>]

Hi Boris,

Am Montag, den 02.02.2015, 16:48 +0100 schrieb Boris Brezillon:
> Hi Philip,
> 
> On Mon, 02 Feb 2015 16:34:42 +0100
> Philipp Zabel <p.zabel@pengutronix.de> wrote:
> 
> > Am Montag, den 02.02.2015, 13:24 -0200 schrieb Mauro Carvalho Chehab:
> > > Em Mon, 02 Feb 2015 16:21:00 +0100
> > > Philipp Zabel <p.zabel@pengutronix.de> escreveu:
> > > 
> > > > Am Montag, den 02.02.2015, 16:08 +0100 schrieb Hans Verkuil:
> > > > > On 12/03/2014 02:53 PM, Philipp Zabel wrote:
> > > > > > This patch adds two more 24-bit RGB formats. BGR888 is more or less common,
> > > > > > GBR888 is used on the internal connection between the IPU display interface
> > > > > > and the TVE (VGA DAC) on i.MX53 SoCs.
> > > > > > 
> > > > > > Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> > > > > > Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > > > > 
> > > > > This three-part patch series doesn't apply. Is it on top of another patch
> > > > > series?
> > > > 
> > > > It is on top of "Add RGB444_1X12 and RGB565_1X16 media bus formats" and
> > > > "Add LVDS RGB media bus formats".
> > > > 
> > > > > Anyway, it can't be merged unless it is actually used in a driver.
> > > > 
> > > > I'd like to use these in the imx-drm driver, so this is kind of a
> > > > chicken and egg situation. Shall I submit a patch that uses the defines
> > > > to dri-devel and reference it here?
> > > 
> > > Submit the full patch series with the imx-drm driver, mentioning at the
> > > V4L2 patch that it will be applied via the DRM tree. We'll review
> > > and give our ack for it to be applied via DRM tree.
> > 
> > I'll do that, thanks.
> 
> Don't know if you plan to keep the dependency on my RGB444 and RGB565
> addition, but if you do, I guess you don't want to wait for my
> atmel-hlcdc changes, so the best solution is to include my patch in
> your series ;-).

Ok, I'll include your patch since I need the RGB565 format description
as well.

regards
Philipp