Re: [PATCH v3 0/9] Add MT8173 Video Decoder Driver - andrew-ct chen

From: andrew-ct chen <andrew-ct.chen@mediatek.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
	tiffany lin <tiffany.lin@mediatek.com>,
	<devicetree@vger.kernel.org>, <daniel.thompson@linaro.org>,
	<linux-mediatek@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>, Hans Verkuil <hansverk@cisco.com>,
	<PoChun.Lin@mediatek.com>, Rob Herring <robh+dt@kernel.org>,
	Hans Verkuil <hans.verkuil@cisco.com>,
	<linux-arm-kernel@lists.infradead.org>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	Yingjoe Chen <yingjoe.chen@mediatek.com>,
	Eddie Huang <eddie.huang@mediatek.com>,
	Pawel Osciak <posciak@chromium.org>,
	<linux-media@vger.kernel.org>
Subject: Re: [PATCH v3 0/9] Add MT8173 Video Decoder Driver
Date: Fri, 1 Jul 2016 19:53:15 +0800	[thread overview]
Message-ID: <1467373995.17297.2.camel@mtksdaap41> (raw)
In-Reply-To: <8a46c1e7-1f27-1e67-8c05-b133598b6a66@xs4all.nl>

On Fri, 2016-07-01 at 12:11 +0200, Hans Verkuil wrote:
> On 06/16/2016 12:54 PM, Mauro Carvalho Chehab wrote:
> > Em Tue, 14 Jun 2016 19:08:08 +0800
> > tiffany lin <tiffany.lin@mediatek.com> escreveu:
> > 
> >> Hi Mauro,
> >>
> >>
> >> On Wed, 2016-06-08 at 07:13 +0900, Hans Verkuil wrote:
> >>>
> >>> On 06/07/2016 11:22 PM, Mauro Carvalho Chehab wrote:  
> >>>> Em Mon, 30 May 2016 20:29:14 +0800
> >>>> Tiffany Lin <tiffany.lin@mediatek.com> escreveu:
> >>>>  
> >>>>> ==============
> >>>>>   Introduction
> >>>>> ==============
> >>>>>
> >>>>> The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
> >>>>> Mediatek Video Codec is able to handle video decoding of in a range of formats.
> >>>>>
> >>>>> This patch series add Mediatek block format V4L2_PIX_FMT_MT21, the decoder driver will decoded bitstream to
> >>>>> V4L2_PIX_FMT_MT21 format.
> >>>>>
> >>>>> This patch series rely on MTK VPU driver in patch series "Add MT8173 Video Encoder Driver and VPU Driver"[1]
> >>>>> and patch "CHROMIUM: v4l: Add V4L2_PIX_FMT_VP9 definition"[2] for VP9 support.
> >>>>> Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.
> >>>>>
> >>>>> Internally the driver uses videobuf2 framework and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.
> >>>>>
> >>>>> [1]https://patchwork.linuxtv.org/patch/33734/
> >>>>> [2]https://chromium-review.googlesource.com/#/c/245241/  
> >>>>
> >>>> Hmm... I'm not seeing the firmware for this driver at the
> >>>> linux-firmware tree:
> >>>> 	https://git.kernel.org/cgit/linux/kernel/git/firmware/linux-firmware.git/log/
> >>>>
> >>>> Nor I'm seeing any pull request for them. Did you send it?
> >>>> I'll only merge the driver upstream after seeing such pull request.  
> >>>   
> >> Sorry, I am not familiar with how to upstream firmware.
> >> Do you mean we need to upstream vpu firmware first before merge encoder
> >> driver upstream?
> > 
> > Please look at this page:
> > 	https://linuxtv.org/wiki/index.php/Development:_How_to_submit_patches#Firmware_submission
> > 
> > The information here can also be useful:
> > 	https://www.kernel.org/doc/readme/firmware-README.AddingFirmware
> > 
> > In summary, you need to provide redistribution rights for the
> > firmware blob. You can either submit it to me or directly to
> > linux-firmware. In the latter, please c/c me on such patch.
> 
> Tiffany, what is the status of the firmware submission?
> 
> Regards,
> 
> 	Hans

Hi Hans,
We are working on firmware test to make sure that both decoder and
encoder work well. Hopes it can be ready (firmware submission) on July 4
or July 5.

Thanks.
Andrew

> 
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek