From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751928AbcFNIo7 (ORCPT ); Tue, 14 Jun 2016 04:44:59 -0400 Received: from mail-vk0-f43.google.com ([209.85.213.43]:33319 "EHLO mail-vk0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751153AbcFNIo5 (ORCPT ); Tue, 14 Jun 2016 04:44:57 -0400 MIME-Version: 1.0 In-Reply-To: <575746EE.3030706@cisco.com> References: <1464611363-14936-1-git-send-email-tiffany.lin@mediatek.com> <20160607112235.475c2e4c@recife.lan> <575746EE.3030706@cisco.com> From: =?UTF-8?B?V3UtQ2hlbmcgTGkgKOadjuWLmeiqoCk=?= Date: Tue, 14 Jun 2016 16:44:36 +0800 X-Google-Sender-Auth: DlA9po3GjWN9NF39be5d_cvPnWU Message-ID: Subject: Re: [PATCH v3 0/9] Add MT8173 Video Decoder Driver To: Hans Verkuil Cc: Mauro Carvalho Chehab , Tiffany Lin , Hans Verkuil , Daniel Thompson , Rob Herring , Matthias Brugger , Daniel Kurtz , Pawel Osciak , Eddie Huang , Yingjoe Chen , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org, linux-mediatek@lists.infradead.org, Lin PoChun Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 8, 2016 at 6:13 AM, 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 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/ Tiffany. Can you check the license and add the firmware to linux-firmware? For the information, both encoder and decoder drivers require the firmware to work. >> >> 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. > > > Mauro, are you confusing the decoder and encoder driver? I haven't > thoroughly reviewed the decoder driver > yet, so there is no pull request for the decoder driver. > > The only pull request I made was for the encoder driver. > > Regards, > > Hans > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?V3UtQ2hlbmcgTGkgKOadjuWLmeiqoCk=?= Subject: Re: [PATCH v3 0/9] Add MT8173 Video Decoder Driver Date: Tue, 14 Jun 2016 16:44:36 +0800 Message-ID: References: <1464611363-14936-1-git-send-email-tiffany.lin@mediatek.com> <20160607112235.475c2e4c@recife.lan> <575746EE.3030706@cisco.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <575746EE.3030706-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Hans Verkuil Cc: Mauro Carvalho Chehab , Tiffany Lin , Hans Verkuil , Daniel Thompson , Rob Herring , Matthias Brugger , Daniel Kurtz , Pawel Osciak , Eddie Huang , Yingjoe Chen , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Lin PoChun List-Id: devicetree@vger.kernel.org On Wed, Jun 8, 2016 at 6:13 AM, 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 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/ Tiffany. Can you check the license and add the firmware to linux-firmware? For the information, both encoder and decoder drivers require the firmware to work. >> >> 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. > > > Mauro, are you confusing the decoder and encoder driver? I haven't > thoroughly reviewed the decoder driver > yet, so there is no pull request for the decoder driver. > > The only pull request I made was for the encoder driver. > > Regards, > > Hans > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: wuchengli@chromium.org (=?UTF-8?B?V3UtQ2hlbmcgTGkgKOadjuWLmeiqoCk=?=) Date: Tue, 14 Jun 2016 16:44:36 +0800 Subject: [PATCH v3 0/9] Add MT8173 Video Decoder Driver In-Reply-To: <575746EE.3030706@cisco.com> References: <1464611363-14936-1-git-send-email-tiffany.lin@mediatek.com> <20160607112235.475c2e4c@recife.lan> <575746EE.3030706@cisco.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Jun 8, 2016 at 6:13 AM, 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 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/ Tiffany. Can you check the license and add the firmware to linux-firmware? For the information, both encoder and decoder drivers require the firmware to work. >> >> 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. > > > Mauro, are you confusing the decoder and encoder driver? I haven't > thoroughly reviewed the decoder driver > yet, so there is no pull request for the decoder driver. > > The only pull request I made was for the encoder driver. > > Regards, > > Hans > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html