From: Hans Verkuil <hverkuil@xs4all.nl>
To: Tiffany Lin <tiffany.lin@mediatek.com>,
Hans Verkuil <hans.verkuil@cisco.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
Daniel Kurtz <djkurtz@chromium.org>,
Pawel Osciak <posciak@chromium.org>
Cc: Eddie Huang <eddie.huang@mediatek.com>,
Yingjoe Chen <yingjoe.chen@mediatek.com>,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
linux-mediatek@lists.infradead.org
Subject: Re: [PATCH 2/4] docs-rst: Add compressed video formats used on MT8173 codec driver
Date: Wed, 7 Sep 2016 11:23:55 +0200 [thread overview]
Message-ID: <246f1aca-0b46-b2f1-edd0-158a2a07b1d9@xs4all.nl> (raw)
In-Reply-To: <1473231403-14900-3-git-send-email-tiffany.lin@mediatek.com>
On 09/07/16 08:56, Tiffany Lin wrote:
> Add V4L2_PIX_FMT_MT21C documentation
>
> Signed-off-by: Tiffany Lin <tiffany.lin@mediatek.com>
> ---
> Documentation/media/uapi/v4l/pixfmt-reserved.rst | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/Documentation/media/uapi/v4l/pixfmt-reserved.rst b/Documentation/media/uapi/v4l/pixfmt-reserved.rst
> index 0dd2f7f..2e21fbc 100644
> --- a/Documentation/media/uapi/v4l/pixfmt-reserved.rst
> +++ b/Documentation/media/uapi/v4l/pixfmt-reserved.rst
> @@ -339,7 +339,13 @@ please make a proposal on the linux-media mailing list.
> array. Anything what's in between the UYVY lines is JPEG data and
> should be concatenated to form the JPEG stream.
>
> + - .. _V4L2-PIX-FMT-MT21C:
>
> + - ``V4L2_PIX_FMT_MT21C``
> +
> + - 'MT21C'
> +
> + - Compressed two-planar YVU420 format used by Mediatek MT8173.
This really needs to be expanded.
Ideally this should reference the precise specification of this format if
available.
It certainly should explain which HW blocks of the mediatek SoC use this
format, it should explain that is it meant as an opaque intermediate format
between those blocks.
If you have some characteristics (i.e. is it lossy or lossless
compression, I
presume it's lossless), then that will be useful to add as well.
We like to have as much information about formats as possible.
Regards,
Hans
>
> .. tabularcolumns:: |p{6.6cm}|p{2.2cm}|p{8.7cm}|
>
>
next prev parent reply other threads:[~2016-09-07 9:24 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-07 6:56 [PATCH 0/4] Add V4L2_PIX_FMT_MT21C format for MT8173 codec driver Tiffany Lin
2016-09-07 6:56 ` [PATCH 1/4] v4l: add Mediatek compressed video block format Tiffany Lin
2016-09-07 6:56 ` [PATCH 2/4] docs-rst: Add compressed video formats used on MT8173 codec driver Tiffany Lin
2016-09-07 6:56 ` [PATCH 3/4] vcodec: mediatek: Add V4L2_PIX_FMT_MT21C support for v4l2 decoder Tiffany Lin
2016-09-07 6:56 ` [PATCH 4/4] arm64: dts: mediatek: Add Video Decoder for MT8173 Tiffany Lin
2016-09-07 9:23 ` Hans Verkuil [this message]
2016-09-08 6:44 ` [PATCH 2/4] docs-rst: Add compressed video formats used on MT8173 codec driver Tiffany Lin
2016-09-08 7:21 ` [PATCH 0/4] Add V4L2_PIX_FMT_MT21C format for " Hans Verkuil
2016-09-08 9:11 ` Tiffany Lin
2016-09-08 9:27 ` Hans Verkuil
2016-09-14 4:52 ` Tiffany Lin
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=246f1aca-0b46-b2f1-edd0-158a2a07b1d9@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=djkurtz@chromium.org \
--cc=eddie.huang@mediatek.com \
--cc=hans.verkuil@cisco.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=matthias.bgg@gmail.com \
--cc=mchehab@osg.samsung.com \
--cc=posciak@chromium.org \
--cc=tiffany.lin@mediatek.com \
--cc=yingjoe.chen@mediatek.com \
/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;
as well as URLs for NNTP newsgroup(s).