* [PATCH] Add LVDS RGB media bus formats
@ 2014-12-02 19:41 Philipp Zabel
2014-12-03 15:47 ` Sakari Ailus
0 siblings, 1 reply; 7+ messages in thread
From: Philipp Zabel @ 2014-12-02 19:41 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Hans Verkuil, Laurent Pinchart, Boris Brezillon, linux-media,
kernel, Philipp Zabel
This patch adds three new RGB media bus formats that describe
18-bit or 24-bit samples transferred over an LVDS bus with three
or four differential data pairs, serialized into 7 time slots,
using standard SPWG/PSWG/VESA or JEIDA data ordering.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
Documentation/DocBook/media/v4l/subdev-formats.xml | 189 +++++++++++++++++++++
include/uapi/linux/media-bus-format.h | 5 +-
2 files changed, 193 insertions(+), 1 deletion(-)
diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml
index 0d6f731..52d7f04 100644
--- a/Documentation/DocBook/media/v4l/subdev-formats.xml
+++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
@@ -89,6 +89,11 @@
<constant>MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE</constant>.
</para>
+ <para>On LVDS buses, usually each sample is transferred in seven time slots
+ on three (18-bit) or four (24-bit) differential data pairs at the same time.
+ The remaining bits are used for control signals as defined by SPWG/PSWG/VESA
+ or JEIDA standards.</para>
+
<para>The following tables list existing packed RGB formats.</para>
<table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb">
@@ -606,6 +611,190 @@
</tbody>
</tgroup>
</table>
+ <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb-lvds">
+ <title>LVDS RGB formats</title>
+ <tgroup cols="11">
+ <colspec colname="id" align="left" />
+ <colspec colname="code" align="center" />
+ <colspec colname="pair" align="center" />
+ <colspec colname="slot" />
+ <colspec colnum="4" colname="s00" align="center" />
+ <colspec colnum="5" colname="s01" align="center" />
+ <colspec colnum="6" colname="s02" align="center" />
+ <colspec colnum="7" colname="s03" align="center" />
+ <colspec colnum="8" colname="s04" align="center" />
+ <colspec colnum="9" colname="s05" align="center" />
+ <colspec colnum="10" colname="s06" align="center" />
+ <spanspec namest="s00" nameend="s06" spanname="s0" />
+ <thead>
+ <row>
+ <entry>Identifier</entry>
+ <entry>Code</entry>
+ <entry></entry>
+ <entry></entry>
+ <entry spanname="s0">Data organization</entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>Pair</entry>
+ <entry>Slot</entry>
+ <entry>0</entry>
+ <entry>1</entry>
+ <entry>2</entry>
+ <entry>3</entry>
+ <entry>4</entry>
+ <entry>5</entry>
+ <entry>6</entry>
+ </row>
+ </thead>
+ <tbody valign="top">
+ <row id="MEDIA-BUS-FMT-RGB666-LVDS-SPWG">
+ <entry>MEDIA_BUS_FMT_RGB666_LVDS_SPWG</entry>
+ <entry>0x1010</entry>
+ <entry>data0</entry>
+ <entry></entry>
+ <entry>g<subscript>0</subscript></entry>
+ <entry>r<subscript>5</subscript></entry>
+ <entry>r<subscript>4</subscript></entry>
+ <entry>r<subscript>3</subscript></entry>
+ <entry>r<subscript>2</subscript></entry>
+ <entry>r<subscript>1</subscript></entry>
+ <entry>r<subscript>0</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>data1</entry>
+ <entry></entry>
+ <entry>b<subscript>1</subscript></entry>
+ <entry>b<subscript>0</subscript></entry>
+ <entry>g<subscript>5</subscript></entry>
+ <entry>g<subscript>4</subscript></entry>
+ <entry>g<subscript>3</subscript></entry>
+ <entry>g<subscript>2</subscript></entry>
+ <entry>g<subscript>1</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>data2</entry>
+ <entry></entry>
+ <entry>de</entry>
+ <entry>vs</entry>
+ <entry>hs</entry>
+ <entry>b<subscript>5</subscript></entry>
+ <entry>b<subscript>4</subscript></entry>
+ <entry>b<subscript>3</subscript></entry>
+ <entry>b<subscript>2</subscript></entry>
+ </row>
+ <row id="MEDIA-BUS-FMT-RGB888-LVDS-SPWG">
+ <entry>MEDIA_BUS_FMT_RGB888_LVDS_SPWG</entry>
+ <entry>0x1011</entry>
+ <entry>data0</entry>
+ <entry></entry>
+ <entry>g<subscript>0</subscript></entry>
+ <entry>r<subscript>5</subscript></entry>
+ <entry>r<subscript>4</subscript></entry>
+ <entry>r<subscript>3</subscript></entry>
+ <entry>r<subscript>2</subscript></entry>
+ <entry>r<subscript>1</subscript></entry>
+ <entry>r<subscript>0</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>data1</entry>
+ <entry></entry>
+ <entry>b<subscript>1</subscript></entry>
+ <entry>b<subscript>0</subscript></entry>
+ <entry>g<subscript>5</subscript></entry>
+ <entry>g<subscript>4</subscript></entry>
+ <entry>g<subscript>3</subscript></entry>
+ <entry>g<subscript>2</subscript></entry>
+ <entry>g<subscript>1</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>data2</entry>
+ <entry></entry>
+ <entry>de</entry>
+ <entry>vs</entry>
+ <entry>hs</entry>
+ <entry>b<subscript>5</subscript></entry>
+ <entry>b<subscript>4</subscript></entry>
+ <entry>b<subscript>3</subscript></entry>
+ <entry>b<subscript>2</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>data3</entry>
+ <entry></entry>
+ <entry>ctl</entry>
+ <entry>b<subscript>7</subscript></entry>
+ <entry>b<subscript>6</subscript></entry>
+ <entry>g<subscript>7</subscript></entry>
+ <entry>g<subscript>6</subscript></entry>
+ <entry>r<subscript>7</subscript></entry>
+ <entry>r<subscript>6</subscript></entry>
+ </row>
+ <row id="MEDIA-BUS-FMT-RGB888-LVDS-JEIDA">
+ <entry>MEDIA_BUS_FMT_RGB888_LVDS_JEIDA</entry>
+ <entry>0x1012</entry>
+ <entry>data0</entry>
+ <entry></entry>
+ <entry>g<subscript>2</subscript></entry>
+ <entry>r<subscript>7</subscript></entry>
+ <entry>r<subscript>6</subscript></entry>
+ <entry>r<subscript>5</subscript></entry>
+ <entry>r<subscript>4</subscript></entry>
+ <entry>r<subscript>3</subscript></entry>
+ <entry>r<subscript>2</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>data1</entry>
+ <entry></entry>
+ <entry>b<subscript>3</subscript></entry>
+ <entry>b<subscript>2</subscript></entry>
+ <entry>g<subscript>7</subscript></entry>
+ <entry>g<subscript>6</subscript></entry>
+ <entry>g<subscript>5</subscript></entry>
+ <entry>g<subscript>4</subscript></entry>
+ <entry>g<subscript>3</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>data2</entry>
+ <entry></entry>
+ <entry>de</entry>
+ <entry>vs</entry>
+ <entry>hs</entry>
+ <entry>b<subscript>7</subscript></entry>
+ <entry>b<subscript>6</subscript></entry>
+ <entry>b<subscript>5</subscript></entry>
+ <entry>b<subscript>4</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>data3</entry>
+ <entry></entry>
+ <entry>ctl</entry>
+ <entry>b<subscript>1</subscript></entry>
+ <entry>b<subscript>0</subscript></entry>
+ <entry>g<subscript>1</subscript></entry>
+ <entry>g<subscript>0</subscript></entry>
+ <entry>r<subscript>1</subscript></entry>
+ <entry>r<subscript>0</subscript></entry>
+ </row>
+ </tbody>
+ </tgroup>
+ </table>
</section>
<section>
diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h
index 37091c6..7f8b1e2 100644
--- a/include/uapi/linux/media-bus-format.h
+++ b/include/uapi/linux/media-bus-format.h
@@ -33,7 +33,7 @@
#define MEDIA_BUS_FMT_FIXED 0x0001
-/* RGB - next is 0x1010 */
+/* RGB - next is 0x1013 */
#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
@@ -45,9 +45,12 @@
#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_LVDS_SPWG 0x1010
#define MEDIA_BUS_FMT_RGB888_1X24 0x100a
#define MEDIA_BUS_FMT_RGB888_2X12_BE 0x100b
#define MEDIA_BUS_FMT_RGB888_2X12_LE 0x100c
+#define MEDIA_BUS_FMT_RGB888_LVDS_SPWG 0x1011
+#define MEDIA_BUS_FMT_RGB888_LVDS_JEIDA 0x1012
#define MEDIA_BUS_FMT_ARGB8888_1X32 0x100d
/* YUV (including grey) - next is 0x2024 */
--
2.1.3
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] Add LVDS RGB media bus formats
2014-12-02 19:41 [PATCH] Add LVDS RGB media bus formats Philipp Zabel
@ 2014-12-03 15:47 ` Sakari Ailus
2014-12-04 10:44 ` Philipp Zabel
0 siblings, 1 reply; 7+ messages in thread
From: Sakari Ailus @ 2014-12-03 15:47 UTC (permalink / raw)
To: Philipp Zabel
Cc: Mauro Carvalho Chehab, Hans Verkuil, Laurent Pinchart,
Boris Brezillon, linux-media, kernel
Hi Philipp,
On Tue, Dec 02, 2014 at 08:41:24PM +0100, Philipp Zabel wrote:
> This patch adds three new RGB media bus formats that describe
> 18-bit or 24-bit samples transferred over an LVDS bus with three
> or four differential data pairs, serialized into 7 time slots,
> using standard SPWG/PSWG/VESA or JEIDA data ordering.
>
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
> Documentation/DocBook/media/v4l/subdev-formats.xml | 189 +++++++++++++++++++++
> include/uapi/linux/media-bus-format.h | 5 +-
> 2 files changed, 193 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml
> index 0d6f731..52d7f04 100644
> --- a/Documentation/DocBook/media/v4l/subdev-formats.xml
> +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
> @@ -89,6 +89,11 @@
> <constant>MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE</constant>.
> </para>
>
> + <para>On LVDS buses, usually each sample is transferred in seven time slots
> + on three (18-bit) or four (24-bit) differential data pairs at the same time.
> + The remaining bits are used for control signals as defined by SPWG/PSWG/VESA
> + or JEIDA standards.</para>
> +
> <para>The following tables list existing packed RGB formats.</para>
>
> <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb">
> @@ -606,6 +611,190 @@
> </tbody>
> </tgroup>
> </table>
> + <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb-lvds">
> + <title>LVDS RGB formats</title>
> + <tgroup cols="11">
> + <colspec colname="id" align="left" />
> + <colspec colname="code" align="center" />
> + <colspec colname="pair" align="center" />
> + <colspec colname="slot" />
> + <colspec colnum="4" colname="s00" align="center" />
> + <colspec colnum="5" colname="s01" align="center" />
> + <colspec colnum="6" colname="s02" align="center" />
> + <colspec colnum="7" colname="s03" align="center" />
> + <colspec colnum="8" colname="s04" align="center" />
> + <colspec colnum="9" colname="s05" align="center" />
> + <colspec colnum="10" colname="s06" align="center" />
> + <spanspec namest="s00" nameend="s06" spanname="s0" />
> + <thead>
> + <row>
> + <entry>Identifier</entry>
> + <entry>Code</entry>
> + <entry></entry>
> + <entry></entry>
> + <entry spanname="s0">Data organization</entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>Pair</entry>
> + <entry>Slot</entry>
> + <entry>0</entry>
> + <entry>1</entry>
> + <entry>2</entry>
> + <entry>3</entry>
> + <entry>4</entry>
> + <entry>5</entry>
> + <entry>6</entry>
> + </row>
> + </thead>
> + <tbody valign="top">
> + <row id="MEDIA-BUS-FMT-RGB666-LVDS-SPWG">
> + <entry>MEDIA_BUS_FMT_RGB666_LVDS_SPWG</entry>
> + <entry>0x1010</entry>
> + <entry>data0</entry>
> + <entry></entry>
> + <entry>g<subscript>0</subscript></entry>
> + <entry>r<subscript>5</subscript></entry>
> + <entry>r<subscript>4</subscript></entry>
> + <entry>r<subscript>3</subscript></entry>
> + <entry>r<subscript>2</subscript></entry>
> + <entry>r<subscript>1</subscript></entry>
> + <entry>r<subscript>0</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>data1</entry>
> + <entry></entry>
> + <entry>b<subscript>1</subscript></entry>
> + <entry>b<subscript>0</subscript></entry>
> + <entry>g<subscript>5</subscript></entry>
> + <entry>g<subscript>4</subscript></entry>
> + <entry>g<subscript>3</subscript></entry>
> + <entry>g<subscript>2</subscript></entry>
> + <entry>g<subscript>1</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>data2</entry>
> + <entry></entry>
> + <entry>de</entry>
> + <entry>vs</entry>
> + <entry>hs</entry>
> + <entry>b<subscript>5</subscript></entry>
> + <entry>b<subscript>4</subscript></entry>
> + <entry>b<subscript>3</subscript></entry>
> + <entry>b<subscript>2</subscript></entry>
> + </row>
> + <row id="MEDIA-BUS-FMT-RGB888-LVDS-SPWG">
> + <entry>MEDIA_BUS_FMT_RGB888_LVDS_SPWG</entry>
> + <entry>0x1011</entry>
> + <entry>data0</entry>
> + <entry></entry>
> + <entry>g<subscript>0</subscript></entry>
> + <entry>r<subscript>5</subscript></entry>
> + <entry>r<subscript>4</subscript></entry>
> + <entry>r<subscript>3</subscript></entry>
> + <entry>r<subscript>2</subscript></entry>
> + <entry>r<subscript>1</subscript></entry>
> + <entry>r<subscript>0</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>data1</entry>
> + <entry></entry>
> + <entry>b<subscript>1</subscript></entry>
> + <entry>b<subscript>0</subscript></entry>
> + <entry>g<subscript>5</subscript></entry>
> + <entry>g<subscript>4</subscript></entry>
> + <entry>g<subscript>3</subscript></entry>
> + <entry>g<subscript>2</subscript></entry>
> + <entry>g<subscript>1</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>data2</entry>
> + <entry></entry>
> + <entry>de</entry>
> + <entry>vs</entry>
> + <entry>hs</entry>
> + <entry>b<subscript>5</subscript></entry>
> + <entry>b<subscript>4</subscript></entry>
> + <entry>b<subscript>3</subscript></entry>
> + <entry>b<subscript>2</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>data3</entry>
> + <entry></entry>
> + <entry>ctl</entry>
> + <entry>b<subscript>7</subscript></entry>
> + <entry>b<subscript>6</subscript></entry>
> + <entry>g<subscript>7</subscript></entry>
> + <entry>g<subscript>6</subscript></entry>
> + <entry>r<subscript>7</subscript></entry>
> + <entry>r<subscript>6</subscript></entry>
> + </row>
> + <row id="MEDIA-BUS-FMT-RGB888-LVDS-JEIDA">
> + <entry>MEDIA_BUS_FMT_RGB888_LVDS_JEIDA</entry>
> + <entry>0x1012</entry>
> + <entry>data0</entry>
> + <entry></entry>
> + <entry>g<subscript>2</subscript></entry>
> + <entry>r<subscript>7</subscript></entry>
> + <entry>r<subscript>6</subscript></entry>
> + <entry>r<subscript>5</subscript></entry>
> + <entry>r<subscript>4</subscript></entry>
> + <entry>r<subscript>3</subscript></entry>
> + <entry>r<subscript>2</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>data1</entry>
> + <entry></entry>
> + <entry>b<subscript>3</subscript></entry>
> + <entry>b<subscript>2</subscript></entry>
> + <entry>g<subscript>7</subscript></entry>
> + <entry>g<subscript>6</subscript></entry>
> + <entry>g<subscript>5</subscript></entry>
> + <entry>g<subscript>4</subscript></entry>
> + <entry>g<subscript>3</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>data2</entry>
> + <entry></entry>
> + <entry>de</entry>
> + <entry>vs</entry>
> + <entry>hs</entry>
> + <entry>b<subscript>7</subscript></entry>
> + <entry>b<subscript>6</subscript></entry>
> + <entry>b<subscript>5</subscript></entry>
> + <entry>b<subscript>4</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>data3</entry>
> + <entry></entry>
> + <entry>ctl</entry>
> + <entry>b<subscript>1</subscript></entry>
> + <entry>b<subscript>0</subscript></entry>
> + <entry>g<subscript>1</subscript></entry>
> + <entry>g<subscript>0</subscript></entry>
> + <entry>r<subscript>1</subscript></entry>
> + <entry>r<subscript>0</subscript></entry>
> + </row>
> + </tbody>
> + </tgroup>
> + </table>
In general, I'd be more concerned with the pixel data itself than on low
level bus signalling related matters. For that reason I'd just say the
non-data bits are simply undefined.
We already support the CSI-2 bus, but the formats we use make no mention of
the standard itself. (A note to myself, they still should have been
separated from parallel format definitions.)
Could you instead create more generic format definitions that are not
specific to SPWG/PSWG/VESA or JEIDA?
I think this falls to the domain of the parallel-like interfaces since it
defines per bus pin which data is transferred over which pin. Is the wire
order defined by the above standards as well? It is exactly the opposite to
what is currently defined there.
--
Kind regards,
Sakari Ailus
e-mail: sakari.ailus@iki.fi XMPP: sailus@retiisi.org.uk
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Add LVDS RGB media bus formats
2014-12-03 15:47 ` Sakari Ailus
@ 2014-12-04 10:44 ` Philipp Zabel
2014-12-08 0:59 ` Sakari Ailus
0 siblings, 1 reply; 7+ messages in thread
From: Philipp Zabel @ 2014-12-04 10:44 UTC (permalink / raw)
To: Sakari Ailus
Cc: Mauro Carvalho Chehab, Hans Verkuil, Laurent Pinchart,
Boris Brezillon, linux-media, kernel
Hi Sakari,
Am Mittwoch, den 03.12.2014, 17:47 +0200 schrieb Sakari Ailus:
> Hi Philipp,
>
> On Tue, Dec 02, 2014 at 08:41:24PM +0100, Philipp Zabel wrote:
> > This patch adds three new RGB media bus formats that describe
> > 18-bit or 24-bit samples transferred over an LVDS bus with three
> > or four differential data pairs, serialized into 7 time slots,
> > using standard SPWG/PSWG/VESA or JEIDA data ordering.
> >
> > Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> > ---
> > Documentation/DocBook/media/v4l/subdev-formats.xml | 189 +++++++++++++++++++++
> > include/uapi/linux/media-bus-format.h | 5 +-
> > 2 files changed, 193 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml
> > index 0d6f731..52d7f04 100644
> > --- a/Documentation/DocBook/media/v4l/subdev-formats.xml
> > +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
> > @@ -89,6 +89,11 @@
> > <constant>MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE</constant>.
> > </para>
> >
> > + <para>On LVDS buses, usually each sample is transferred in seven time slots
> > + on three (18-bit) or four (24-bit) differential data pairs at the same time.
> > + The remaining bits are used for control signals as defined by SPWG/PSWG/VESA
> > + or JEIDA standards.</para>
> > +
> > <para>The following tables list existing packed RGB formats.</para>
> >
> > <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb">
> > @@ -606,6 +611,190 @@
> > </tbody>
> > </tgroup>
> > </table>
> > + <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb-lvds">
> > + <title>LVDS RGB formats</title>
> > + <tgroup cols="11">
> > + <colspec colname="id" align="left" />
> > + <colspec colname="code" align="center" />
> > + <colspec colname="pair" align="center" />
> > + <colspec colname="slot" />
> > + <colspec colnum="4" colname="s00" align="center" />
> > + <colspec colnum="5" colname="s01" align="center" />
> > + <colspec colnum="6" colname="s02" align="center" />
> > + <colspec colnum="7" colname="s03" align="center" />
> > + <colspec colnum="8" colname="s04" align="center" />
> > + <colspec colnum="9" colname="s05" align="center" />
> > + <colspec colnum="10" colname="s06" align="center" />
> > + <spanspec namest="s00" nameend="s06" spanname="s0" />
> > + <thead>
> > + <row>
> > + <entry>Identifier</entry>
> > + <entry>Code</entry>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry spanname="s0">Data organization</entry>
> > + </row>
> > + <row>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry>Pair</entry>
> > + <entry>Slot</entry>
> > + <entry>0</entry>
> > + <entry>1</entry>
> > + <entry>2</entry>
> > + <entry>3</entry>
> > + <entry>4</entry>
> > + <entry>5</entry>
> > + <entry>6</entry>
> > + </row>
> > + </thead>
> > + <tbody valign="top">
> > + <row id="MEDIA-BUS-FMT-RGB666-LVDS-SPWG">
> > + <entry>MEDIA_BUS_FMT_RGB666_LVDS_SPWG</entry>
> > + <entry>0x1010</entry>
> > + <entry>data0</entry>
> > + <entry></entry>
> > + <entry>g<subscript>0</subscript></entry>
> > + <entry>r<subscript>5</subscript></entry>
> > + <entry>r<subscript>4</subscript></entry>
> > + <entry>r<subscript>3</subscript></entry>
> > + <entry>r<subscript>2</subscript></entry>
> > + <entry>r<subscript>1</subscript></entry>
> > + <entry>r<subscript>0</subscript></entry>
> > + </row>
> > + <row>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry>data1</entry>
> > + <entry></entry>
> > + <entry>b<subscript>1</subscript></entry>
> > + <entry>b<subscript>0</subscript></entry>
> > + <entry>g<subscript>5</subscript></entry>
> > + <entry>g<subscript>4</subscript></entry>
> > + <entry>g<subscript>3</subscript></entry>
> > + <entry>g<subscript>2</subscript></entry>
> > + <entry>g<subscript>1</subscript></entry>
> > + </row>
> > + <row>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry>data2</entry>
> > + <entry></entry>
> > + <entry>de</entry>
> > + <entry>vs</entry>
> > + <entry>hs</entry>
> > + <entry>b<subscript>5</subscript></entry>
> > + <entry>b<subscript>4</subscript></entry>
> > + <entry>b<subscript>3</subscript></entry>
> > + <entry>b<subscript>2</subscript></entry>
> > + </row>
> > + <row id="MEDIA-BUS-FMT-RGB888-LVDS-SPWG">
> > + <entry>MEDIA_BUS_FMT_RGB888_LVDS_SPWG</entry>
> > + <entry>0x1011</entry>
> > + <entry>data0</entry>
> > + <entry></entry>
> > + <entry>g<subscript>0</subscript></entry>
> > + <entry>r<subscript>5</subscript></entry>
> > + <entry>r<subscript>4</subscript></entry>
> > + <entry>r<subscript>3</subscript></entry>
> > + <entry>r<subscript>2</subscript></entry>
> > + <entry>r<subscript>1</subscript></entry>
> > + <entry>r<subscript>0</subscript></entry>
> > + </row>
> > + <row>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry>data1</entry>
> > + <entry></entry>
> > + <entry>b<subscript>1</subscript></entry>
> > + <entry>b<subscript>0</subscript></entry>
> > + <entry>g<subscript>5</subscript></entry>
> > + <entry>g<subscript>4</subscript></entry>
> > + <entry>g<subscript>3</subscript></entry>
> > + <entry>g<subscript>2</subscript></entry>
> > + <entry>g<subscript>1</subscript></entry>
> > + </row>
> > + <row>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry>data2</entry>
> > + <entry></entry>
> > + <entry>de</entry>
> > + <entry>vs</entry>
> > + <entry>hs</entry>
> > + <entry>b<subscript>5</subscript></entry>
> > + <entry>b<subscript>4</subscript></entry>
> > + <entry>b<subscript>3</subscript></entry>
> > + <entry>b<subscript>2</subscript></entry>
> > + </row>
> > + <row>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry>data3</entry>
> > + <entry></entry>
> > + <entry>ctl</entry>
> > + <entry>b<subscript>7</subscript></entry>
> > + <entry>b<subscript>6</subscript></entry>
> > + <entry>g<subscript>7</subscript></entry>
> > + <entry>g<subscript>6</subscript></entry>
> > + <entry>r<subscript>7</subscript></entry>
> > + <entry>r<subscript>6</subscript></entry>
> > + </row>
> > + <row id="MEDIA-BUS-FMT-RGB888-LVDS-JEIDA">
> > + <entry>MEDIA_BUS_FMT_RGB888_LVDS_JEIDA</entry>
> > + <entry>0x1012</entry>
> > + <entry>data0</entry>
> > + <entry></entry>
> > + <entry>g<subscript>2</subscript></entry>
> > + <entry>r<subscript>7</subscript></entry>
> > + <entry>r<subscript>6</subscript></entry>
> > + <entry>r<subscript>5</subscript></entry>
> > + <entry>r<subscript>4</subscript></entry>
> > + <entry>r<subscript>3</subscript></entry>
> > + <entry>r<subscript>2</subscript></entry>
> > + </row>
> > + <row>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry>data1</entry>
> > + <entry></entry>
> > + <entry>b<subscript>3</subscript></entry>
> > + <entry>b<subscript>2</subscript></entry>
> > + <entry>g<subscript>7</subscript></entry>
> > + <entry>g<subscript>6</subscript></entry>
> > + <entry>g<subscript>5</subscript></entry>
> > + <entry>g<subscript>4</subscript></entry>
> > + <entry>g<subscript>3</subscript></entry>
> > + </row>
> > + <row>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry>data2</entry>
> > + <entry></entry>
> > + <entry>de</entry>
> > + <entry>vs</entry>
> > + <entry>hs</entry>
> > + <entry>b<subscript>7</subscript></entry>
> > + <entry>b<subscript>6</subscript></entry>
> > + <entry>b<subscript>5</subscript></entry>
> > + <entry>b<subscript>4</subscript></entry>
> > + </row>
> > + <row>
> > + <entry></entry>
> > + <entry></entry>
> > + <entry>data3</entry>
> > + <entry></entry>
> > + <entry>ctl</entry>
> > + <entry>b<subscript>1</subscript></entry>
> > + <entry>b<subscript>0</subscript></entry>
> > + <entry>g<subscript>1</subscript></entry>
> > + <entry>g<subscript>0</subscript></entry>
> > + <entry>r<subscript>1</subscript></entry>
> > + <entry>r<subscript>0</subscript></entry>
> > + </row>
> > + </tbody>
> > + </tgroup>
> > + </table>
>
> In general, I'd be more concerned with the pixel data itself than on low
> level bus signalling related matters. For that reason I'd just say the
> non-data bits are simply undefined.
Sure, I could change the vs/hs/de/ctl to undefined.
> Could you instead create more generic format definitions that are not
> specific to SPWG/PSWG/VESA or JEIDA?
Then what should the nomenclature be? On the LVDS bus we have one sample
per pixel clock, but the data lanes change at seven times the pixel
clock rate, at least for the proposed formats:
time ------------>
RGB666_LVDS_SPWG:
CLK -----.________.-----
DATA0 G0 R5 R4 R3 R2 R1 R0
DATA1 B1 B0 G5 G4 G3 G2 G1
DATA2 x x x B5 B4 B3 B2
RGB888_LVDS_SPWG:
CLK -----.________.-----
DATA0 G0 R5 R4 R3 R2 R1 R0
DATA1 B1 B0 G5 G4 G3 G2 G1
DATA2 x x x B5 B4 B3 B2
DATA3 x B7 B6 G7 G6 R7 R6
RGB888_LVDS_JEIDA:
CLK -----.________.-----
DATA0 G2 R7 R6 R5 R4 R3 R2
DATA1 B3 B2 G7 G6 G5 G4 G3
DATA2 x x x B7 B6 B5 B4
DATA3 x B1 B0 G1 G0 R1 R0
We could describe this as "RGB888_LVDS_1X7X4_%s", but what should "%s"
be to differentiate between the MSB on DATA3 (SPWG) and LSB on DATA3
(JEIDA) variants?
Also, I think it would helpful to at least have a comment which format
definitions correspond to the SPWG/JEIDA ordering, because that's what
you often find in LVDS panel data sheets. I'd prefer to just keep them.
> I think this falls to the domain of the parallel-like interfaces since it
> defines per bus pin which data is transferred over which pin.
That might be, but as you can see above the ordering is a bit special.
> Is the wire order defined by the above standards as well?
Yes.
> It is exactly the opposite to what is currently defined there.
Note that the LVDS table has the wire pairs in rows, the columns
represent the seven time slots, because I thought a four-wide table with
seven rows per entry looked a bit strange, and that's the order these
formats are commonly depicted.
regards
Philipp
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Add LVDS RGB media bus formats
2014-12-04 10:44 ` Philipp Zabel
@ 2014-12-08 0:59 ` Sakari Ailus
0 siblings, 0 replies; 7+ messages in thread
From: Sakari Ailus @ 2014-12-08 0:59 UTC (permalink / raw)
To: Philipp Zabel
Cc: Mauro Carvalho Chehab, Hans Verkuil, Laurent Pinchart,
Boris Brezillon, linux-media, kernel
Hi Philipp,
On Thu, Dec 04, 2014 at 11:44:32AM +0100, Philipp Zabel wrote:
> Hi Sakari,
>
> Am Mittwoch, den 03.12.2014, 17:47 +0200 schrieb Sakari Ailus:
> > Hi Philipp,
> >
> > On Tue, Dec 02, 2014 at 08:41:24PM +0100, Philipp Zabel wrote:
> > > This patch adds three new RGB media bus formats that describe
> > > 18-bit or 24-bit samples transferred over an LVDS bus with three
> > > or four differential data pairs, serialized into 7 time slots,
> > > using standard SPWG/PSWG/VESA or JEIDA data ordering.
> > >
> > > Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> > > ---
> > > Documentation/DocBook/media/v4l/subdev-formats.xml | 189 +++++++++++++++++++++
> > > include/uapi/linux/media-bus-format.h | 5 +-
> > > 2 files changed, 193 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml
> > > index 0d6f731..52d7f04 100644
> > > --- a/Documentation/DocBook/media/v4l/subdev-formats.xml
> > > +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
> > > @@ -89,6 +89,11 @@
> > > <constant>MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE</constant>.
> > > </para>
> > >
> > > + <para>On LVDS buses, usually each sample is transferred in seven time slots
> > > + on three (18-bit) or four (24-bit) differential data pairs at the same time.
> > > + The remaining bits are used for control signals as defined by SPWG/PSWG/VESA
> > > + or JEIDA standards.</para>
> > > +
> > > <para>The following tables list existing packed RGB formats.</para>
> > >
> > > <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb">
> > > @@ -606,6 +611,190 @@
> > > </tbody>
> > > </tgroup>
> > > </table>
> > > + <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb-lvds">
> > > + <title>LVDS RGB formats</title>
> > > + <tgroup cols="11">
> > > + <colspec colname="id" align="left" />
> > > + <colspec colname="code" align="center" />
> > > + <colspec colname="pair" align="center" />
> > > + <colspec colname="slot" />
> > > + <colspec colnum="4" colname="s00" align="center" />
> > > + <colspec colnum="5" colname="s01" align="center" />
> > > + <colspec colnum="6" colname="s02" align="center" />
> > > + <colspec colnum="7" colname="s03" align="center" />
> > > + <colspec colnum="8" colname="s04" align="center" />
> > > + <colspec colnum="9" colname="s05" align="center" />
> > > + <colspec colnum="10" colname="s06" align="center" />
> > > + <spanspec namest="s00" nameend="s06" spanname="s0" />
> > > + <thead>
> > > + <row>
> > > + <entry>Identifier</entry>
> > > + <entry>Code</entry>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry spanname="s0">Data organization</entry>
> > > + </row>
> > > + <row>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry>Pair</entry>
> > > + <entry>Slot</entry>
> > > + <entry>0</entry>
> > > + <entry>1</entry>
> > > + <entry>2</entry>
> > > + <entry>3</entry>
> > > + <entry>4</entry>
> > > + <entry>5</entry>
> > > + <entry>6</entry>
> > > + </row>
> > > + </thead>
> > > + <tbody valign="top">
> > > + <row id="MEDIA-BUS-FMT-RGB666-LVDS-SPWG">
> > > + <entry>MEDIA_BUS_FMT_RGB666_LVDS_SPWG</entry>
> > > + <entry>0x1010</entry>
> > > + <entry>data0</entry>
> > > + <entry></entry>
> > > + <entry>g<subscript>0</subscript></entry>
> > > + <entry>r<subscript>5</subscript></entry>
> > > + <entry>r<subscript>4</subscript></entry>
> > > + <entry>r<subscript>3</subscript></entry>
> > > + <entry>r<subscript>2</subscript></entry>
> > > + <entry>r<subscript>1</subscript></entry>
> > > + <entry>r<subscript>0</subscript></entry>
> > > + </row>
> > > + <row>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry>data1</entry>
> > > + <entry></entry>
> > > + <entry>b<subscript>1</subscript></entry>
> > > + <entry>b<subscript>0</subscript></entry>
> > > + <entry>g<subscript>5</subscript></entry>
> > > + <entry>g<subscript>4</subscript></entry>
> > > + <entry>g<subscript>3</subscript></entry>
> > > + <entry>g<subscript>2</subscript></entry>
> > > + <entry>g<subscript>1</subscript></entry>
> > > + </row>
> > > + <row>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry>data2</entry>
> > > + <entry></entry>
> > > + <entry>de</entry>
> > > + <entry>vs</entry>
> > > + <entry>hs</entry>
> > > + <entry>b<subscript>5</subscript></entry>
> > > + <entry>b<subscript>4</subscript></entry>
> > > + <entry>b<subscript>3</subscript></entry>
> > > + <entry>b<subscript>2</subscript></entry>
> > > + </row>
> > > + <row id="MEDIA-BUS-FMT-RGB888-LVDS-SPWG">
> > > + <entry>MEDIA_BUS_FMT_RGB888_LVDS_SPWG</entry>
> > > + <entry>0x1011</entry>
> > > + <entry>data0</entry>
> > > + <entry></entry>
> > > + <entry>g<subscript>0</subscript></entry>
> > > + <entry>r<subscript>5</subscript></entry>
> > > + <entry>r<subscript>4</subscript></entry>
> > > + <entry>r<subscript>3</subscript></entry>
> > > + <entry>r<subscript>2</subscript></entry>
> > > + <entry>r<subscript>1</subscript></entry>
> > > + <entry>r<subscript>0</subscript></entry>
> > > + </row>
> > > + <row>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry>data1</entry>
> > > + <entry></entry>
> > > + <entry>b<subscript>1</subscript></entry>
> > > + <entry>b<subscript>0</subscript></entry>
> > > + <entry>g<subscript>5</subscript></entry>
> > > + <entry>g<subscript>4</subscript></entry>
> > > + <entry>g<subscript>3</subscript></entry>
> > > + <entry>g<subscript>2</subscript></entry>
> > > + <entry>g<subscript>1</subscript></entry>
> > > + </row>
> > > + <row>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry>data2</entry>
> > > + <entry></entry>
> > > + <entry>de</entry>
> > > + <entry>vs</entry>
> > > + <entry>hs</entry>
> > > + <entry>b<subscript>5</subscript></entry>
> > > + <entry>b<subscript>4</subscript></entry>
> > > + <entry>b<subscript>3</subscript></entry>
> > > + <entry>b<subscript>2</subscript></entry>
> > > + </row>
> > > + <row>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry>data3</entry>
> > > + <entry></entry>
> > > + <entry>ctl</entry>
> > > + <entry>b<subscript>7</subscript></entry>
> > > + <entry>b<subscript>6</subscript></entry>
> > > + <entry>g<subscript>7</subscript></entry>
> > > + <entry>g<subscript>6</subscript></entry>
> > > + <entry>r<subscript>7</subscript></entry>
> > > + <entry>r<subscript>6</subscript></entry>
> > > + </row>
> > > + <row id="MEDIA-BUS-FMT-RGB888-LVDS-JEIDA">
> > > + <entry>MEDIA_BUS_FMT_RGB888_LVDS_JEIDA</entry>
> > > + <entry>0x1012</entry>
> > > + <entry>data0</entry>
> > > + <entry></entry>
> > > + <entry>g<subscript>2</subscript></entry>
> > > + <entry>r<subscript>7</subscript></entry>
> > > + <entry>r<subscript>6</subscript></entry>
> > > + <entry>r<subscript>5</subscript></entry>
> > > + <entry>r<subscript>4</subscript></entry>
> > > + <entry>r<subscript>3</subscript></entry>
> > > + <entry>r<subscript>2</subscript></entry>
> > > + </row>
> > > + <row>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry>data1</entry>
> > > + <entry></entry>
> > > + <entry>b<subscript>3</subscript></entry>
> > > + <entry>b<subscript>2</subscript></entry>
> > > + <entry>g<subscript>7</subscript></entry>
> > > + <entry>g<subscript>6</subscript></entry>
> > > + <entry>g<subscript>5</subscript></entry>
> > > + <entry>g<subscript>4</subscript></entry>
> > > + <entry>g<subscript>3</subscript></entry>
> > > + </row>
> > > + <row>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry>data2</entry>
> > > + <entry></entry>
> > > + <entry>de</entry>
> > > + <entry>vs</entry>
> > > + <entry>hs</entry>
> > > + <entry>b<subscript>7</subscript></entry>
> > > + <entry>b<subscript>6</subscript></entry>
> > > + <entry>b<subscript>5</subscript></entry>
> > > + <entry>b<subscript>4</subscript></entry>
> > > + </row>
> > > + <row>
> > > + <entry></entry>
> > > + <entry></entry>
> > > + <entry>data3</entry>
> > > + <entry></entry>
> > > + <entry>ctl</entry>
> > > + <entry>b<subscript>1</subscript></entry>
> > > + <entry>b<subscript>0</subscript></entry>
> > > + <entry>g<subscript>1</subscript></entry>
> > > + <entry>g<subscript>0</subscript></entry>
> > > + <entry>r<subscript>1</subscript></entry>
> > > + <entry>r<subscript>0</subscript></entry>
> > > + </row>
> > > + </tbody>
> > > + </tgroup>
> > > + </table>
> >
> > In general, I'd be more concerned with the pixel data itself than on low
> > level bus signalling related matters. For that reason I'd just say the
> > non-data bits are simply undefined.
>
> Sure, I could change the vs/hs/de/ctl to undefined.
Agreed.
> > Could you instead create more generic format definitions that are not
> > specific to SPWG/PSWG/VESA or JEIDA?
>
> Then what should the nomenclature be? On the LVDS bus we have one sample
> per pixel clock, but the data lanes change at seven times the pixel
> clock rate, at least for the proposed formats:
>
> time ------------>
>
> RGB666_LVDS_SPWG:
> CLK -----.________.-----
> DATA0 G0 R5 R4 R3 R2 R1 R0
> DATA1 B1 B0 G5 G4 G3 G2 G1
> DATA2 x x x B5 B4 B3 B2
>
> RGB888_LVDS_SPWG:
> CLK -----.________.-----
> DATA0 G0 R5 R4 R3 R2 R1 R0
> DATA1 B1 B0 G5 G4 G3 G2 G1
> DATA2 x x x B5 B4 B3 B2
> DATA3 x B7 B6 G7 G6 R7 R6
>
> RGB888_LVDS_JEIDA:
> CLK -----.________.-----
> DATA0 G2 R7 R6 R5 R4 R3 R2
> DATA1 B3 B2 G7 G6 G5 G4 G3
> DATA2 x x x B7 B6 B5 B4
> DATA3 x B1 B0 G1 G0 R1 R0
> We could describe this as "RGB888_LVDS_1X7X4_%s", but what should "%s"
> be to differentiate between the MSB on DATA3 (SPWG) and LSB on DATA3
> (JEIDA) variants?
> Also, I think it would helpful to at least have a comment which format
> definitions correspond to the SPWG/JEIDA ordering, because that's what
> you often find in LVDS panel data sheets. I'd prefer to just keep them.
I got confused by the ordering actually. Could you use the same as the other
definitions?
That arrangement is quite special indeed. I guess it then makes sense indeed
to keep the name of the standard there to differentiate them from more
conventional bit order. However it the same bit order with this format would
be used elsewhere, then this same format definition would be valid.
I'd still drop LVDS from the name since it makes no difference from the
user's point of view whether the data is actually transferred using a low
voltage differential signal or not. :-)
> > I think this falls to the domain of the parallel-like interfaces since it
> > defines per bus pin which data is transferred over which pin.
>
> That might be, but as you can see above the ordering is a bit special.
>
> > Is the wire order defined by the above standards as well?
>
> Yes.
>
> > It is exactly the opposite to what is currently defined there.
>
> Note that the LVDS table has the wire pairs in rows, the columns
> represent the seven time slots, because I thought a four-wide table with
> seven rows per entry looked a bit strange, and that's the order these
> formats are commonly depicted.
--
Kind regards,
Sakari Ailus
e-mail: sakari.ailus@iki.fi XMPP: sailus@retiisi.org.uk
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Add LVDS RGB media bus formats
2014-12-12 23:07 ` Sakari Ailus
@ 2014-12-15 9:18 ` Philipp Zabel
0 siblings, 0 replies; 7+ messages in thread
From: Philipp Zabel @ 2014-12-15 9:18 UTC (permalink / raw)
To: Sakari Ailus
Cc: Mauro Carvalho Chehab, Hans Verkuil, Laurent Pinchart,
Boris Brezillon, linux-media, kernel
Hi Sakari,
Am Samstag, den 13.12.2014, 01:07 +0200 schrieb Sakari Ailus:
[...]
> > diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml
> > index 0d6f731..6d59a0e 100644
> > --- a/Documentation/DocBook/media/v4l/subdev-formats.xml
> > +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
> > @@ -89,6 +89,14 @@
> > <constant>MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE</constant>.
> > </para>
> >
> > + <para>On LVDS buses, usually each sample is transferred serialized in seven
>
> 80 characters per line, please.
>
> Could you move this paragraph just before the LVDS table?
Will do.
[...]
> > diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h
> > index 37091c6..3fb9cbb 100644
> > --- a/include/uapi/linux/media-bus-format.h
> > +++ b/include/uapi/linux/media-bus-format.h
> > @@ -33,7 +33,7 @@
> >
> > #define MEDIA_BUS_FMT_FIXED 0x0001
> >
> > -/* RGB - next is 0x1010 */
>
> Does your patch depend on another patch which is not merged yet?
Yes, it depends on Boris Brezillon's patch to
"Add RGB444_1X12 and RGB565_1X16 media bus formats":
https://lkml.org/lkml/2014/11/16/12
thanks
Philipp
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Add LVDS RGB media bus formats
2014-12-12 16:51 Philipp Zabel
@ 2014-12-12 23:07 ` Sakari Ailus
2014-12-15 9:18 ` Philipp Zabel
0 siblings, 1 reply; 7+ messages in thread
From: Sakari Ailus @ 2014-12-12 23:07 UTC (permalink / raw)
To: Philipp Zabel
Cc: Mauro Carvalho Chehab, Hans Verkuil, Laurent Pinchart,
Boris Brezillon, linux-media, kernel
Hi Pilipp,
On Fri, Dec 12, 2014 at 05:51:02PM +0100, Philipp Zabel wrote:
> This patch adds three new RGB media bus formats that describe
> 18-bit or 24-bit samples transferred over an LVDS bus with three
> or four differential data pairs, serialized into 7 time slots,
> using standard SPWG/PSWG/VESA or JEIDA data ordering.
>
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
> Documentation/DocBook/media/v4l/subdev-formats.xml | 253 +++++++++++++++++++++
> include/uapi/linux/media-bus-format.h | 5 +-
> 2 files changed, 257 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml
> index 0d6f731..6d59a0e 100644
> --- a/Documentation/DocBook/media/v4l/subdev-formats.xml
> +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
> @@ -89,6 +89,14 @@
> <constant>MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE</constant>.
> </para>
>
> + <para>On LVDS buses, usually each sample is transferred serialized in seven
80 characters per line, please.
Could you move this paragraph just before the LVDS table?
> + time slots per pixel clock, on three (18-bit) or four (24-bit) differential
> + data pairs at the same time. The remaining bits are used for control signals
> + as defined by SPWG/PSWG/VESA or JEIDA standards. The 24-bit RGB format serialized
> + in seven time slots on four lanes using JEIDA defined bit mapping will be
> + named <constant>MEDIA_BUS_FMT_RGB888_1X7X3_JEIDA</constant>, for example.
> + </para>
> +
> <para>The following tables list existing packed RGB formats.</para>
>
> <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb">
> @@ -606,6 +614,251 @@
> </tbody>
> </tgroup>
> </table>
> + <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb-lvds">
> + <title>LVDS RGB formats</title>
> + <tgroup cols="8">
> + <colspec colname="id" align="left" />
> + <colspec colname="code" align="center" />
> + <colspec colname="slot" align="center" />
> + <colspec colname="lane" />
> + <colspec colnum="5" colname="l03" align="center" />
> + <colspec colnum="6" colname="l02" align="center" />
> + <colspec colnum="7" colname="l01" align="center" />
> + <colspec colnum="8" colname="l00" align="center" />
> + <spanspec namest="l03" nameend="l00" spanname="l0" />
> + <thead>
> + <row>
> + <entry>Identifier</entry>
> + <entry>Code</entry>
> + <entry></entry>
> + <entry></entry>
> + <entry spanname="l0">Data organization</entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>Timeslot</entry>
> + <entry>Lane</entry>
> + <entry>3</entry>
> + <entry>2</entry>
> + <entry>1</entry>
> + <entry>0</entry>
> + </row>
> + </thead>
> + <tbody valign="top">
> + <row id="MEDIA-BUS-FMT-RGB666-1X7X3-SPWG">
> + <entry>MEDIA_BUS_FMT_RGB666_1X7X3_SPWG</entry>
> + <entry>0x1010</entry>
> + <entry>0</entry>
> + <entry></entry>
> + <entry>-</entry>
> + <entry>d</entry>
> + <entry>b<subscript>1</subscript></entry>
> + <entry>g<subscript>0</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>1</entry>
> + <entry></entry>
> + <entry>-</entry>
> + <entry>d</entry>
> + <entry>b<subscript>0</subscript></entry>
> + <entry>r<subscript>5</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>2</entry>
> + <entry></entry>
> + <entry>-</entry>
> + <entry>d</entry>
> + <entry>g<subscript>5</subscript></entry>
> + <entry>r<subscript>4</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>3</entry>
> + <entry></entry>
> + <entry>-</entry>
> + <entry>b<subscript>5</subscript></entry>
> + <entry>g<subscript>4</subscript></entry>
> + <entry>r<subscript>3</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>4</entry>
> + <entry></entry>
> + <entry>-</entry>
> + <entry>b<subscript>4</subscript></entry>
> + <entry>g<subscript>3</subscript></entry>
> + <entry>r<subscript>2</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>5</entry>
> + <entry></entry>
> + <entry>-</entry>
> + <entry>b<subscript>3</subscript></entry>
> + <entry>g<subscript>2</subscript></entry>
> + <entry>r<subscript>1</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>6</entry>
> + <entry></entry>
> + <entry>-</entry>
> + <entry>b<subscript>2</subscript></entry>
> + <entry>g<subscript>1</subscript></entry>
> + <entry>r<subscript>0</subscript></entry>
> + </row>
> + <row id="MEDIA-BUS-FMT-RGB888-1X7X4-SPWG">
> + <entry>MEDIA_BUS_FMT_RGB888_1X7X4_SPWG</entry>
> + <entry>0x1011</entry>
> + <entry>0</entry>
> + <entry></entry>
> + <entry>d</entry>
> + <entry>d</entry>
> + <entry>b<subscript>1</subscript></entry>
> + <entry>g<subscript>0</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>1</entry>
> + <entry></entry>
> + <entry>b<subscript>7</subscript></entry>
> + <entry>d</entry>
> + <entry>b<subscript>0</subscript></entry>
> + <entry>r<subscript>5</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>2</entry>
> + <entry></entry>
> + <entry>b<subscript>6</subscript></entry>
> + <entry>d</entry>
> + <entry>g<subscript>5</subscript></entry>
> + <entry>r<subscript>4</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>3</entry>
> + <entry></entry>
> + <entry>g<subscript>7</subscript></entry>
> + <entry>b<subscript>5</subscript></entry>
> + <entry>g<subscript>4</subscript></entry>
> + <entry>r<subscript>3</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>4</entry>
> + <entry></entry>
> + <entry>g<subscript>6</subscript></entry>
> + <entry>b<subscript>4</subscript></entry>
> + <entry>g<subscript>3</subscript></entry>
> + <entry>r<subscript>2</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>5</entry>
> + <entry></entry>
> + <entry>r<subscript>7</subscript></entry>
> + <entry>b<subscript>3</subscript></entry>
> + <entry>g<subscript>2</subscript></entry>
> + <entry>r<subscript>1</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>6</entry>
> + <entry></entry>
> + <entry>r<subscript>6</subscript></entry>
> + <entry>b<subscript>2</subscript></entry>
> + <entry>g<subscript>1</subscript></entry>
> + <entry>r<subscript>0</subscript></entry>
> + </row>
> + <row id="MEDIA-BUS-FMT-RGB888-1X7X4-JEIDA">
> + <entry>MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA</entry>
> + <entry>0x1012</entry>
> + <entry>0</entry>
> + <entry></entry>
> + <entry>d</entry>
> + <entry>d</entry>
> + <entry>b<subscript>3</subscript></entry>
> + <entry>g<subscript>2</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>1</entry>
> + <entry></entry>
> + <entry>b<subscript>1</subscript></entry>
> + <entry>d</entry>
> + <entry>b<subscript>2</subscript></entry>
> + <entry>r<subscript>7</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>2</entry>
> + <entry></entry>
> + <entry>b<subscript>0</subscript></entry>
> + <entry>d</entry>
> + <entry>g<subscript>7</subscript></entry>
> + <entry>r<subscript>6</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>3</entry>
> + <entry></entry>
> + <entry>g<subscript>1</subscript></entry>
> + <entry>b<subscript>7</subscript></entry>
> + <entry>g<subscript>6</subscript></entry>
> + <entry>r<subscript>5</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>4</entry>
> + <entry></entry>
> + <entry>g<subscript>0</subscript></entry>
> + <entry>b<subscript>6</subscript></entry>
> + <entry>g<subscript>5</subscript></entry>
> + <entry>r<subscript>4</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>5</entry>
> + <entry></entry>
> + <entry>r<subscript>1</subscript></entry>
> + <entry>b<subscript>5</subscript></entry>
> + <entry>g<subscript>4</subscript></entry>
> + <entry>r<subscript>3</subscript></entry>
> + </row>
> + <row>
> + <entry></entry>
> + <entry></entry>
> + <entry>6</entry>
> + <entry></entry>
> + <entry>r<subscript>0</subscript></entry>
> + <entry>b<subscript>4</subscript></entry>
> + <entry>g<subscript>3</subscript></entry>
> + <entry>r<subscript>2</subscript></entry>
> + </row>
> + </tbody>
> + </tgroup>
> + </table>
> </section>
>
> <section>
> diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h
> index 37091c6..3fb9cbb 100644
> --- a/include/uapi/linux/media-bus-format.h
> +++ b/include/uapi/linux/media-bus-format.h
> @@ -33,7 +33,7 @@
>
> #define MEDIA_BUS_FMT_FIXED 0x0001
>
> -/* RGB - next is 0x1010 */
Does your patch depend on another patch which is not merged yet?
> +/* RGB - next is 0x1013 */
> #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
> @@ -45,9 +45,12 @@
> #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_1X7X3_SPWG 0x1010
> #define MEDIA_BUS_FMT_RGB888_1X24 0x100a
> #define MEDIA_BUS_FMT_RGB888_2X12_BE 0x100b
> #define MEDIA_BUS_FMT_RGB888_2X12_LE 0x100c
> +#define MEDIA_BUS_FMT_RGB888_1X7X4_SPWG 0x1011
> +#define MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA 0x1012
> #define MEDIA_BUS_FMT_ARGB8888_1X32 0x100d
>
> /* YUV (including grey) - next is 0x2024 */
--
Kind regards,
Sakari Ailus
e-mail: sakari.ailus@iki.fi XMPP: sailus@retiisi.org.uk
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] Add LVDS RGB media bus formats
@ 2014-12-12 16:51 Philipp Zabel
2014-12-12 23:07 ` Sakari Ailus
0 siblings, 1 reply; 7+ messages in thread
From: Philipp Zabel @ 2014-12-12 16:51 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Hans Verkuil, Sakari Ailus, Laurent Pinchart, Boris Brezillon,
linux-media, kernel, Philipp Zabel
This patch adds three new RGB media bus formats that describe
18-bit or 24-bit samples transferred over an LVDS bus with three
or four differential data pairs, serialized into 7 time slots,
using standard SPWG/PSWG/VESA or JEIDA data ordering.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
Documentation/DocBook/media/v4l/subdev-formats.xml | 253 +++++++++++++++++++++
include/uapi/linux/media-bus-format.h | 5 +-
2 files changed, 257 insertions(+), 1 deletion(-)
diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml
index 0d6f731..6d59a0e 100644
--- a/Documentation/DocBook/media/v4l/subdev-formats.xml
+++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
@@ -89,6 +89,14 @@
<constant>MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE</constant>.
</para>
+ <para>On LVDS buses, usually each sample is transferred serialized in seven
+ time slots per pixel clock, on three (18-bit) or four (24-bit) differential
+ data pairs at the same time. The remaining bits are used for control signals
+ as defined by SPWG/PSWG/VESA or JEIDA standards. The 24-bit RGB format serialized
+ in seven time slots on four lanes using JEIDA defined bit mapping will be
+ named <constant>MEDIA_BUS_FMT_RGB888_1X7X3_JEIDA</constant>, for example.
+ </para>
+
<para>The following tables list existing packed RGB formats.</para>
<table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb">
@@ -606,6 +614,251 @@
</tbody>
</tgroup>
</table>
+ <table pgwide="0" frame="none" id="v4l2-mbus-pixelcode-rgb-lvds">
+ <title>LVDS RGB formats</title>
+ <tgroup cols="8">
+ <colspec colname="id" align="left" />
+ <colspec colname="code" align="center" />
+ <colspec colname="slot" align="center" />
+ <colspec colname="lane" />
+ <colspec colnum="5" colname="l03" align="center" />
+ <colspec colnum="6" colname="l02" align="center" />
+ <colspec colnum="7" colname="l01" align="center" />
+ <colspec colnum="8" colname="l00" align="center" />
+ <spanspec namest="l03" nameend="l00" spanname="l0" />
+ <thead>
+ <row>
+ <entry>Identifier</entry>
+ <entry>Code</entry>
+ <entry></entry>
+ <entry></entry>
+ <entry spanname="l0">Data organization</entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>Timeslot</entry>
+ <entry>Lane</entry>
+ <entry>3</entry>
+ <entry>2</entry>
+ <entry>1</entry>
+ <entry>0</entry>
+ </row>
+ </thead>
+ <tbody valign="top">
+ <row id="MEDIA-BUS-FMT-RGB666-1X7X3-SPWG">
+ <entry>MEDIA_BUS_FMT_RGB666_1X7X3_SPWG</entry>
+ <entry>0x1010</entry>
+ <entry>0</entry>
+ <entry></entry>
+ <entry>-</entry>
+ <entry>d</entry>
+ <entry>b<subscript>1</subscript></entry>
+ <entry>g<subscript>0</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>1</entry>
+ <entry></entry>
+ <entry>-</entry>
+ <entry>d</entry>
+ <entry>b<subscript>0</subscript></entry>
+ <entry>r<subscript>5</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>2</entry>
+ <entry></entry>
+ <entry>-</entry>
+ <entry>d</entry>
+ <entry>g<subscript>5</subscript></entry>
+ <entry>r<subscript>4</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>3</entry>
+ <entry></entry>
+ <entry>-</entry>
+ <entry>b<subscript>5</subscript></entry>
+ <entry>g<subscript>4</subscript></entry>
+ <entry>r<subscript>3</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>4</entry>
+ <entry></entry>
+ <entry>-</entry>
+ <entry>b<subscript>4</subscript></entry>
+ <entry>g<subscript>3</subscript></entry>
+ <entry>r<subscript>2</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>5</entry>
+ <entry></entry>
+ <entry>-</entry>
+ <entry>b<subscript>3</subscript></entry>
+ <entry>g<subscript>2</subscript></entry>
+ <entry>r<subscript>1</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>6</entry>
+ <entry></entry>
+ <entry>-</entry>
+ <entry>b<subscript>2</subscript></entry>
+ <entry>g<subscript>1</subscript></entry>
+ <entry>r<subscript>0</subscript></entry>
+ </row>
+ <row id="MEDIA-BUS-FMT-RGB888-1X7X4-SPWG">
+ <entry>MEDIA_BUS_FMT_RGB888_1X7X4_SPWG</entry>
+ <entry>0x1011</entry>
+ <entry>0</entry>
+ <entry></entry>
+ <entry>d</entry>
+ <entry>d</entry>
+ <entry>b<subscript>1</subscript></entry>
+ <entry>g<subscript>0</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>1</entry>
+ <entry></entry>
+ <entry>b<subscript>7</subscript></entry>
+ <entry>d</entry>
+ <entry>b<subscript>0</subscript></entry>
+ <entry>r<subscript>5</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>2</entry>
+ <entry></entry>
+ <entry>b<subscript>6</subscript></entry>
+ <entry>d</entry>
+ <entry>g<subscript>5</subscript></entry>
+ <entry>r<subscript>4</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>3</entry>
+ <entry></entry>
+ <entry>g<subscript>7</subscript></entry>
+ <entry>b<subscript>5</subscript></entry>
+ <entry>g<subscript>4</subscript></entry>
+ <entry>r<subscript>3</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>4</entry>
+ <entry></entry>
+ <entry>g<subscript>6</subscript></entry>
+ <entry>b<subscript>4</subscript></entry>
+ <entry>g<subscript>3</subscript></entry>
+ <entry>r<subscript>2</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>5</entry>
+ <entry></entry>
+ <entry>r<subscript>7</subscript></entry>
+ <entry>b<subscript>3</subscript></entry>
+ <entry>g<subscript>2</subscript></entry>
+ <entry>r<subscript>1</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>6</entry>
+ <entry></entry>
+ <entry>r<subscript>6</subscript></entry>
+ <entry>b<subscript>2</subscript></entry>
+ <entry>g<subscript>1</subscript></entry>
+ <entry>r<subscript>0</subscript></entry>
+ </row>
+ <row id="MEDIA-BUS-FMT-RGB888-1X7X4-JEIDA">
+ <entry>MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA</entry>
+ <entry>0x1012</entry>
+ <entry>0</entry>
+ <entry></entry>
+ <entry>d</entry>
+ <entry>d</entry>
+ <entry>b<subscript>3</subscript></entry>
+ <entry>g<subscript>2</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>1</entry>
+ <entry></entry>
+ <entry>b<subscript>1</subscript></entry>
+ <entry>d</entry>
+ <entry>b<subscript>2</subscript></entry>
+ <entry>r<subscript>7</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>2</entry>
+ <entry></entry>
+ <entry>b<subscript>0</subscript></entry>
+ <entry>d</entry>
+ <entry>g<subscript>7</subscript></entry>
+ <entry>r<subscript>6</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>3</entry>
+ <entry></entry>
+ <entry>g<subscript>1</subscript></entry>
+ <entry>b<subscript>7</subscript></entry>
+ <entry>g<subscript>6</subscript></entry>
+ <entry>r<subscript>5</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>4</entry>
+ <entry></entry>
+ <entry>g<subscript>0</subscript></entry>
+ <entry>b<subscript>6</subscript></entry>
+ <entry>g<subscript>5</subscript></entry>
+ <entry>r<subscript>4</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>5</entry>
+ <entry></entry>
+ <entry>r<subscript>1</subscript></entry>
+ <entry>b<subscript>5</subscript></entry>
+ <entry>g<subscript>4</subscript></entry>
+ <entry>r<subscript>3</subscript></entry>
+ </row>
+ <row>
+ <entry></entry>
+ <entry></entry>
+ <entry>6</entry>
+ <entry></entry>
+ <entry>r<subscript>0</subscript></entry>
+ <entry>b<subscript>4</subscript></entry>
+ <entry>g<subscript>3</subscript></entry>
+ <entry>r<subscript>2</subscript></entry>
+ </row>
+ </tbody>
+ </tgroup>
+ </table>
</section>
<section>
diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h
index 37091c6..3fb9cbb 100644
--- a/include/uapi/linux/media-bus-format.h
+++ b/include/uapi/linux/media-bus-format.h
@@ -33,7 +33,7 @@
#define MEDIA_BUS_FMT_FIXED 0x0001
-/* RGB - next is 0x1010 */
+/* RGB - next is 0x1013 */
#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
@@ -45,9 +45,12 @@
#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_1X7X3_SPWG 0x1010
#define MEDIA_BUS_FMT_RGB888_1X24 0x100a
#define MEDIA_BUS_FMT_RGB888_2X12_BE 0x100b
#define MEDIA_BUS_FMT_RGB888_2X12_LE 0x100c
+#define MEDIA_BUS_FMT_RGB888_1X7X4_SPWG 0x1011
+#define MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA 0x1012
#define MEDIA_BUS_FMT_ARGB8888_1X32 0x100d
/* YUV (including grey) - next is 0x2024 */
--
2.1.3
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-12-15 9:18 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-02 19:41 [PATCH] Add LVDS RGB media bus formats Philipp Zabel
2014-12-03 15:47 ` Sakari Ailus
2014-12-04 10:44 ` Philipp Zabel
2014-12-08 0:59 ` Sakari Ailus
2014-12-12 16:51 Philipp Zabel
2014-12-12 23:07 ` Sakari Ailus
2014-12-15 9:18 ` Philipp Zabel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).