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 AAFFDD149DC for ; Fri, 25 Oct 2024 18:25:43 +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=kE21A7TWN/DnEVblc3o8IsC88azjmCL6y+yrS2R8cUM=; b=l2clLOU0CntvoD hz2JCtDAalsRIuG7fd81s8Gg+zJQ6pvzWS+y/kwkajgH0g0Z02NWJqmy5k0VGMG6s4cdgSbYkX1ZX WCGlYD98ipWqFnqHSkiqI+RFZ+KMmSW//di/iSOyB41Pg+Dk/XNcwgcpWYSmqrJLg//2TyBwcAPor B1taMG9VSwrUGM/1zKlHfC0q6gRf8h/s92qlXvohqh/PEkOFecMsLgxY3jo6DXa0wwFEqBZmV1agU uJ7hdLcSMbexz88gRls0UBeyLPesGFmTe2vs2SozuM0h3vjs/Nx0KKD+rfcsg0FeyFqwZ/DoFaFjo sGBEYFJeFbOt0Pif4yAg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t4P0G-00000004nQt-2LYQ; Fri, 25 Oct 2024 18:25:40 +0000 Received: from bali.collaboradmins.com ([2a01:4f8:201:9162::2]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t4OPp-00000004fkV-0dwl for linux-rockchip@lists.infradead.org; Fri, 25 Oct 2024 17:48:02 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1729878479; bh=JEMC93gOTFEVpQhQTHlWxoYkWl6H9AItyV6mDgrZKvc=; h=Subject:From:To:Cc:Date:In-Reply-To:References:From; b=km/nXGHudisgqioUPVzEwXqToc9twh7P7zHvh2AFNeYMT7k0a3lGZ4bJox3mz8udW iVvXJLXnubsh1jagA22JlX4yoTjRUSNBPj9zxioyFTRVrupeyd/xNf0TDsIeEsyIoL kHxz9imqRGRRd9qoYotL/JSsZ8kHwhUJCUoacwxY91cEti7P/hYvN0p5rRzK/+20RT SZVL6zXoBIRjkkI481KlVef3qEPzzgGCC5yhqvH1y9gZtT2H/r1bpOX6+K/iOyTWSP 9hzhD64n/3T9ZIrWYKDc6Sv2eM0dBnNQvCu1ZyTEKhrhshMAkstvAMZDGI/zOeWsw1 y289FFjpJPrkg== Received: from nicolas-tpx395.lan (unknown [IPv6:2606:6d00:15:862e::7a9]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: nicolas) by bali.collaboradmins.com (Postfix) with ESMTPSA id 07B0117E36B5; Fri, 25 Oct 2024 19:47:57 +0200 (CEST) Message-ID: Subject: Re: [PATCH v6 00/11] media: rkvdec: Add H.264 High 10 and 4:2:2 profile support From: Nicolas Dufresne To: Sebastian Fricke , Jonas Karlman Cc: Ezequiel Garcia , Mauro Carvalho Chehab , Alex Bee , Benjamin Gaignard , Detlev Casanova , Dan Carpenter , linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Date: Fri, 25 Oct 2024 13:47:57 -0400 In-Reply-To: <07674bcb4b7650c21bbb3dbe9855b2240444d4f3.camel@collabora.com> References: <20240909192522.1076704-1-jonas@kwiboo.se> <71159f58-be8b-41a4-9fed-522e09a7a564@kwiboo.se> <20241025103022.yuaepqxllwi7gghb@basti-XPS-13-9310> <07674bcb4b7650c21bbb3dbe9855b2240444d4f3.camel@collabora.com> Organization: Collabora User-Agent: Evolution 3.52.4 (3.52.4-1.fc40) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241025_104801_394226_AEC28250 X-CRM114-Status: GOOD ( 29.61 ) 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 TGUgdmVuZHJlZGkgMjUgb2N0b2JyZSAyMDI0IMOgIDA4OjU0IC0wNDAwLCBOaWNvbGFzIER1ZnJl c25lIGEgw6ljcml0wqA6Cj4gTGUgdmVuZHJlZGkgMjUgb2N0b2JyZSAyMDI0IMOgIDEyOjMwICsw MjAwLCBTZWJhc3RpYW4gRnJpY2tlIGEgw6ljcml0wqA6Cj4gPiBIZXkgSm9uYXMsCj4gPiAKPiA+ IE9uIDI1LjEwLjIwMjQgMTA6MjAsIEpvbmFzIEthcmxtYW4gd3JvdGU6Cj4gPiA+IEhpIFNlYmFz dGlhbiwKPiA+ID4gCj4gPiA+IFdpbGwgeW91IGhhdmUgdGltZSB0byBsb29rIGF0IHRoaXMgc2Vy aWVzIGFueSB0aW1lIHNvb24/Cj4gPiA+IAo+ID4gPiBXb3VsZCBsaWtlIHRvIHNlbmQgYSB2MiBv ZiB0aGUgbm93IG9uZSB5ZWFyIG9sZCBya3ZkZWMgaGV2YyBzZXJpZXMgYnV0Cj4gPiA+IHRoaXMg c2VyaWVzIGlzIHNvcnQgb2YgaG9sZGluZyB0aGF0IGJhY2sgOy0pCj4gPiAKPiA+IFNvcnJ5IGZv ciB0aGUgZGVsYXkgd2UgKG1haW50YWluZXIgJiByZXZpZXdlciBmcm9tIENvbGxhYm9yYSkgYXJl Cj4gPiBjdXJyZW50bHkgYSBiaXQgYnVzeSwgdGhpcyBpcyBvbiB0b3Agb2Ygb3VyIHJldmlldyBs aXN0IGhvd2V2ZXIsIHNvCj4gPiBwbGVhc2UgYW5vdGhlciAxIG9yIDIgd2Vla3Mgb2YgcGF0aWVu Y2UgOikuCj4gCj4gQWN0dWFsbHksIEknbGwgZ28gb3ZlciBpdCB0b2RheS4KCkknbSBkb25lLCB0 aGVyZSBpcyBvbmx5IGNvc21ldGljIGNvbW1lbnQsIGlmIHlvdSByZXNwaW4gcXVpY2tseSwgSSdt IHN1cmUgd2UgY2FuCmdldCB0aGlzIG9uZSBpbiBzb29uLgoKPiAKPiByZWdhcmRzLAo+IE5pY29s YXMKPiAKPiA+IAo+ID4gPiAKPiA+ID4gUmVnYXJkcywKPiA+ID4gSm9uYXMKPiA+IAo+ID4gVGhh bmtzIGZvciB5b3VyIHdvcmsgYW5kIHJlZ2FyZHMhCj4gPiBTZWJhc3RpYW4KPiA+IAo+ID4gPiAK PiA+ID4gT24gMjAyNC0wOS0wOSAyMToyNCwgSm9uYXMgS2FybG1hbiB3cm90ZToKPiA+ID4gPiBU aGlzIHNlcmllcyBhZGQgSC4yNjQgSGlnaCAxMCBhbmQgNDoyOjIgcHJvZmlsZSBzdXBwb3J0IHRv IHRoZSBSb2NrY2hpcAo+ID4gPiA+IFZpZGVvIERlY29kZXIgZHJpdmVyLgo+ID4gPiA+IAo+ID4g PiA+IFBhdGNoIDEgYWRkIGhlbHBlcnMgZm9yIGNhbGN1bGF0aW5nIHBsYW5lIGJ5dGVzcGVybGlu ZSBhbmQgc2l6ZWltYWdlLgo+ID4gPiA+IFBhdGNoIDIgYWRkIHR3byBuZXcgcGl4ZWxmb3JtYXRz IGZvciBzZW1pLXBsYW5lciAxMC1iaXQgNDoyOjAvNDoyOjIgWVVWLgo+ID4gPiA+IAo+ID4gPiA+ IFBhdGNoIDMgY2hhbmdlIHRvIHVzZSBieXRlc3BlcmxpbmUgYW5kIGJ1ZmZlciBoZWlnaHQgdG8g Y29uZmlndXJlIHN0cmlkZXMuCj4gPiA+ID4gUGF0Y2ggNCBjaGFuZ2UgdG8gdXNlIHZhbHVlcyBm cm9tIFNQUy9QUFMgY29udHJvbCB0byBjb25maWd1cmUgdGhlIEhXLgo+ID4gPiA+IAo+ID4gPiA+ IFBhdGNoIDUtOSByZWZhY3RvciBjb2RlIHRvIHN1cHBvcnQgZmlsdGVyaW5nIG9mIENBUFVUUkUg Zm9ybWF0cyBiYXNlZAo+ID4gPiA+IG9uIHRoZSBpbWFnZSBmb3JtYXQgcmV0dXJuZWQgZnJvbSBh IGdldF9pbWFnZV9mbXQgb3BzLgo+ID4gPiA+IAo+ID4gPiA+IFBhdGNoIDEwIGFkZCBmaW5hbCBi aXRzIHRvIHN1cHBvcnQgSC4yNjQgSGlnaCAxMCBhbmQgNDoyOjIgcHJvZmlsZXMuCj4gPiA+ID4g Cj4gPiA+ID4gUGF0Y2ggMTEgYWRkIGEgZml4IGZvciBlbnVtZXJhdGVkIGZyYW1lIHNpemVzIHJl dHVybmVkIHRvIHVzZXJzcGFjZS4KPiA+ID4gPiAKPiA+ID4gPiBUZXN0ZWQgb24gYSBST0NLIFBp IDQgKFJLMzM5OSkgYW5kIFJvY2s2NCAoUkszMzI4KToKPiA+ID4gPiAKPiA+ID4gPiAgIHY0bDIt Y29tcGxpYW5jZSAxLjI4LjEsIDY0IGJpdHMsIDY0LWJpdCB0aW1lX3QKPiA+ID4gPiAgIC4uLgo+ ID4gPiA+ICAgVG90YWwgZm9yIHJrdmRlYyBkZXZpY2UgL2Rldi92aWRlbzE6IDQ4LCBTdWNjZWVk ZWQ6IDQ4LCBGYWlsZWQ6IDAsIFdhcm5pbmdzOiAwCj4gPiA+ID4gCj4gPiA+ID4gICBSdW5uaW5n IHRlc3Qgc3VpdGUgSlZULUZSLUVYVCB3aXRoIGRlY29kZXIgRkZtcGVnLUguMjY0LXY0bDJyZXF1 ZXN0Cj4gPiA+ID4gICAuLi4KPiA+ID4gPiAgIFJhbiA2NS82OSB0ZXN0cyBzdWNjZXNzZnVsbHkK PiA+ID4gPiAKPiA+ID4gPiAgIFJ1bm5pbmcgdGVzdCBzdWl0ZSBKVlQtQVZDX1YxIHdpdGggZGVj b2RlciBGRm1wZWctSC4yNjQtdjRsMnJlcXVlc3QKPiA+ID4gPiAgIC4uLgo+ID4gPiA+ICAgUmFu IDEyOS8xMzUgdGVzdHMgc3VjY2Vzc2Z1bGx5Cj4gPiA+ID4gCj4gPiA+ID4gQmVmb3JlIHRoaXMg c2VyaWVzOgo+ID4gPiA+IAo+ID4gPiA+ICAgUnVubmluZyB0ZXN0IHN1aXRlIEpWVC1GUi1FWFQg d2l0aCBkZWNvZGVyIEZGbXBlZy1ILjI2NC12NGwycmVxdWVzdAo+ID4gPiA+ICAgLi4uCj4gPiA+ ID4gICBSYW4gNDQvNjkgdGVzdHMgc3VjY2Vzc2Z1bGx5Cj4gPiA+ID4gCj4gPiA+ID4gQ2hhbmdl cyBpbiB2NjoKPiA+ID4gPiAtIENoYW5nZSB0byB1c2UgZm10X2lkeCBpbnN0ZWFkIG9mIGorKyB0 dWNrZWQgaW5zaWRlIGEgY29uZGl0aW9uIChEYW4pCj4gPiA+ID4gLSBBZGQgcGF0Y2ggdG8gZml4 IGVudW1lcmF0ZWQgZnJhbWUgc2l6ZXMgcmV0dXJuZWQgdG8gdXNlcnNwYWNlIChBbGV4KQo+ID4g PiA+IC0gRmx1c3RlciB0ZXN0IHNjb3JlIGlzIHNhbWUgYXMgdjQgYW5kIHY1LCBzZWUgWzRdIGFu ZCBbNV0KPiA+ID4gPiBMaW5rIHRvIHY1OiBodHRwczovL2xvcmUua2VybmVsLm9yZy9saW51eC1t ZWRpYS8yMDI0MDYxODE5NDY0Ny43NDIwMzctMS1qb25hc0Brd2lib28uc2UvCj4gPiA+ID4gCj4g PiA+ID4gQ2hhbmdlcyBpbiB2NToKPiA+ID4gPiAtIERyb3AgUmVtb3ZlIFNQUyB2YWxpZGF0aW9u IGF0IHN0cmVhbWluZyBzdGFydCBwYXRjaAo+ID4gPiA+IC0gTW92ZSBidWZmZXIgYWxpZ24gZnJv bSBya3ZkZWNfZmlsbF9kZWNvZGVkX3BpeGZtdCB0byBtaW4vc3RlcF93aWR0aAo+ID4gPiA+IC0g VXNlIGNvcnJlY3QgcHJvZmlsZXMgZm9yIFY0TDJfQ0lEX01QRUdfVklERU9fSDI2NF9QUk9GSUxF Cj4gPiA+ID4gLSBDb2xsZWN0IHItYiBhbmQgdC1iIHRhZ3MKPiA+ID4gPiAtIEZsdXN0ZXIgdGVz dCBzY29yZSBpcyBzYW1lIGFzIHY0LCBzZWUgWzRdIGFuZCBbNV0KPiA+ID4gPiBMaW5rIHRvIHY0 OiBodHRwczovL2xvcmUua2VybmVsLm9yZy9saW51eC1tZWRpYS8yMDIzMTEwNTE2NTUyMS4zNTky MDM3LTEtam9uYXNAa3dpYm9vLnNlLwo+ID4gPiA+IAo+ID4gPiA+IENoYW5nZXMgaW4gdjQ6Cj4g PiA+ID4gLSBGaXggZmFpbGVkIHY0bDItY29tcGxpYW5jZSB0ZXN0cyByZWxhdGVkIHRvIENBUFRV UkUgcXVldWUKPiA+ID4gPiAtIFJld29yayBDQVBUVVJFIGZvcm1hdCBmaWx0ZXIgYW52IHZhbGlk YXRlIHRvIHVzZSBhbiBpbWFnZSBmb3JtYXQKPiA+ID4gPiAtIFJ1biBmbHVzdGVyIHRlc3Qgc3Vp dGUgSlZULUZSLUVYVCBbNF0gYW5kIEpWVC1BVkNfVjEgWzVdCj4gPiA+ID4gTGluayB0byB2Mzog aHR0cHM6Ly9sb3JlLmtlcm5lbC5vcmcvbGludXgtbWVkaWEvMjAyMzEwMjkxODM0MjcuMTc4MTU1 NC0xLWpvbmFzQGt3aWJvby5zZS8KPiA+ID4gPiAKPiA+ID4gPiBDaGFuZ2VzIGluIHYzOgo+ID4g PiA+IC0gRHJvcCBtZXJnZWQgcGF0Y2hlcwo+ID4gPiA+IC0gVXNlIGJwcCBhbmQgYnBwX2RpdiBp bnN0ZWFkIG9mIHByaW9yIG1pc3VzZSBvZiBibG9ja193L2Jsb2NrX2gKPiA+ID4gPiAtIE5ldyBw YXRjaCB0byB1c2UgdmFsdWVzIGZyb20gU1BTL1BQUyBjb250cm9sIHRvIGNvbmZpZ3VyZSB0aGUg SFcKPiA+ID4gPiAtIE5ldyBwYXRjaCB0byByZW1vdmUgYW4gdW5uZWNlc3NhcnkgY2FsbCB0byB2 YWxpZGF0ZSBzcHMgYXQgc3RyZWFtaW5nIHN0YXJ0Cj4gPiA+ID4gLSBSZXdvcmtlZCBwaXhlbCBm b3JtYXQgdmFsaWRhdGlvbgo+ID4gPiA+IExpbmsgdG8gdjI6IGh0dHBzOi8vbG9yZS5rZXJuZWwu b3JnL2xpbnV4LW1lZGlhLzIwMjAwNzA2MjE1NDMwLjIyODU5LTEtam9uYXNAa3dpYm9vLnNlLwo+ ID4gPiA+IAo+ID4gPiA+IENoYW5nZXMgaW4gdjI6Cj4gPiA+ID4gLSBDb2xsZWN0IHItYiB0YWdz Cj4gPiA+ID4gLSBTUFMgcGljIHdpZHRoIGFuZCBoZWlnaHQgaW4gbWJzIHZhbGlkYXRpb24gbW92 ZWQgdG8gcmt2ZGVjX3RyeV9jdHJsCj4gPiA+ID4gLSBOZXcgcGF0Y2ggdG8gbm90IG92ZXJyaWRl IG91dHB1dCBidWZmZXIgc2l6ZWltYWdlCj4gPiA+ID4gLSBSZXdvcmtlZCBwaXhlbCBmb3JtYXQg dmFsaWRhdGlvbgo+ID4gPiA+IC0gT25seSBhbGlnbiBkZWNvZGVkIGJ1ZmZlciBpbnN0ZWFkIG9m IGNoYW5naW5nIGZybXNpemUgc3RlcF93aWR0aAo+ID4gPiA+IExpbmsgdG8gdjE6IGh0dHBzOi8v bG9yZS5rZXJuZWwub3JnL2xpbnV4LW1lZGlhLzIwMjAwNzAxMjE1NjE2LjMwODc0LTEtam9uYXNA a3dpYm9vLnNlLwo+ID4gPiA+IAo+ID4gPiA+IFRvIGZ1bGx5IHJ1bnRpbWUgdGVzdCB0aGlzIHNl cmllcyB5b3UgbWF5IG5lZWQgRkZtcGVnIHBhdGNoZXMgZnJvbSBbMV0KPiA+ID4gPiBhbmQgZmx1 c3RlciBwYXRjaGVzIGZyb20gWzJdLCB0aGlzIHNlcmllcyBpcyBhbHNvIGF2YWlsYWJsZSBhdCBb M10uCj4gPiA+ID4gCj4gPiA+ID4gWzFdIGh0dHBzOi8vZ2l0aHViLmNvbS9Ld2lib28vRkZtcGVn L2NvbW1pdHMvdjRsMnJlcXVlc3QtMjAyNC12Mi1ya3ZkZWMvCj4gPiA+ID4gWzJdIGh0dHBzOi8v Z2l0aHViLmNvbS9Ld2lib28vZmx1c3Rlci9jb21taXRzL2ZmbXBlZy12NGwycmVxdWVzdC1ya3Zk ZWMvCj4gPiA+ID4gWzNdIGh0dHBzOi8vZ2l0aHViLmNvbS9Ld2lib28vbGludXgtcm9ja2NoaXAv Y29tbWl0cy9saW51eHR2LXJrdmRlYy1oaWdoLTEwLXY2Lwo+ID4gPiA+IFs0XSBodHRwczovL2dp c3QuZ2l0aHViLmNvbS9Ld2lib28vZjRhYzE1NTc2YjJjNzI4ODdhZTJiYzVkNThiNWM4NjUKPiA+ ID4gPiBbNV0gaHR0cHM6Ly9naXN0LmdpdGh1Yi5jb20vS3dpYm9vLzQ1OWExYzhmMWRjYjU2ZTQ1 ZGM3YTdhMjljYzI4YWRmCj4gPiA+ID4gCj4gPiA+ID4gUmVnYXJkcywKPiA+ID4gPiBKb25hcwo+ ID4gPiA+IAo+ID4gPiA+IEFsZXggQmVlICgxKToKPiA+ID4gPiAgIG1lZGlhOiBya3ZkZWM6IGgy NjQ6IERvbid0IGhhcmRjb2RlIFNQUy9QUFMgcGFyYW1ldGVycwo+ID4gPiA+IAo+ID4gPiA+IEpv bmFzIEthcmxtYW4gKDEwKToKPiA+ID4gPiAgIG1lZGlhOiB2NGwyLWNvbW1vbjogQWRkIGhlbHBl cnMgdG8gY2FsY3VsYXRlIGJ5dGVzcGVybGluZSBhbmQKPiA+ID4gPiAgICAgc2l6ZWltYWdlCj4g PiA+ID4gICBtZWRpYTogdjRsMjogQWRkIE5WMTUgYW5kIE5WMjAgcGl4ZWwgZm9ybWF0cwo+ID4g PiA+ICAgbWVkaWE6IHJrdmRlYzogaDI2NDogVXNlIGJ5dGVzcGVybGluZSBhbmQgYnVmZmVyIGhl aWdodCBhcyB2aXJzdHJpZGUKPiA+ID4gPiAgIG1lZGlhOiBya3ZkZWM6IEV4dHJhY3Qgcmt2ZGVj X2ZpbGxfZGVjb2RlZF9waXhmbXQgaW50byBoZWxwZXIKPiA+ID4gPiAgIG1lZGlhOiBya3ZkZWM6 IE1vdmUgcmt2ZGVjX3Jlc2V0X2RlY29kZWRfZm10IGhlbHBlcgo+ID4gPiA+ICAgbWVkaWE6IHJr dmRlYzogRXh0cmFjdCBkZWNvZGVkIGZvcm1hdCBlbnVtZXJhdGlvbiBpbnRvIGhlbHBlcgo+ID4g PiA+ICAgbWVkaWE6IHJrdmRlYzogQWRkIGltYWdlIGZvcm1hdCBjb25jZXB0Cj4gPiA+ID4gICBt ZWRpYTogcmt2ZGVjOiBBZGQgZ2V0X2ltYWdlX2ZtdCBvcHMKPiA+ID4gPiAgIG1lZGlhOiBya3Zk ZWM6IGgyNjQ6IFN1cHBvcnQgSGlnaCAxMCBhbmQgNDoyOjIgcHJvZmlsZXMKPiA+ID4gPiAgIG1l ZGlhOiBya3ZkZWM6IEZpeCBlbnVtZXJhdGUgZnJhbWUgc2l6ZXMKPiA+ID4gPiAKPiA+ID4gPiAg Li4uL21lZGlhL3Y0bC9waXhmbXQteXV2LXBsYW5hci5yc3QgICAgICAgICAgIHwgMTI4ICsrKysr KysrKysKPiA+ID4gPiAgZHJpdmVycy9tZWRpYS92NGwyLWNvcmUvdjRsMi1jb21tb24uYyAgICAg ICAgIHwgIDgwICsrKy0tLQo+ID4gPiA+ICBkcml2ZXJzL21lZGlhL3Y0bDItY29yZS92NGwyLWlv Y3RsLmMgICAgICAgICAgfCAgIDIgKwo+ID4gPiA+ICBkcml2ZXJzL3N0YWdpbmcvbWVkaWEvcmt2 ZGVjL3JrdmRlYy1oMjY0LmMgICAgfCAgNjQgKysrLS0KPiA+ID4gPiAgZHJpdmVycy9zdGFnaW5n L21lZGlhL3JrdmRlYy9ya3ZkZWMuYyAgICAgICAgIHwgMjM5ICsrKysrKysrKysrKystLS0tLQo+ ID4gPiA+ICBkcml2ZXJzL3N0YWdpbmcvbWVkaWEvcmt2ZGVjL3JrdmRlYy5oICAgICAgICAgfCAg MTggKy0KPiA+ID4gPiAgaW5jbHVkZS91YXBpL2xpbnV4L3ZpZGVvZGV2Mi5oICAgICAgICAgICAg ICAgIHwgICAyICsKPiA+ID4gPiAgNyBmaWxlcyBjaGFuZ2VkLCA0MTAgaW5zZXJ0aW9ucygrKSwg MTIzIGRlbGV0aW9ucygtKQo+ID4gPiA+IAo+ID4gPiAKPiA+ID4gCj4gPiBTZWJhc3RpYW4gRnJp Y2tlCj4gPiBDb25zdWx0YW50IFNvZnR3YXJlIEVuZ2luZWVyCj4gPiAKPiA+IENvbGxhYm9yYSBM dGQKPiA+IFBsYXRpbnVtIEJ1aWxkaW5nLCBTdCBKb2huJ3MgSW5ub3ZhdGlvbiBQYXJrLCBDYW1i cmlkZ2UgQ0I0IDBEUywgVUsKPiA+IFJlZ2lzdGVyZWQgaW4gRW5nbGFuZCAmIFdhbGVzIG5vIDU1 MTM3MTguCj4gCj4gCgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KTGludXgtcm9ja2NoaXAgbWFpbGluZyBsaXN0CkxpbnV4LXJvY2tjaGlwQGxpc3RzLmlu ZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9s aW51eC1yb2NrY2hpcAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bali.collaboradmins.com (bali.collaboradmins.com [148.251.105.195]) (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 6FDE714D6EF for ; Fri, 25 Oct 2024 17:48:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=148.251.105.195 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729878484; cv=none; b=Ns4CigiU5Iu4a8pmCxpgZSgCvYUMpXuZNvEhINdlNJUY2B2rYIZ+nu8bpOdMHv/xAmLMhiEd4YufRISoBnyNf1DjgmiH3TJfWXT8DqRDRoPmvEcWrg+1DkaKtsTImaQynjeFfTiW+7ciDLt6ff9eu/MS6T3DWYP8Us9tdF8G2B0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729878484; c=relaxed/simple; bh=JEMC93gOTFEVpQhQTHlWxoYkWl6H9AItyV6mDgrZKvc=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=J3LaQS6afoSwfFiQ+Djg97x7ntULMLdZBGcnRHT6KtXH5a3Su2UeBFljsmZC63OwtA2XVTCjTGgzq7F2d/5otX/IEiZ4NYVBCYKwbpRA5RjReGE0aGp0R23zFne1xTT03eojcv+1jpJOwt2RYK/qUeXQdiSxa3Bi3BxHQM8iMUA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b=km/nXGHu; arc=none smtp.client-ip=148.251.105.195 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="km/nXGHu" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1729878479; bh=JEMC93gOTFEVpQhQTHlWxoYkWl6H9AItyV6mDgrZKvc=; h=Subject:From:To:Cc:Date:In-Reply-To:References:From; b=km/nXGHudisgqioUPVzEwXqToc9twh7P7zHvh2AFNeYMT7k0a3lGZ4bJox3mz8udW iVvXJLXnubsh1jagA22JlX4yoTjRUSNBPj9zxioyFTRVrupeyd/xNf0TDsIeEsyIoL kHxz9imqRGRRd9qoYotL/JSsZ8kHwhUJCUoacwxY91cEti7P/hYvN0p5rRzK/+20RT SZVL6zXoBIRjkkI481KlVef3qEPzzgGCC5yhqvH1y9gZtT2H/r1bpOX6+K/iOyTWSP 9hzhD64n/3T9ZIrWYKDc6Sv2eM0dBnNQvCu1ZyTEKhrhshMAkstvAMZDGI/zOeWsw1 y289FFjpJPrkg== Received: from nicolas-tpx395.lan (unknown [IPv6:2606:6d00:15:862e::7a9]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: nicolas) by bali.collaboradmins.com (Postfix) with ESMTPSA id 07B0117E36B5; Fri, 25 Oct 2024 19:47:57 +0200 (CEST) Message-ID: Subject: Re: [PATCH v6 00/11] media: rkvdec: Add H.264 High 10 and 4:2:2 profile support From: Nicolas Dufresne To: Sebastian Fricke , Jonas Karlman Cc: Ezequiel Garcia , Mauro Carvalho Chehab , Alex Bee , Benjamin Gaignard , Detlev Casanova , Dan Carpenter , linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Date: Fri, 25 Oct 2024 13:47:57 -0400 In-Reply-To: <07674bcb4b7650c21bbb3dbe9855b2240444d4f3.camel@collabora.com> References: <20240909192522.1076704-1-jonas@kwiboo.se> <71159f58-be8b-41a4-9fed-522e09a7a564@kwiboo.se> <20241025103022.yuaepqxllwi7gghb@basti-XPS-13-9310> <07674bcb4b7650c21bbb3dbe9855b2240444d4f3.camel@collabora.com> Organization: Collabora Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.52.4 (3.52.4-1.fc40) Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Le vendredi 25 octobre 2024 à 08:54 -0400, Nicolas Dufresne a écrit : > Le vendredi 25 octobre 2024 à 12:30 +0200, Sebastian Fricke a écrit : > > Hey Jonas, > > > > On 25.10.2024 10:20, Jonas Karlman wrote: > > > Hi Sebastian, > > > > > > Will you have time to look at this series any time soon? > > > > > > Would like to send a v2 of the now one year old rkvdec hevc series but > > > this series is sort of holding that back ;-) > > > > Sorry for the delay we (maintainer & reviewer from Collabora) are > > currently a bit busy, this is on top of our review list however, so > > please another 1 or 2 weeks of patience :). > > Actually, I'll go over it today. I'm done, there is only cosmetic comment, if you respin quickly, I'm sure we can get this one in soon. > > regards, > Nicolas > > > > > > > > > Regards, > > > Jonas > > > > Thanks for your work and regards! > > Sebastian > > > > > > > > On 2024-09-09 21:24, Jonas Karlman wrote: > > > > This series add H.264 High 10 and 4:2:2 profile support to the Rockchip > > > > Video Decoder driver. > > > > > > > > Patch 1 add helpers for calculating plane bytesperline and sizeimage. > > > > Patch 2 add two new pixelformats for semi-planer 10-bit 4:2:0/4:2:2 YUV. > > > > > > > > Patch 3 change to use bytesperline and buffer height to configure strides. > > > > Patch 4 change to use values from SPS/PPS control to configure the HW. > > > > > > > > Patch 5-9 refactor code to support filtering of CAPUTRE formats based > > > > on the image format returned from a get_image_fmt ops. > > > > > > > > Patch 10 add final bits to support H.264 High 10 and 4:2:2 profiles. > > > > > > > > Patch 11 add a fix for enumerated frame sizes returned to userspace. > > > > > > > > Tested on a ROCK Pi 4 (RK3399) and Rock64 (RK3328): > > > > > > > > v4l2-compliance 1.28.1, 64 bits, 64-bit time_t > > > > ... > > > > Total for rkvdec device /dev/video1: 48, Succeeded: 48, Failed: 0, Warnings: 0 > > > > > > > > Running test suite JVT-FR-EXT with decoder FFmpeg-H.264-v4l2request > > > > ... > > > > Ran 65/69 tests successfully > > > > > > > > Running test suite JVT-AVC_V1 with decoder FFmpeg-H.264-v4l2request > > > > ... > > > > Ran 129/135 tests successfully > > > > > > > > Before this series: > > > > > > > > Running test suite JVT-FR-EXT with decoder FFmpeg-H.264-v4l2request > > > > ... > > > > Ran 44/69 tests successfully > > > > > > > > Changes in v6: > > > > - Change to use fmt_idx instead of j++ tucked inside a condition (Dan) > > > > - Add patch to fix enumerated frame sizes returned to userspace (Alex) > > > > - Fluster test score is same as v4 and v5, see [4] and [5] > > > > Link to v5: https://lore.kernel.org/linux-media/20240618194647.742037-1-jonas@kwiboo.se/ > > > > > > > > Changes in v5: > > > > - Drop Remove SPS validation at streaming start patch > > > > - Move buffer align from rkvdec_fill_decoded_pixfmt to min/step_width > > > > - Use correct profiles for V4L2_CID_MPEG_VIDEO_H264_PROFILE > > > > - Collect r-b and t-b tags > > > > - Fluster test score is same as v4, see [4] and [5] > > > > Link to v4: https://lore.kernel.org/linux-media/20231105165521.3592037-1-jonas@kwiboo.se/ > > > > > > > > Changes in v4: > > > > - Fix failed v4l2-compliance tests related to CAPTURE queue > > > > - Rework CAPTURE format filter anv validate to use an image format > > > > - Run fluster test suite JVT-FR-EXT [4] and JVT-AVC_V1 [5] > > > > Link to v3: https://lore.kernel.org/linux-media/20231029183427.1781554-1-jonas@kwiboo.se/ > > > > > > > > Changes in v3: > > > > - Drop merged patches > > > > - Use bpp and bpp_div instead of prior misuse of block_w/block_h > > > > - New patch to use values from SPS/PPS control to configure the HW > > > > - New patch to remove an unnecessary call to validate sps at streaming start > > > > - Reworked pixel format validation > > > > Link to v2: https://lore.kernel.org/linux-media/20200706215430.22859-1-jonas@kwiboo.se/ > > > > > > > > Changes in v2: > > > > - Collect r-b tags > > > > - SPS pic width and height in mbs validation moved to rkvdec_try_ctrl > > > > - New patch to not override output buffer sizeimage > > > > - Reworked pixel format validation > > > > - Only align decoded buffer instead of changing frmsize step_width > > > > Link to v1: https://lore.kernel.org/linux-media/20200701215616.30874-1-jonas@kwiboo.se/ > > > > > > > > To fully runtime test this series you may need FFmpeg patches from [1] > > > > and fluster patches from [2], this series is also available at [3]. > > > > > > > > [1] https://github.com/Kwiboo/FFmpeg/commits/v4l2request-2024-v2-rkvdec/ > > > > [2] https://github.com/Kwiboo/fluster/commits/ffmpeg-v4l2request-rkvdec/ > > > > [3] https://github.com/Kwiboo/linux-rockchip/commits/linuxtv-rkvdec-high-10-v6/ > > > > [4] https://gist.github.com/Kwiboo/f4ac15576b2c72887ae2bc5d58b5c865 > > > > [5] https://gist.github.com/Kwiboo/459a1c8f1dcb56e45dc7a7a29cc28adf > > > > > > > > Regards, > > > > Jonas > > > > > > > > Alex Bee (1): > > > > media: rkvdec: h264: Don't hardcode SPS/PPS parameters > > > > > > > > Jonas Karlman (10): > > > > media: v4l2-common: Add helpers to calculate bytesperline and > > > > sizeimage > > > > media: v4l2: Add NV15 and NV20 pixel formats > > > > media: rkvdec: h264: Use bytesperline and buffer height as virstride > > > > media: rkvdec: Extract rkvdec_fill_decoded_pixfmt into helper > > > > media: rkvdec: Move rkvdec_reset_decoded_fmt helper > > > > media: rkvdec: Extract decoded format enumeration into helper > > > > media: rkvdec: Add image format concept > > > > media: rkvdec: Add get_image_fmt ops > > > > media: rkvdec: h264: Support High 10 and 4:2:2 profiles > > > > media: rkvdec: Fix enumerate frame sizes > > > > > > > > .../media/v4l/pixfmt-yuv-planar.rst | 128 ++++++++++ > > > > drivers/media/v4l2-core/v4l2-common.c | 80 +++--- > > > > drivers/media/v4l2-core/v4l2-ioctl.c | 2 + > > > > drivers/staging/media/rkvdec/rkvdec-h264.c | 64 +++-- > > > > drivers/staging/media/rkvdec/rkvdec.c | 239 +++++++++++++----- > > > > drivers/staging/media/rkvdec/rkvdec.h | 18 +- > > > > include/uapi/linux/videodev2.h | 2 + > > > > 7 files changed, 410 insertions(+), 123 deletions(-) > > > > > > > > > > > > Sebastian Fricke > > Consultant Software Engineer > > > > Collabora Ltd > > Platinum Building, St John's Innovation Park, Cambridge CB4 0DS, UK > > Registered in England & Wales no 5513718. > >