[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5b40bc25-da65-108f-9a7b-0a856cb89330@xs4all.nl>
Date: Mon, 20 Mar 2017 15:33:11 +0100
From: Hans Verkuil <hverkuil@...all.nl>
To: Neil Armstrong <narmstrong@...libre.com>,
dri-devel@...ts.freedesktop.org,
laurent.pinchart+renesas@...asonboard.com, architt@...eaurora.org,
mchehab@...nel.org
Cc: Jose.Abreu@...opsys.com, kieran.bingham@...asonboard.com,
linux-amlogic@...ts.infradead.org, linux-kernel@...r.kernel.org,
linux-doc@...r.kernel.org, linux-media@...r.kernel.org,
hans.verkuil@...co.com, sakari.ailus@...ux.intel.com
Subject: Re: [PATCH v3 3/6] documentation: media: Add documentation for new
RGB and YUV bus formats
On 03/07/2017 05:42 PM, Neil Armstrong wrote:
> Add documentation for added Bus Formats to describe RGB and YUS formats used
> as input to the Synopsys DesignWare HDMI TX Controller.
>
> Signed-off-by: Neil Armstrong <narmstrong@...libre.com>
> ---
> Documentation/media/uapi/v4l/subdev-formats.rst | 4992 ++++++++++++++++++-----
> 1 file changed, 3963 insertions(+), 1029 deletions(-)
>
> diff --git a/Documentation/media/uapi/v4l/subdev-formats.rst b/Documentation/media/uapi/v4l/subdev-formats.rst
> index d6152c9..feb55b5 100644
> --- a/Documentation/media/uapi/v4l/subdev-formats.rst
> +++ b/Documentation/media/uapi/v4l/subdev-formats.rst
> @@ -114,7 +114,7 @@ The following tables list existing packed RGB formats.
> .. it switches to long table, and there's no way to override it.
>
>
> -.. tabularcolumns:: |p{4.0cm}|p{0.7cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|
> +.. tabularcolumns:: |p{4.0cm}|p{0.7cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|p{0.22cm}|
>
> .. _v4l2-mbus-pixelcode-rgb:
>
<snip>
> + * .. _MEDIA-BUS-FMT-RGB161616-1X48:
> +
> + - MEDIA_BUS_FMT_RGB161616_1X48
> + - 0x10
0x10 -> 0x101a
> + -
> + - r\ :sub:`15`
<snip>
> + * .. _MEDIA-BUS-FMT-YUV16-1X48:
> +
> + - MEDIA_BUS_FMT_YUV16_1X48
> + - 0x202a
Needs an extra line:
-
The first cell is the 'Bit' column, which should be an empty cell.
If you look at the output, then you'll see that without this the bit 0 cell is empty.
Same for the other three 48 bit YUV formats.
> + - y\ :sub:`15`
> + - y\ :sub:`14`
> + - y\ :sub:`13`
> + - y\ :sub:`12`
> - y\ :sub:`11`
> - y\ :sub:`10`
> - - y\ :sub:`9`
> + - y\ :sub:`8`
Typo: should remain 9.
> - y\ :sub:`8`
> - y\ :sub:`7`
> - y\ :sub:`6`
> @@ -6203,6 +9124,26 @@ the following codes.
> - y\ :sub:`2`
> - y\ :sub:`1`
> - y\ :sub:`0`
> + - u\ :sub:`15`
> + - u\ :sub:`14`
> + - u\ :sub:`13`
> + - u\ :sub:`12`
> + - u\ :sub:`11`
> + - u\ :sub:`10`
> + - u\ :sub:`9`
> + - u\ :sub:`8`
> + - u\ :sub:`7`
> + - u\ :sub:`6`
> + - u\ :sub:`5`
> + - u\ :sub:`4`
> + - u\ :sub:`3`
> + - u\ :sub:`2`
> + - u\ :sub:`1`
> + - u\ :sub:`0`
> + - v\ :sub:`15`
> + - v\ :sub:`14`
> + - v\ :sub:`13`
> + - v\ :sub:`12`
> - v\ :sub:`11`
> - v\ :sub:`10`
> - v\ :sub:`9`
> @@ -6215,29 +9156,14 @@ the following codes.
> - v\ :sub:`2`
> - v\ :sub:`1`
> - v\ :sub:`0`
> - * -
> - -
> - -
> - -
> - -
> - -
> - -
> - -
> - -
> - -
> - -
> - - y\ :sub:`11`
> - - y\ :sub:`10`
> - - y\ :sub:`9`
> - - y\ :sub:`8`
> - - y\ :sub:`7`
> - - y\ :sub:`6`
> - - y\ :sub:`5`
> - - y\ :sub:`4`
> - - y\ :sub:`3`
> - - y\ :sub:`2`
> - - y\ :sub:`1`
> - - y\ :sub:`0`
> + * .. _MEDIA-BUS-FMT-UYVY16-1-1X48:
> +
> + - MEDIA_BUS_FMT_UYVY16_1_1X48
> + - 0x202b
> + - u\ :sub:`15`
> + - u\ :sub:`14`
> + - u\ :sub:`13`
> + - u\ :sub:`12`
> - u\ :sub:`11`
> - u\ :sub:`10`
> - u\ :sub:`9`
> @@ -6250,13 +9176,12 @@ the following codes.
> - u\ :sub:`2`
> - u\ :sub:`1`
> - u\ :sub:`0`
> - * .. _MEDIA-BUS-FMT-YUV10-1X30:
> -
> - - MEDIA_BUS_FMT_YUV10_1X30
> - - 0x2016
> - -
> - -
> - -
> + - y\ :sub:`15`
> + - y\ :sub:`14`
> + - y\ :sub:`13`
> + - y\ :sub:`12`
> + - y\ :sub:`11`
> + - y\ :sub:`10`
> - y\ :sub:`9`
> - y\ :sub:`8`
> - y\ :sub:`7`
> @@ -6267,16 +9192,30 @@ the following codes.
> - y\ :sub:`2`
> - y\ :sub:`1`
> - y\ :sub:`0`
> - - u\ :sub:`9`
> - - u\ :sub:`8`
> - - u\ :sub:`7`
> - - u\ :sub:`6`
> - - u\ :sub:`5`
> - - u\ :sub:`4`
> - - u\ :sub:`3`
> - - u\ :sub:`2`
> - - u\ :sub:`1`
> - - u\ :sub:`0`
> + - y\ :sub:`15`
> + - y\ :sub:`14`
> + - y\ :sub:`13`
> + - y\ :sub:`12`
> + - y\ :sub:`11`
> + - y\ :sub:`10`
> + - y\ :sub:`8`
8 -> 9
> + - y\ :sub:`8`
> + - y\ :sub:`7`
> + - y\ :sub:`6`
> + - y\ :sub:`5`
> + - y\ :sub:`4`
> + - y\ :sub:`3`
> + - y\ :sub:`2`
> + - y\ :sub:`1`
> + - y\ :sub:`0`
> + * -
> + -
> + - v\ :sub:`15`
> + - v\ :sub:`14`
> + - v\ :sub:`13`
> + - v\ :sub:`12`
> + - v\ :sub:`11`
> + - v\ :sub:`10`
> - v\ :sub:`9`
> - v\ :sub:`8`
> - v\ :sub:`7`
> @@ -6287,19 +9226,30 @@ the following codes.
> - v\ :sub:`2`
> - v\ :sub:`1`
> - v\ :sub:`0`
> - * .. _MEDIA-BUS-FMT-AYUV8-1X32:
> -
> - - MEDIA_BUS_FMT_AYUV8_1X32
> - - 0x2017
> - -
> - - a\ :sub:`7`
> - - a\ :sub:`6`
> - - a\ :sub:`5`
> - - a\ :sub:`4`
> - - a\ :sub:`3`
> - - a\ :sub:`2`
> - - a\ :sub:`1`
> - - a\ :sub:`0`
> + - y\ :sub:`15`
> + - y\ :sub:`14`
> + - y\ :sub:`13`
> + - y\ :sub:`12`
> + - y\ :sub:`11`
> + - y\ :sub:`10`
> + - y\ :sub:`9`
> + - y\ :sub:`8`
> + - y\ :sub:`7`
> + - y\ :sub:`6`
> + - y\ :sub:`5`
> + - y\ :sub:`4`
> + - y\ :sub:`3`
> + - y\ :sub:`2`
> + - y\ :sub:`1`
> + - y\ :sub:`0`
> + - y\ :sub:`15`
> + - y\ :sub:`14`
> + - y\ :sub:`13`
> + - y\ :sub:`12`
> + - y\ :sub:`11`
> + - y\ :sub:`10`
> + - y\ :sub:`8`
8 -> 9
> + - y\ :sub:`8`
> - y\ :sub:`7`
> - y\ :sub:`6`
> - y\ :sub:`5`
> @@ -6308,22 +9258,6 @@ the following codes.
> - y\ :sub:`2`
> - y\ :sub:`1`
> - y\ :sub:`0`
> - - u\ :sub:`7`
> - - u\ :sub:`6`
> - - u\ :sub:`5`
> - - u\ :sub:`4`
> - - u\ :sub:`3`
> - - u\ :sub:`2`
> - - u\ :sub:`1`
> - - u\ :sub:`0`
> - - v\ :sub:`7`
> - - v\ :sub:`6`
> - - v\ :sub:`5`
> - - v\ :sub:`4`
> - - v\ :sub:`3`
> - - v\ :sub:`2`
> - - v\ :sub:`1`
> - - v\ :sub:`0`
>
>
> .. raw:: latex
>
Regards,
Hans
Powered by blists - more mailing lists