mirror of https://lore.kernel.org/linux-amlogic/
 help / color / mirror / Atom feed
From: hverkuil@xs4all.nl (Hans Verkuil)
To: linus-amlogic@lists.infradead.org
Subject: [PATCH v3 3/6] documentation: media: Add documentation for new RGB and YUV bus formats
Date: Mon, 20 Mar 2017 15:33:11 +0100	[thread overview]
Message-ID: <5b40bc25-da65-108f-9a7b-0a856cb89330@xs4all.nl> (raw)
In-Reply-To: <1488904944-14285-4-git-send-email-narmstrong@baylibre.com>

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@baylibre.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

  parent reply	other threads:[~2017-03-20 14:33 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-07 16:42 [PATCH v3 0/6] drm: bridge: dw-hdmi: Add support for Custom PHYs Neil Armstrong
2017-03-07 16:42 ` [PATCH v3 1/6] drm: bridge: dw-hdmi: Extract PHY interrupt setup to a function Neil Armstrong
2017-03-07 17:12   ` Jose Abreu
2017-03-08 10:25     ` Neil Armstrong
2017-03-07 16:42 ` [PATCH v3 2/6] media: uapi: Add RGB and YUV bus formats for Synopsys HDMI TX Controller Neil Armstrong
2017-03-16 17:06   ` Archit Taneja
2017-03-17 16:10     ` Neil Armstrong
2017-03-20 15:35   ` Hans Verkuil
2017-03-07 16:42 ` [PATCH v3 3/6] documentation: media: Add documentation for new RGB and YUV bus formats Neil Armstrong
2017-03-16 17:01   ` Archit Taneja
2017-03-17 16:11     ` Neil Armstrong
2017-03-20 15:04       ` Hans Verkuil
2017-03-20 14:33   ` Hans Verkuil [this message]
2017-03-07 16:42 ` [PATCH v3 4/6] drm: bridge: dw-hdmi: Switch to V4L bus format and encodings Neil Armstrong
2017-03-07 17:35   ` Jose Abreu
2017-03-08  9:35     ` Neil Armstrong
2017-03-08 12:12     ` Neil Armstrong
2017-03-09 14:27       ` Jose Abreu
2017-03-13 11:43         ` Jose Abreu
2017-03-14  8:14           ` Neil Armstrong
2017-03-07 16:42 ` [PATCH v3 5/6] drm: bridge: dw-hdmi: Add Documentation on supported input formats Neil Armstrong
2017-03-16 16:43   ` Archit Taneja
2017-03-07 16:42 ` [PATCH v3 6/6] drm: bridge: dw-hdmi: Move HPD handling to PHY operations Neil Armstrong
2017-03-07 17:19   ` Jose Abreu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5b40bc25-da65-108f-9a7b-0a856cb89330@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=linus-amlogic@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®