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

[Boris Brezillon <boris.brezillon@free-electrons.com>]

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 ;-).

Best Regards,

Boris
-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com