From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([92.198.50.35]:54113 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932416AbbBBQvx (ORCPT ); Mon, 2 Feb 2015 11:51:53 -0500 Message-ID: <1422895907.6112.19.camel@pengutronix.de> Subject: Re: [PATCH v2 1/3] Add BGR888_1X24 and GBR888_1X24 media bus formats From: Philipp Zabel To: Boris Brezillon Cc: Hans Verkuil , Hans Verkuil , Laurent Pinchart , kernel@pengutronix.de, Mauro Carvalho Chehab , linux-media@vger.kernel.org Date: Mon, 02 Feb 2015 17:51:47 +0100 In-Reply-To: <20150202164809.459eaf26@bbrezillon> References: <1417614811-15634-1-git-send-email-p.zabel@pengutronix.de> <54CF92DD.6020308@xs4all.nl> <1422890460.6112.9.camel@pengutronix.de> <20150202132421.6f2ecbd3.m.chehab@samsung.com> <1422891282.6112.15.camel@pengutronix.de> <20150202164809.459eaf26@bbrezillon> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: 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 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 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 > > > > > > Acked-by: Laurent Pinchart > > > > > > > > > > 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