From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [85.220.165.71]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BDC742C80 for ; Wed, 8 Dec 2021 15:39:41 +0000 (UTC) Received: from gallifrey.ext.pengutronix.de ([2001:67c:670:201:5054:ff:fe8d:eefb] helo=[IPv6:::1]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1muz2N-0000ft-SH; Wed, 08 Dec 2021 16:39:19 +0100 Message-ID: <7bbcee7d8d52cab00635037c35c81b43ce34ec76.camel@pengutronix.de> Subject: Re: [RFC V2 0/6] media: Hantro: Split iMX8MQ VPU into G1 and G2 with blk-ctrl support From: Lucas Stach To: Adam Ford , Benjamin Gaignard Cc: linux-media , Chris Healy , Hans Verkuil , Ezequiel Garcia , Philipp Zabel , Mauro Carvalho Chehab , Rob Herring , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Greg Kroah-Hartman , "open list:HANTRO VPU CODEC DRIVER" , devicetree , arm-soc , Linux Kernel Mailing List , "open list:STAGING SUBSYSTEM" Date: Wed, 08 Dec 2021 16:39:16 +0100 In-Reply-To: References: <20211207015446.1250854-1-aford173@gmail.com> <5a8b84e91bd8e7670a0d0108e4affe9b964202cb.camel@pengutronix.de> <4cc3404b-7f8a-6c87-8c1c-ace399f9edd3@collabora.com> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.40.4 (3.40.4-1.fc34) Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SA-Exim-Connect-IP: 2001:67c:670:201:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: l.stach@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-staging@lists.linux.dev Am Mittwoch, dem 08.12.2021 um 09:13 -0600 schrieb Adam Ford: > On Wed, Dec 8, 2021 at 7:36 AM Benjamin Gaignard > wrote: > > > > > > Le 08/12/2021 à 11:32, Lucas Stach a écrit : > > > Am Mittwoch, dem 08.12.2021 um 10:32 +0100 schrieb Benjamin Gaignard: > > > > Le 07/12/2021 à 02:54, Adam Ford a écrit : > > > > > > > > > Currently, the VPU in the i.MQ8MQ is appearing as one codec, but in > > > > > reality, it's two IP blocks called G1 and G2. There is initialization > > > > > code in VPU code to pull some clocks, resets and other features which > > > > > has been integrated into the vpu-blk-ctrl for the i.MX8M Mini and a > > > > > similar method can be used to make the VPU codec's operate as > > > > > stand-alone cores without having to know the details of each other > > > > > or the quirks unique to the i.MX8MQ, so the remaining code can be > > > > > left more generic. > > > > > > > > > > This series was started by Lucas Stach with one by Benjamin Gaignard. > > > > > Most patches have been modified slightly by me. It's in an RFC state > > > > > because I wasn't sure how to best handle the signatures and wasn't sure > > > > > if I could base it off the branch I did. > > > > > > > > > > Since the g-streamer and media trees are in a constant state of > > > > > change, this series is based on > > > > > > > > > > git://linuxtv.org/hverkuil/media_tree.git for-v5.17e > > > > > > > > > > The downstream code from NXP shows the G1 and G2 clocks running > > > > > at 600MHz, but between the TRM and the datasheet, there is some > > > > > discrepancy. Because the NXP reference code used 600MHz, that is > > > > > what was chosen here. Users who need to adjust their G1 and G2 > > > > > clocks can do so in their board files. > > > > Hi Adam, > > > > > > > > Thanks for your patches, I have been able to reproduce VP9 results on my side (Fluster 147/303). > > > > In past I have notice spurious errors when using 600MHz clock on HEVC decode but not with 300MHz. > > > > The results for Fluster HEVC are 77/147 so no regressions :-) > > > > Regards, > > Benjamin > > > > > > The max supported G2 clock frequency is 660MHz but needs a higher > > > > voltage. The maximum supported frequency at the default 0.9V is > > > > 550MHz. We should not configure the clocks for the higher than that, as > > > > long as there is no support in the VPU driver to scale the voltage > > > > along with the frequency. Same as with the GPU we should stick to base > > > > frequency levels for the nominal operating mode. > > Lucas, > > After reviewing the comments from Benjamin, I re-ran the VP9 tests > with the G2 running at 300MHz,and the number of passing VP9 tests > increased to 148 from 144 with an increase of time to 250.502 secs > from 162.665 secs. > > While the datasheet reads that the G2 can run faster, the i.MX 8M > Dual/8M QuadLite/8M Quad Applications Processors Reference Manual, > Rev. 3.1, 06/2021, table 5-1 shows the VPU_G2_CLK_ROOT has a max > clock of 300MHz. I might be inclined to agree with Benjamin on the > 300MHz and let people who want to push their hardware overwrite the > default clocks since it increases functionality. > I'm quite surprised that the G2 codec should have such a much lower max frequency compared with the G1. While the table from the RM hasn't been obviously incorrect for any other clock, I'm still inclined to believe the frequencies stated in the datasheet. > I wonder if someone from NXP can comment > Yea, not sure how to proceed here. 300MHz is the safe bet, but we are leaving quite some performance on the table if the datasheet is correct. Without some help from NXP it's probably hard to validate which max frequency statement is correct. It doesn't really help that the NXP downstream kernel seems to drive the G2 at 600MHz, which is neither the 550MHz nominal mode max, nor the 660MHz overdrive mode max. Regards, Lucas From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B98A6C433F5 for ; Wed, 8 Dec 2021 15:39:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Date:Cc:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=aJ7e/DbzR8jIu6UkzyIqyUl4Jwx90fgl4+Tqgwj+Km4=; b=veIhdOvrG7P49T ajNs3KPnTxmcFtfglXyjow+EZ046NtduUBBChnS15M7HGcxb1QbTG+5hCCCcW8c6ctyUlPLb902wh S+k+9fRZ3D02R9mojIyu6oCEMQzICl0x0IuAth0y4mtH2sbw8RzO0mU9E/rIKQBhQ5WaDkypPSX1s 0ZTmUbU0LCDO3VAqoedT8d9hcpWkUvsC28wxgLEDuVRzUDZdtrk2jDHEWVSKaDIVtDh3jvpHRwnUN PRuBzgkTEmTVmMnpAS2TzeRcQ9ifwKcaB2IGgmyP+1+0G0fk2U3rUmaZ0tTAoicmM0MBeFkeu5LPa hpXy7Y8HutIfPcba2ztg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1muz2d-00DK4E-C8; Wed, 08 Dec 2021 15:39:35 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1muz2X-00DK1S-AR for linux-rockchip@lists.infradead.org; Wed, 08 Dec 2021 15:39:30 +0000 Received: from gallifrey.ext.pengutronix.de ([2001:67c:670:201:5054:ff:fe8d:eefb] helo=[IPv6:::1]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1muz2N-0000ft-SH; Wed, 08 Dec 2021 16:39:19 +0100 Message-ID: <7bbcee7d8d52cab00635037c35c81b43ce34ec76.camel@pengutronix.de> Subject: Re: [RFC V2 0/6] media: Hantro: Split iMX8MQ VPU into G1 and G2 with blk-ctrl support From: Lucas Stach To: Adam Ford , Benjamin Gaignard Cc: linux-media , Chris Healy , Hans Verkuil , Ezequiel Garcia , Philipp Zabel , Mauro Carvalho Chehab , Rob Herring , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Greg Kroah-Hartman , "open list:HANTRO VPU CODEC DRIVER" , devicetree , arm-soc , Linux Kernel Mailing List , "open list:STAGING SUBSYSTEM" Date: Wed, 08 Dec 2021 16:39:16 +0100 In-Reply-To: References: <20211207015446.1250854-1-aford173@gmail.com> <5a8b84e91bd8e7670a0d0108e4affe9b964202cb.camel@pengutronix.de> <4cc3404b-7f8a-6c87-8c1c-ace399f9edd3@collabora.com> User-Agent: Evolution 3.40.4 (3.40.4-1.fc34) MIME-Version: 1.0 X-SA-Exim-Connect-IP: 2001:67c:670:201:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: l.stach@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-rockchip@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211208_073929_384251_4D67B0CE X-CRM114-Status: GOOD ( 43.77 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org QW0gTWl0dHdvY2gsIGRlbSAwOC4xMi4yMDIxIHVtIDA5OjEzIC0wNjAwIHNjaHJpZWIgQWRhbSBG b3JkOgo+IE9uIFdlZCwgRGVjIDgsIDIwMjEgYXQgNzozNiBBTSBCZW5qYW1pbiBHYWlnbmFyZAo+ IDxiZW5qYW1pbi5nYWlnbmFyZEBjb2xsYWJvcmEuY29tPiB3cm90ZToKPiA+IAo+ID4gCj4gPiBM ZSAwOC8xMi8yMDIxIMOgIDExOjMyLCBMdWNhcyBTdGFjaCBhIMOpY3JpdCA6Cj4gPiA+IEFtIE1p dHR3b2NoLCBkZW0gMDguMTIuMjAyMSB1bSAxMDozMiArMDEwMCBzY2hyaWViIEJlbmphbWluIEdh aWduYXJkOgo+ID4gPiA+IExlIDA3LzEyLzIwMjEgw6AgMDI6NTQsIEFkYW0gRm9yZCBhIMOpY3Jp dCA6Cj4gPiA+ID4gCj4gPiA+ID4gPiBDdXJyZW50bHksIHRoZSBWUFUgaW4gdGhlIGkuTVE4TVEg aXMgYXBwZWFyaW5nIGFzIG9uZSBjb2RlYywgYnV0IGluCj4gPiA+ID4gPiByZWFsaXR5LCBpdCdz IHR3byBJUCBibG9ja3MgY2FsbGVkIEcxIGFuZCBHMi4gIFRoZXJlIGlzIGluaXRpYWxpemF0aW9u Cj4gPiA+ID4gPiBjb2RlIGluIFZQVSBjb2RlIHRvIHB1bGwgc29tZSBjbG9ja3MsIHJlc2V0cyBh bmQgb3RoZXIgZmVhdHVyZXMgd2hpY2gKPiA+ID4gPiA+IGhhcyBiZWVuIGludGVncmF0ZWQgaW50 byB0aGUgdnB1LWJsay1jdHJsIGZvciB0aGUgaS5NWDhNIE1pbmkgYW5kIGEKPiA+ID4gPiA+IHNp bWlsYXIgbWV0aG9kIGNhbiBiZSB1c2VkIHRvIG1ha2UgdGhlIFZQVSBjb2RlYydzIG9wZXJhdGUg YXMKPiA+ID4gPiA+IHN0YW5kLWFsb25lIGNvcmVzIHdpdGhvdXQgaGF2aW5nIHRvIGtub3cgdGhl IGRldGFpbHMgb2YgZWFjaCBvdGhlcgo+ID4gPiA+ID4gb3IgdGhlIHF1aXJrcyB1bmlxdWUgdG8g dGhlIGkuTVg4TVEsIHNvIHRoZSByZW1haW5pbmcgY29kZSBjYW4gYmUKPiA+ID4gPiA+IGxlZnQg bW9yZSBnZW5lcmljLgo+ID4gPiA+ID4gCj4gPiA+ID4gPiBUaGlzIHNlcmllcyB3YXMgc3RhcnRl ZCBieSBMdWNhcyBTdGFjaCB3aXRoIG9uZSBieSBCZW5qYW1pbiBHYWlnbmFyZC4KPiA+ID4gPiA+ IE1vc3QgcGF0Y2hlcyBoYXZlIGJlZW4gbW9kaWZpZWQgc2xpZ2h0bHkgYnkgbWUuICBJdCdzIGlu IGFuIFJGQyBzdGF0ZQo+ID4gPiA+ID4gYmVjYXVzZSBJIHdhc24ndCBzdXJlIGhvdyB0byBiZXN0 IGhhbmRsZSB0aGUgc2lnbmF0dXJlcyBhbmQgd2Fzbid0IHN1cmUKPiA+ID4gPiA+IGlmIEkgY291 bGQgYmFzZSBpdCBvZmYgdGhlIGJyYW5jaCBJIGRpZC4KPiA+ID4gPiA+IAo+ID4gPiA+ID4gU2lu Y2UgdGhlIGctc3RyZWFtZXIgYW5kIG1lZGlhIHRyZWVzIGFyZSBpbiBhIGNvbnN0YW50IHN0YXRl IG9mCj4gPiA+ID4gPiBjaGFuZ2UsIHRoaXMgc2VyaWVzIGlzIGJhc2VkIG9uCj4gPiA+ID4gPiAK PiA+ID4gPiA+IGdpdDovL2xpbnV4dHYub3JnL2h2ZXJrdWlsL21lZGlhX3RyZWUuZ2l0IGZvci12 NS4xN2UKPiA+ID4gPiA+IAo+ID4gPiA+ID4gVGhlIGRvd25zdHJlYW0gY29kZSBmcm9tIE5YUCBz aG93cyB0aGUgRzEgYW5kIEcyIGNsb2NrcyBydW5uaW5nCj4gPiA+ID4gPiBhdCA2MDBNSHosIGJ1 dCBiZXR3ZWVuIHRoZSBUUk0gYW5kIHRoZSBkYXRhc2hlZXQsIHRoZXJlIGlzIHNvbWUKPiA+ID4g PiA+IGRpc2NyZXBhbmN5LiAgQmVjYXVzZSB0aGUgTlhQIHJlZmVyZW5jZSBjb2RlIHVzZWQgNjAw TUh6LCB0aGF0IGlzCj4gPiA+ID4gPiB3aGF0IHdhcyBjaG9zZW4gaGVyZS4gIFVzZXJzIHdobyBu ZWVkIHRvIGFkanVzdCB0aGVpciBHMSBhbmQgRzIKPiA+ID4gPiA+IGNsb2NrcyBjYW4gZG8gc28g aW4gdGhlaXIgYm9hcmQgZmlsZXMuCj4gPiA+ID4gSGkgQWRhbSwKPiA+ID4gPiAKPiA+ID4gPiBU aGFua3MgZm9yIHlvdXIgcGF0Y2hlcywgSSBoYXZlIGJlZW4gYWJsZSB0byByZXByb2R1Y2UgVlA5 IHJlc3VsdHMgb24gbXkgc2lkZSAoRmx1c3RlciAxNDcvMzAzKS4KPiA+ID4gPiBJbiBwYXN0IEkg aGF2ZSBub3RpY2Ugc3B1cmlvdXMgZXJyb3JzIHdoZW4gdXNpbmcgNjAwTUh6IGNsb2NrIG9uIEhF VkMgZGVjb2RlIGJ1dCBub3Qgd2l0aCAzMDBNSHouCj4gPiAKPiA+IFRoZSByZXN1bHRzIGZvciBG bHVzdGVyIEhFVkMgYXJlIDc3LzE0NyBzbyBubyByZWdyZXNzaW9ucyA6LSkKPiA+IAo+ID4gUmVn YXJkcywKPiA+IEJlbmphbWluCj4gPiAKPiA+ID4gPiBUaGUgbWF4IHN1cHBvcnRlZCBHMiBjbG9j ayBmcmVxdWVuY3kgaXMgNjYwTUh6IGJ1dCBuZWVkcyBhIGhpZ2hlcgo+ID4gPiA+IHZvbHRhZ2Uu IFRoZSBtYXhpbXVtIHN1cHBvcnRlZCAgZnJlcXVlbmN5IGF0IHRoZSBkZWZhdWx0IDAuOVYgaXMK PiA+ID4gPiA1NTBNSHouIFdlIHNob3VsZCBub3QgY29uZmlndXJlIHRoZSBjbG9ja3MgZm9yIHRo ZSBoaWdoZXIgdGhhbiB0aGF0LCBhcwo+ID4gPiA+IGxvbmcgYXMgdGhlcmUgaXMgbm8gc3VwcG9y dCBpbiB0aGUgVlBVIGRyaXZlciB0byBzY2FsZSB0aGUgdm9sdGFnZQo+ID4gPiA+IGFsb25nIHdp dGggdGhlIGZyZXF1ZW5jeS4gU2FtZSBhcyB3aXRoIHRoZSBHUFUgd2Ugc2hvdWxkIHN0aWNrIHRv IGJhc2UKPiA+ID4gPiBmcmVxdWVuY3kgbGV2ZWxzIGZvciB0aGUgbm9taW5hbCBvcGVyYXRpbmcg bW9kZS4KPiAKPiBMdWNhcywKPiAKPiBBZnRlciByZXZpZXdpbmcgdGhlIGNvbW1lbnRzIGZyb20g QmVuamFtaW4sIEkgcmUtcmFuIHRoZSBWUDkgdGVzdHMKPiB3aXRoIHRoZSBHMiBydW5uaW5nIGF0 IDMwME1IeixhbmQgdGhlIG51bWJlciBvZiBwYXNzaW5nIFZQOSB0ZXN0cwo+IGluY3JlYXNlZCB0 byAxNDggZnJvbSAxNDQgd2l0aCBhbiBpbmNyZWFzZSBvZiB0aW1lIHRvIDI1MC41MDIgc2Vjcwo+ IGZyb20gMTYyLjY2NSBzZWNzLgo+IAo+IFdoaWxlIHRoZSBkYXRhc2hlZXQgcmVhZHMgdGhhdCB0 aGUgRzIgY2FuIHJ1biBmYXN0ZXIsIHRoZSBpLk1YIDhNCj4gRHVhbC84TSBRdWFkTGl0ZS84TSBR dWFkIEFwcGxpY2F0aW9ucyBQcm9jZXNzb3JzIFJlZmVyZW5jZSBNYW51YWwsCj4gUmV2LiAzLjEs IDA2LzIwMjEsIHRhYmxlIDUtMSAgc2hvd3MgdGhlIFZQVV9HMl9DTEtfUk9PVCBoYXMgYSBtYXgK PiBjbG9jayBvZiAzMDBNSHouICBJIG1pZ2h0IGJlIGluY2xpbmVkIHRvIGFncmVlIHdpdGggQmVu amFtaW4gb24gdGhlCj4gMzAwTUh6IGFuZCBsZXQgcGVvcGxlIHdobyB3YW50IHRvIHB1c2ggdGhl aXIgaGFyZHdhcmUgb3ZlcndyaXRlIHRoZQo+IGRlZmF1bHQgY2xvY2tzIHNpbmNlIGl0IGluY3Jl YXNlcyBmdW5jdGlvbmFsaXR5Lgo+IApJJ20gcXVpdGUgc3VycHJpc2VkIHRoYXQgdGhlIEcyIGNv ZGVjIHNob3VsZCBoYXZlIHN1Y2ggYSBtdWNoIGxvd2VyIG1heApmcmVxdWVuY3kgY29tcGFyZWQg d2l0aCB0aGUgRzEuIFdoaWxlIHRoZSB0YWJsZSBmcm9tIHRoZSBSTSBoYXNuJ3QgYmVlbgpvYnZp b3VzbHkgaW5jb3JyZWN0IGZvciBhbnkgb3RoZXIgY2xvY2ssIEknbSBzdGlsbCBpbmNsaW5lZCB0 byBiZWxpZXZlCnRoZSBmcmVxdWVuY2llcyBzdGF0ZWQgaW4gdGhlIGRhdGFzaGVldC4KCj4gSSB3 b25kZXIgaWYgc29tZW9uZSBmcm9tIE5YUCBjYW4gY29tbWVudAo+IApZZWEsIG5vdCBzdXJlIGhv dyB0byBwcm9jZWVkIGhlcmUuIDMwME1IeiBpcyB0aGUgc2FmZSBiZXQsIGJ1dCB3ZSBhcmUKbGVh dmluZyBxdWl0ZSBzb21lIHBlcmZvcm1hbmNlIG9uIHRoZSB0YWJsZSBpZiB0aGUgZGF0YXNoZWV0 IGlzCmNvcnJlY3QuIFdpdGhvdXQgc29tZSBoZWxwIGZyb20gTlhQIGl0J3MgcHJvYmFibHkgaGFy ZCB0byB2YWxpZGF0ZSAKd2hpY2ggbWF4IGZyZXF1ZW5jeSBzdGF0ZW1lbnQgaXMgY29ycmVjdC4g SXQgZG9lc24ndCByZWFsbHkgaGVscCB0aGF0CnRoZSBOWFAgZG93bnN0cmVhbSBrZXJuZWwgc2Vl bXMgdG8gZHJpdmUgdGhlIEcyIGF0IDYwME1Ieiwgd2hpY2ggaXMKbmVpdGhlciB0aGUgNTUwTUh6 IG5vbWluYWwgbW9kZSBtYXgsIG5vciB0aGUgNjYwTUh6IG92ZXJkcml2ZSBtb2RlIG1heC4KClJl Z2FyZHMsCkx1Y2FzCgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KTGludXgtcm9ja2NoaXAgbWFpbGluZyBsaXN0CkxpbnV4LXJvY2tjaGlwQGxpc3RzLmlu ZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9s aW51eC1yb2NrY2hpcAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C31C9C433F5 for ; Wed, 8 Dec 2021 15:41:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Date:Cc:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=VOyWFn0Z9yzDQNxl6VSEJ2A5HrEWxKCRYrK83naxGQY=; b=BK6SU/HSvT9QFk lCbG6M6sqrChCBJMnjSInK/G6Tr2qEHRbopdSJqayfCx5BidQBWHM3WE3SHy45CHgBwU9SiOlK3Kr vkfqMk+GOeNmqdY4jPT9GaLSkGshHn9mQVhZ3fbiYKtNMTesDoUhg29OGqR3FVggQrvQlV1+IRKhm 8Z0ShIzNJVom4COLCdvL7BHLAk0xr+nEBfAznzDN4lq/DYOJenAOglWXiochBFK1f+KW8jS2OmtcV PjaXp4EtvCQmj9yEVJbpEFxcx+SzMkbsAMpe7jGfSXgFt5b715EAd8vRI3JYpRkJcPPnWMi6y0NRQ 3np0zpYAZElMZf86OveQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1muz2k-00DK5c-Ug; Wed, 08 Dec 2021 15:39:43 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1muz2Y-00DK2Z-WF for linux-arm-kernel@lists.infradead.org; Wed, 08 Dec 2021 15:39:32 +0000 Received: from gallifrey.ext.pengutronix.de ([2001:67c:670:201:5054:ff:fe8d:eefb] helo=[IPv6:::1]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1muz2N-0000ft-SH; Wed, 08 Dec 2021 16:39:19 +0100 Message-ID: <7bbcee7d8d52cab00635037c35c81b43ce34ec76.camel@pengutronix.de> Subject: Re: [RFC V2 0/6] media: Hantro: Split iMX8MQ VPU into G1 and G2 with blk-ctrl support From: Lucas Stach To: Adam Ford , Benjamin Gaignard Cc: linux-media , Chris Healy , Hans Verkuil , Ezequiel Garcia , Philipp Zabel , Mauro Carvalho Chehab , Rob Herring , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Greg Kroah-Hartman , "open list:HANTRO VPU CODEC DRIVER" , devicetree , arm-soc , Linux Kernel Mailing List , "open list:STAGING SUBSYSTEM" Date: Wed, 08 Dec 2021 16:39:16 +0100 In-Reply-To: References: <20211207015446.1250854-1-aford173@gmail.com> <5a8b84e91bd8e7670a0d0108e4affe9b964202cb.camel@pengutronix.de> <4cc3404b-7f8a-6c87-8c1c-ace399f9edd3@collabora.com> User-Agent: Evolution 3.40.4 (3.40.4-1.fc34) MIME-Version: 1.0 X-SA-Exim-Connect-IP: 2001:67c:670:201:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: l.stach@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-arm-kernel@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211208_073931_133208_A9C04A87 X-CRM114-Status: GOOD ( 44.66 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org QW0gTWl0dHdvY2gsIGRlbSAwOC4xMi4yMDIxIHVtIDA5OjEzIC0wNjAwIHNjaHJpZWIgQWRhbSBG b3JkOgo+IE9uIFdlZCwgRGVjIDgsIDIwMjEgYXQgNzozNiBBTSBCZW5qYW1pbiBHYWlnbmFyZAo+ IDxiZW5qYW1pbi5nYWlnbmFyZEBjb2xsYWJvcmEuY29tPiB3cm90ZToKPiA+IAo+ID4gCj4gPiBM ZSAwOC8xMi8yMDIxIMOgIDExOjMyLCBMdWNhcyBTdGFjaCBhIMOpY3JpdCA6Cj4gPiA+IEFtIE1p dHR3b2NoLCBkZW0gMDguMTIuMjAyMSB1bSAxMDozMiArMDEwMCBzY2hyaWViIEJlbmphbWluIEdh aWduYXJkOgo+ID4gPiA+IExlIDA3LzEyLzIwMjEgw6AgMDI6NTQsIEFkYW0gRm9yZCBhIMOpY3Jp dCA6Cj4gPiA+ID4gCj4gPiA+ID4gPiBDdXJyZW50bHksIHRoZSBWUFUgaW4gdGhlIGkuTVE4TVEg aXMgYXBwZWFyaW5nIGFzIG9uZSBjb2RlYywgYnV0IGluCj4gPiA+ID4gPiByZWFsaXR5LCBpdCdz IHR3byBJUCBibG9ja3MgY2FsbGVkIEcxIGFuZCBHMi4gIFRoZXJlIGlzIGluaXRpYWxpemF0aW9u Cj4gPiA+ID4gPiBjb2RlIGluIFZQVSBjb2RlIHRvIHB1bGwgc29tZSBjbG9ja3MsIHJlc2V0cyBh bmQgb3RoZXIgZmVhdHVyZXMgd2hpY2gKPiA+ID4gPiA+IGhhcyBiZWVuIGludGVncmF0ZWQgaW50 byB0aGUgdnB1LWJsay1jdHJsIGZvciB0aGUgaS5NWDhNIE1pbmkgYW5kIGEKPiA+ID4gPiA+IHNp bWlsYXIgbWV0aG9kIGNhbiBiZSB1c2VkIHRvIG1ha2UgdGhlIFZQVSBjb2RlYydzIG9wZXJhdGUg YXMKPiA+ID4gPiA+IHN0YW5kLWFsb25lIGNvcmVzIHdpdGhvdXQgaGF2aW5nIHRvIGtub3cgdGhl IGRldGFpbHMgb2YgZWFjaCBvdGhlcgo+ID4gPiA+ID4gb3IgdGhlIHF1aXJrcyB1bmlxdWUgdG8g dGhlIGkuTVg4TVEsIHNvIHRoZSByZW1haW5pbmcgY29kZSBjYW4gYmUKPiA+ID4gPiA+IGxlZnQg bW9yZSBnZW5lcmljLgo+ID4gPiA+ID4gCj4gPiA+ID4gPiBUaGlzIHNlcmllcyB3YXMgc3RhcnRl ZCBieSBMdWNhcyBTdGFjaCB3aXRoIG9uZSBieSBCZW5qYW1pbiBHYWlnbmFyZC4KPiA+ID4gPiA+ IE1vc3QgcGF0Y2hlcyBoYXZlIGJlZW4gbW9kaWZpZWQgc2xpZ2h0bHkgYnkgbWUuICBJdCdzIGlu IGFuIFJGQyBzdGF0ZQo+ID4gPiA+ID4gYmVjYXVzZSBJIHdhc24ndCBzdXJlIGhvdyB0byBiZXN0 IGhhbmRsZSB0aGUgc2lnbmF0dXJlcyBhbmQgd2Fzbid0IHN1cmUKPiA+ID4gPiA+IGlmIEkgY291 bGQgYmFzZSBpdCBvZmYgdGhlIGJyYW5jaCBJIGRpZC4KPiA+ID4gPiA+IAo+ID4gPiA+ID4gU2lu Y2UgdGhlIGctc3RyZWFtZXIgYW5kIG1lZGlhIHRyZWVzIGFyZSBpbiBhIGNvbnN0YW50IHN0YXRl IG9mCj4gPiA+ID4gPiBjaGFuZ2UsIHRoaXMgc2VyaWVzIGlzIGJhc2VkIG9uCj4gPiA+ID4gPiAK PiA+ID4gPiA+IGdpdDovL2xpbnV4dHYub3JnL2h2ZXJrdWlsL21lZGlhX3RyZWUuZ2l0IGZvci12 NS4xN2UKPiA+ID4gPiA+IAo+ID4gPiA+ID4gVGhlIGRvd25zdHJlYW0gY29kZSBmcm9tIE5YUCBz aG93cyB0aGUgRzEgYW5kIEcyIGNsb2NrcyBydW5uaW5nCj4gPiA+ID4gPiBhdCA2MDBNSHosIGJ1 dCBiZXR3ZWVuIHRoZSBUUk0gYW5kIHRoZSBkYXRhc2hlZXQsIHRoZXJlIGlzIHNvbWUKPiA+ID4g PiA+IGRpc2NyZXBhbmN5LiAgQmVjYXVzZSB0aGUgTlhQIHJlZmVyZW5jZSBjb2RlIHVzZWQgNjAw TUh6LCB0aGF0IGlzCj4gPiA+ID4gPiB3aGF0IHdhcyBjaG9zZW4gaGVyZS4gIFVzZXJzIHdobyBu ZWVkIHRvIGFkanVzdCB0aGVpciBHMSBhbmQgRzIKPiA+ID4gPiA+IGNsb2NrcyBjYW4gZG8gc28g aW4gdGhlaXIgYm9hcmQgZmlsZXMuCj4gPiA+ID4gSGkgQWRhbSwKPiA+ID4gPiAKPiA+ID4gPiBU aGFua3MgZm9yIHlvdXIgcGF0Y2hlcywgSSBoYXZlIGJlZW4gYWJsZSB0byByZXByb2R1Y2UgVlA5 IHJlc3VsdHMgb24gbXkgc2lkZSAoRmx1c3RlciAxNDcvMzAzKS4KPiA+ID4gPiBJbiBwYXN0IEkg aGF2ZSBub3RpY2Ugc3B1cmlvdXMgZXJyb3JzIHdoZW4gdXNpbmcgNjAwTUh6IGNsb2NrIG9uIEhF VkMgZGVjb2RlIGJ1dCBub3Qgd2l0aCAzMDBNSHouCj4gPiAKPiA+IFRoZSByZXN1bHRzIGZvciBG bHVzdGVyIEhFVkMgYXJlIDc3LzE0NyBzbyBubyByZWdyZXNzaW9ucyA6LSkKPiA+IAo+ID4gUmVn YXJkcywKPiA+IEJlbmphbWluCj4gPiAKPiA+ID4gPiBUaGUgbWF4IHN1cHBvcnRlZCBHMiBjbG9j ayBmcmVxdWVuY3kgaXMgNjYwTUh6IGJ1dCBuZWVkcyBhIGhpZ2hlcgo+ID4gPiA+IHZvbHRhZ2Uu IFRoZSBtYXhpbXVtIHN1cHBvcnRlZCAgZnJlcXVlbmN5IGF0IHRoZSBkZWZhdWx0IDAuOVYgaXMK PiA+ID4gPiA1NTBNSHouIFdlIHNob3VsZCBub3QgY29uZmlndXJlIHRoZSBjbG9ja3MgZm9yIHRo ZSBoaWdoZXIgdGhhbiB0aGF0LCBhcwo+ID4gPiA+IGxvbmcgYXMgdGhlcmUgaXMgbm8gc3VwcG9y dCBpbiB0aGUgVlBVIGRyaXZlciB0byBzY2FsZSB0aGUgdm9sdGFnZQo+ID4gPiA+IGFsb25nIHdp dGggdGhlIGZyZXF1ZW5jeS4gU2FtZSBhcyB3aXRoIHRoZSBHUFUgd2Ugc2hvdWxkIHN0aWNrIHRv IGJhc2UKPiA+ID4gPiBmcmVxdWVuY3kgbGV2ZWxzIGZvciB0aGUgbm9taW5hbCBvcGVyYXRpbmcg bW9kZS4KPiAKPiBMdWNhcywKPiAKPiBBZnRlciByZXZpZXdpbmcgdGhlIGNvbW1lbnRzIGZyb20g QmVuamFtaW4sIEkgcmUtcmFuIHRoZSBWUDkgdGVzdHMKPiB3aXRoIHRoZSBHMiBydW5uaW5nIGF0 IDMwME1IeixhbmQgdGhlIG51bWJlciBvZiBwYXNzaW5nIFZQOSB0ZXN0cwo+IGluY3JlYXNlZCB0 byAxNDggZnJvbSAxNDQgd2l0aCBhbiBpbmNyZWFzZSBvZiB0aW1lIHRvIDI1MC41MDIgc2Vjcwo+ IGZyb20gMTYyLjY2NSBzZWNzLgo+IAo+IFdoaWxlIHRoZSBkYXRhc2hlZXQgcmVhZHMgdGhhdCB0 aGUgRzIgY2FuIHJ1biBmYXN0ZXIsIHRoZSBpLk1YIDhNCj4gRHVhbC84TSBRdWFkTGl0ZS84TSBR dWFkIEFwcGxpY2F0aW9ucyBQcm9jZXNzb3JzIFJlZmVyZW5jZSBNYW51YWwsCj4gUmV2LiAzLjEs IDA2LzIwMjEsIHRhYmxlIDUtMSAgc2hvd3MgdGhlIFZQVV9HMl9DTEtfUk9PVCBoYXMgYSBtYXgK PiBjbG9jayBvZiAzMDBNSHouICBJIG1pZ2h0IGJlIGluY2xpbmVkIHRvIGFncmVlIHdpdGggQmVu amFtaW4gb24gdGhlCj4gMzAwTUh6IGFuZCBsZXQgcGVvcGxlIHdobyB3YW50IHRvIHB1c2ggdGhl aXIgaGFyZHdhcmUgb3ZlcndyaXRlIHRoZQo+IGRlZmF1bHQgY2xvY2tzIHNpbmNlIGl0IGluY3Jl YXNlcyBmdW5jdGlvbmFsaXR5Lgo+IApJJ20gcXVpdGUgc3VycHJpc2VkIHRoYXQgdGhlIEcyIGNv ZGVjIHNob3VsZCBoYXZlIHN1Y2ggYSBtdWNoIGxvd2VyIG1heApmcmVxdWVuY3kgY29tcGFyZWQg d2l0aCB0aGUgRzEuIFdoaWxlIHRoZSB0YWJsZSBmcm9tIHRoZSBSTSBoYXNuJ3QgYmVlbgpvYnZp b3VzbHkgaW5jb3JyZWN0IGZvciBhbnkgb3RoZXIgY2xvY2ssIEknbSBzdGlsbCBpbmNsaW5lZCB0 byBiZWxpZXZlCnRoZSBmcmVxdWVuY2llcyBzdGF0ZWQgaW4gdGhlIGRhdGFzaGVldC4KCj4gSSB3 b25kZXIgaWYgc29tZW9uZSBmcm9tIE5YUCBjYW4gY29tbWVudAo+IApZZWEsIG5vdCBzdXJlIGhv dyB0byBwcm9jZWVkIGhlcmUuIDMwME1IeiBpcyB0aGUgc2FmZSBiZXQsIGJ1dCB3ZSBhcmUKbGVh dmluZyBxdWl0ZSBzb21lIHBlcmZvcm1hbmNlIG9uIHRoZSB0YWJsZSBpZiB0aGUgZGF0YXNoZWV0 IGlzCmNvcnJlY3QuIFdpdGhvdXQgc29tZSBoZWxwIGZyb20gTlhQIGl0J3MgcHJvYmFibHkgaGFy ZCB0byB2YWxpZGF0ZSAKd2hpY2ggbWF4IGZyZXF1ZW5jeSBzdGF0ZW1lbnQgaXMgY29ycmVjdC4g SXQgZG9lc24ndCByZWFsbHkgaGVscCB0aGF0CnRoZSBOWFAgZG93bnN0cmVhbSBrZXJuZWwgc2Vl bXMgdG8gZHJpdmUgdGhlIEcyIGF0IDYwME1Ieiwgd2hpY2ggaXMKbmVpdGhlciB0aGUgNTUwTUh6 IG5vbWluYWwgbW9kZSBtYXgsIG5vciB0aGUgNjYwTUh6IG92ZXJkcml2ZSBtb2RlIG1heC4KClJl Z2FyZHMsCkx1Y2FzCgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KbGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBsaXN0 cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGlu Zm8vbGludXgtYXJtLWtlcm5lbAo=