Linux Media Controller development
 help / color / mirror / Atom feed
From: Tiffany Lin <tiffany.lin@mediatek.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH] pixfmt-reserved.rst: Improve MT21C documentation
Date: Mon, 19 Sep 2016 17:39:20 +0800	[thread overview]
Message-ID: <1474277960.25758.12.camel@mtksdaap41> (raw)
In-Reply-To: <82516dbf-d85e-ac69-0059-8235e4903e5a@xs4all.nl>

Hi Hans,

On Mon, 2016-09-19 at 09:22 +0200, Hans Verkuil wrote:
> Improve the MT21C documentation, making it clearer that this format requires the MDP
> for further processing.
> 
> Also fix the fourcc (it was a fivecc :-) )
> 
reviewed-by: Tiffany Lin <tiffany.lin@mediatek.com>

Thanks. I did not notice it become fivecc.

best regards,
Tiffany

> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
> ---
> diff --git a/Documentation/media/uapi/v4l/pixfmt-reserved.rst b/Documentation/media/uapi/v4l/pixfmt-reserved.rst
> index 0989e99..a019f15 100644
> --- a/Documentation/media/uapi/v4l/pixfmt-reserved.rst
> +++ b/Documentation/media/uapi/v4l/pixfmt-reserved.rst
> @@ -343,13 +343,13 @@ please make a proposal on the linux-media mailing list.
> 
>         -  ``V4L2_PIX_FMT_MT21C``
> 
> -       -  'MT21C'
> +       -  'MT21'
> 
>         -  Compressed two-planar YVU420 format used by Mediatek MT8173.
>            The compression is lossless.
> -          It is an opaque intermediate format, and MDP HW could convert
> -          V4L2_PIX_FMT_MT21C to V4L2_PIX_FMT_NV12M,
> -          V4L2_PIX_FMT_YUV420M and V4L2_PIX_FMT_YVU420.
> +          It is an opaque intermediate format and the MDP hardware must be
> +	  used to convert ``V4L2_PIX_FMT_MT21C`` to ``V4L2_PIX_FMT_NV12M``,
> +          ``V4L2_PIX_FMT_YUV420M`` or ``V4L2_PIX_FMT_YVU420``.
> 
>  .. tabularcolumns:: |p{6.6cm}|p{2.2cm}|p{8.7cm}|
> 



      reply	other threads:[~2016-09-19  9:39 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-19  7:22 [PATCH] pixfmt-reserved.rst: Improve MT21C documentation Hans Verkuil
2016-09-19  9:39 ` Tiffany Lin [this message]

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=1474277960.25758.12.camel@mtksdaap41 \
    --to=tiffany.lin@mediatek.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.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