From: Nicolas Dufresne <nicolas.dufresne@collabora.com>
To: "Yunfei Dong (董云飞)" <Yunfei.Dong@mediatek.com>,
"nhebert@chromium.org" <nhebert@chromium.org>,
"benjamin.gaignard@collabora.com"
<benjamin.gaignard@collabora.com>,
"nfraprado@collabora.com" <nfraprado@collabora.com>,
"angelogioacchino.delregno@collabora.com"
<angelogioacchino.delregno@collabora.com>,
"hverkuil-cisco@xs4all.nl" <hverkuil-cisco@xs4all.nl>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-mediatek@lists.infradead.org"
<linux-mediatek@lists.infradead.org>,
"frkoenig@chromium.org" <frkoenig@chromium.org>,
"stevecho@chromium.org" <stevecho@chromium.org>,
"wenst@chromium.org" <wenst@chromium.org>,
"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"daniel@ffwll.ch" <daniel@ffwll.ch>,
"Mingjia Zhang (张明佳)" <Mingjia.Zhang@mediatek.com>,
Project_Global_Chrome_Upstream_Group
<Project_Global_Chrome_Upstream_Group@mediatek.com>,
"hsinyi@chromium.org" <hsinyi@chromium.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 0/3] media: mediatek: vcodec: Add driver to support 10bit
Date: Thu, 13 Jul 2023 09:31:27 -0400 [thread overview]
Message-ID: <e10baa4d1b7b847b474a0836ca34b9b47098c8f1.camel@collabora.com> (raw)
In-Reply-To: <aa042d72211331e2752f6dfd552911a1315a3324.camel@mediatek.com>
Le mercredi 12 juillet 2023 à 03:11 +0000, Yunfei Dong (董云飞) a écrit :
> > This cover letter is missing v4l2-compliance report. This is needed
> > whenever
> > format mechanism is modified in a driver (not just for new drivers).
> > Please add
> > and make sure there is not regression too.
> >
> Forgot to test v4l2-compliance, will add in next patch.
>
> The flush test not include H264 for test fail, according to mingjia's
> information, maybe gstreamer set wrong format. H264 fluster can test
> pass when kernel driver force h264 in raster mode.
For everyone interested, I've rebased MT2110R and MT2110T support MR in
GStreamer. With the offending "return -EINVAL" removed, the test passes for me.
Consider updating your GStreamer build, let me know if you see further issue
with GStreamer in this regard. I'm only missing a firmware with HEVC and 10bit
combined to confirm HEVC works too.
https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3444
cheer,
Nicolas
next prev parent reply other threads:[~2023-07-13 13:31 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-07-11 12:57 [PATCH 0/3] media: mediatek: vcodec: Add driver to support 10bit Yunfei Dong
2023-07-11 12:57 ` [PATCH 1/3] media: mediatek: vcodec: Add capture format to support 10bit tile mode Yunfei Dong
2023-07-11 20:16 ` Nicolas Dufresne
2023-07-12 3:12 ` Yunfei Dong (董云飞)
2023-07-11 12:57 ` [PATCH 2/3] media: mediatek: vcodec: Add capture format to support 10bit raster mode Yunfei Dong
2023-07-11 12:57 ` [PATCH 3/3] media: mediatek: vcodec: Add driver to support 10bit Yunfei Dong
2023-07-11 16:53 ` Nicolas Dufresne
2023-07-12 3:32 ` Yunfei Dong (董云飞)
2023-07-11 20:12 ` Nicolas Dufresne
2023-07-12 3:02 ` Yunfei Dong (董云飞)
2023-07-11 19:15 ` [PATCH 0/3] " Nicolas Dufresne
2023-07-11 19:39 ` Nicolas Dufresne
2023-07-11 19:40 ` Nícolas F. R. A. Prado
2023-07-11 20:10 ` Nicolas Dufresne
2023-07-12 3:11 ` Yunfei Dong (董云飞)
2023-07-13 13:31 ` Nicolas Dufresne [this message]
2023-07-11 20:05 ` Nicolas Dufresne
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=e10baa4d1b7b847b474a0836ca34b9b47098c8f1.camel@collabora.com \
--to=nicolas.dufresne@collabora.com \
--cc=Mingjia.Zhang@mediatek.com \
--cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
--cc=Yunfei.Dong@mediatek.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=benjamin.gaignard@collabora.com \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=frkoenig@chromium.org \
--cc=hsinyi@chromium.org \
--cc=hverkuil-cisco@xs4all.nl \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=nfraprado@collabora.com \
--cc=nhebert@chromium.org \
--cc=stevecho@chromium.org \
--cc=wenst@chromium.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;
as well as URLs for NNTP newsgroup(s).