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 10737D63939 for ; Wed, 20 Nov 2024 13:42:17 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=sdd6w/9TA+HzYnD3hY5bPOGd481t2BiRonTO02mJB6Y=; b=DVrIXnUUs4ixwI jYAAvoctaJgN+EiRZ59BTZoH8TElNMzVR3JVEj8eqykpBbzTYMxWEvV2goLvAvetx7I0ay4Au5rBI G0k3j9UKrTjWkNPx+HmPEzL0wkadOJ5LxOdgEhVGQpN9myfcjb4UAZRxq4T+GZJxSS62bdxSmJIJ8 4igHEiLe5cOtxeubySNMO9ujS9pyagpfQpVj/irCtGLUpBgocGmrBkd9ozj443AmLL4nBN7h6Iz/C +Y3btgr6K5g+hNgh0OAWAgHpT0OOLUjkNPj4rC+5BSgfK8A7QOc6WfylAeyPawvzAHEV+7c5irFU0 tZmMgwSHxp1b36KAIe2Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tDkyA-0000000FQCc-4Aiy; Wed, 20 Nov 2024 13:42:10 +0000 Received: from [2a01:e0a:828:c7c0:e2d5:5eff:fe2d:8e8] (helo=luna.linkmauve.fr) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tDky8-0000000FQBo-2VSY for linux-rockchip@lists.infradead.org; Wed, 20 Nov 2024 13:42:10 +0000 Received: by luna.linkmauve.fr (Postfix, from userid 1000) id D522618559CF; Wed, 20 Nov 2024 14:42:03 +0100 (CET) Date: Wed, 20 Nov 2024 14:42:03 +0100 From: Link Mauve To: Hugues Fruchet Cc: Mauro Carvalho Chehab , Ezequiel Garcia , Philipp Zabel , Hans Verkuil , Fritz Koenig , Sebastian Fricke , Daniel Almeida , Andrzej Pietrasiewicz , Nicolas Dufresne , Benjamin Gaignard , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com Subject: Re: [PATCH v2 1/3] media: uapi: add WebP uAPI Message-ID: References: <20241120110105.244413-1-hugues.fruchet@foss.st.com> <20241120110105.244413-2-hugues.fruchet@foss.st.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20241120110105.244413-2-hugues.fruchet@foss.st.com> Jabber-ID: linkmauve@linkmauve.fr X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241120_054208_787095_39741065 X-CRM114-Status: GOOD ( 25.48 ) 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 SGksCgpPbiBXZWQsIE5vdiAyMCwgMjAyNCBhdCAxMjowMTowM1BNICswMTAwLCBIdWd1ZXMgRnJ1 Y2hldCB3cm90ZToKPiBUaGlzIHBhdGNoIGFkZHMgdGhlIFdlYlAgcGljdHVyZSBkZWNvZGluZyBr ZXJuZWwgdUFQSS4KPiAKPiBUaGlzIGRlc2lnbiBpcyBiYXNlZCBvbiBjdXJyZW50bHkgYXZhaWxh YmxlIFZQOCBBUEkgaW1wbGVtZW50YXRpb24gYW5kCj4gYWltcyB0byBzdXBwb3J0IHRoZSBkZXZl bG9wbWVudCBvZiBXZWJQIHN0YXRlbGVzcyB2aWRlbyBjb2RlY3MKPiBvbiBMaW51eC4KCldoeSBk byB5b3UgbmVlZCB0aGlzIG5ldyB1QVBJIGV4YWN0bHk/ICBUaGUgV2ViUCBmb3JtYXQgaXMgbW9y ZSBjb21wbGV4CnRoYW4gdGhlIHNpbXBsZSAnVlA4ICcgZm9ybWF0LCB0aGUgJ1ZQOFgnIGZvdXJj YyBmb3IgaW5zdGFuY2UgaXMgYW4KYW5pbWF0ZWQgZm9ybWF0IHdoaWNoIG1heSBjb250YWluIG11 bHRpcGxlIFZQOCBrZXlmcmFtZXMsIG9yIGFuIGFscGhhCnNpZGUgY2hhbm5lbCwgYW5kIGp1c3Qg bGlrZSBhbnkgb3RoZXIgdmlkZW8gY29udGFpbmVyIHdlIHF1ZXVlIGVhY2gKVlA4IGZyYW1lIHNl cGFyYXRlbHkgaW4gVjRMMiBmb3IgZGVjb2RpbmcsIG5vdCB0aGUgd2hvbGUgZmlsZS4KCkluIE9u aXhbMV0gSSBwYXJzZSB0aGUgV2ViUCBoZWFkZXIgYW5kIHBhc3MgdGhlIHJhdyBWUDggZnJhbWUg dG8gVjRMMgp3aXRob3V0IHRoZSBSSUZGIGFyb3VuZCBpdC4KClNvIEnigJlkIHJhdGhlciBOQUNL IHRoaXMgcGF0Y2gsIEkgZG9u4oCZdCB0aGluayBpdOKAmXMgYSBnb29kIGlkZWEgdG8KaGFyZGNv ZGUgdGhlIHNpbXBsZXN0IHZlcnNpb24gb2YgdGhlIFdlYlAgY29udGFpbmVyIGluIHRoZSB1QVBJ LCB0byB0aGUKZGV0cmltZW50IG9mIGFsbCBvdGhlciBwb3NzaWJsZSBXZWJQIGZpbGVzLgoKWzFd IGdpdCBjbG9uZSBodHRwczovL2dpdC5saW5rbWF1dmUuZnIvb25peC5naXQvCgo+IAo+IFNpZ25l ZC1vZmYtYnk6IEh1Z3VlcyBGcnVjaGV0IDxodWd1ZXMuZnJ1Y2hldEBmb3NzLnN0LmNvbT4KPiAt LS0KPiAgRG9jdW1lbnRhdGlvbi91c2Vyc3BhY2UtYXBpL21lZGlhL3Y0bC9iaWJsaW8ucnN0ICB8 ICA5ICsrKysrKysrKwo+ICAuLi4vdXNlcnNwYWNlLWFwaS9tZWRpYS92NGwvcGl4Zm10LWNvbXBy ZXNzZWQucnN0IHwgMTUgKysrKysrKysrKysrKysrCj4gIGRyaXZlcnMvbWVkaWEvdjRsMi1jb3Jl L3Y0bDItaW9jdGwuYyAgICAgICAgICAgICAgfCAgMSArCj4gIGluY2x1ZGUvdWFwaS9saW51eC92 aWRlb2RldjIuaCAgICAgICAgICAgICAgICAgICAgfCAgMSArCj4gIDQgZmlsZXMgY2hhbmdlZCwg MjYgaW5zZXJ0aW9ucygrKQo+IAo+IGRpZmYgLS1naXQgYS9Eb2N1bWVudGF0aW9uL3VzZXJzcGFj ZS1hcGkvbWVkaWEvdjRsL2JpYmxpby5yc3QgYi9Eb2N1bWVudGF0aW9uL3VzZXJzcGFjZS1hcGkv bWVkaWEvdjRsL2JpYmxpby5yc3QKPiBpbmRleCAzNTY3NGVlYWUyMGQuLmRmM2U5NjNmYzU0ZiAx MDA2NDQKPiAtLS0gYS9Eb2N1bWVudGF0aW9uL3VzZXJzcGFjZS1hcGkvbWVkaWEvdjRsL2JpYmxp by5yc3QKPiArKysgYi9Eb2N1bWVudGF0aW9uL3VzZXJzcGFjZS1hcGkvbWVkaWEvdjRsL2JpYmxp by5yc3QKPiBAQCAtNDQ3LDMgKzQ0NywxMiBAQCBBVjEKPiAgOnRpdGxlOiAgICAgQVYxIEJpdHN0 cmVhbSAmIERlY29kaW5nIFByb2Nlc3MgU3BlY2lmaWNhdGlvbgo+ICAKPiAgOmF1dGhvcjogICAg UGV0ZXIgZGUgUml2YXosIEFyZ29uIERlc2lnbiBMdGQsIEphY2sgSGF1Z2h0b24sIEFyZ29uIERl c2lnbiBMdGQKPiArCj4gKy4uIF93ZWJwOgo+ICsKPiArV0VCUAo+ICs9PT09Cj4gKwo+ICs6dGl0 bGU6ICAgICBXRUJQIHBpY3R1cmUgQml0c3RyZWFtICYgRGVjb2RpbmcgUHJvY2VzcyBTcGVjaWZp Y2F0aW9uCj4gKwo+ICs6YXV0aG9yOiAgICBHb29nbGUgKGh0dHBzOi8vZGV2ZWxvcGVycy5nb29n bGUuY29tL3NwZWVkL3dlYnApCj4gZGlmZiAtLWdpdCBhL0RvY3VtZW50YXRpb24vdXNlcnNwYWNl LWFwaS9tZWRpYS92NGwvcGl4Zm10LWNvbXByZXNzZWQucnN0IGIvRG9jdW1lbnRhdGlvbi91c2Vy c3BhY2UtYXBpL21lZGlhL3Y0bC9waXhmbXQtY29tcHJlc3NlZC5yc3QKPiBpbmRleCA4MDZlZDcz YWM0NzQuLmU2NjRlNzBiMDYxOSAxMDA2NDQKPiAtLS0gYS9Eb2N1bWVudGF0aW9uL3VzZXJzcGFj ZS1hcGkvbWVkaWEvdjRsL3BpeGZtdC1jb21wcmVzc2VkLnJzdAo+ICsrKyBiL0RvY3VtZW50YXRp b24vdXNlcnNwYWNlLWFwaS9tZWRpYS92NGwvcGl4Zm10LWNvbXByZXNzZWQucnN0Cj4gQEAgLTE2 OSw2ICsxNjksMjEgQEAgQ29tcHJlc3NlZCBGb3JtYXRzCj4gIAl0aGlzIHBpeGVsIGZvcm1hdC4g VGhlIG91dHB1dCBidWZmZXIgbXVzdCBjb250YWluIHRoZSBhcHByb3ByaWF0ZSBudW1iZXIKPiAg CW9mIG1hY3JvYmxvY2tzIHRvIGRlY29kZSBhIGZ1bGwgY29ycmVzcG9uZGluZyBmcmFtZSB0byB0 aGUgbWF0Y2hpbmcKPiAgCWNhcHR1cmUgYnVmZmVyLgo+ICsgICAgKiAuLiBfVjRMMi1QSVgtRk1U LVdFQlAtRlJBTUU6Cj4gKwo+ICsgICAgICAtIGBgVjRMMl9QSVhfRk1UX1dFQlBfRlJBTUVgYAo+ ICsgICAgICAtICdXRUJQJwo+ICsgICAgICAtIFdFQlAgVlA4IHBhcnNlZCBmcmFtZSwgZXhjbHVk aW5nIFdFQlAgUklGRiBoZWFkZXIsIGtlZXBpbmcgb25seSB0aGUgVlA4Cj4gKwliaXN0cmVhbSBp bmNsdWRpbmcgdGhlIGZyYW1lIGhlYWRlciwgYXMgZXh0cmFjdGVkIGZyb20gdGhlIGNvbnRhaW5l ci4KPiArCVRoaXMgZm9ybWF0IGlzIGFkYXB0ZWQgZm9yIHN0YXRlbGVzcyB2aWRlbyBkZWNvZGVy cyB0aGF0IGltcGxlbWVudCBhCj4gKwlXRUJQIHBpcGVsaW5lIHdpdGggdGhlIDpyZWY6YHN0YXRl bGVzc19kZWNvZGVyYC4KPiArCU1ldGFkYXRhIGFzc29jaWF0ZWQgd2l0aCB0aGUgZnJhbWUgdG8g ZGVjb2RlIGlzIHJlcXVpcmVkIHRvIGJlIHBhc3NlZAo+ICsJdGhyb3VnaCB0aGUgYGBWNEwyX0NJ RF9TVEFURUxFU1NfVlA4X0ZSQU1FYGAgY29udHJvbC4KPiArCVNlZSB0aGUgOnJlZjpgYXNzb2Np YXRlZCBDb2RlYyBDb250cm9sIElEcyA8djRsMi1jb2RlYy1zdGF0ZWxlc3MtdnA4PmAuCj4gKwlF eGFjdGx5IG9uZSBvdXRwdXQgYW5kIG9uZSBjYXB0dXJlIGJ1ZmZlciBtdXN0IGJlIHByb3ZpZGVk IGZvciB1c2Ugd2l0aAo+ICsJdGhpcyBwaXhlbCBmb3JtYXQuIFRoZSBvdXRwdXQgYnVmZmVyIG11 c3QgY29udGFpbiB0aGUgYXBwcm9wcmlhdGUgbnVtYmVyCj4gKwlvZiBtYWNyb2Jsb2NrcyB0byBk ZWNvZGUgYSBmdWxsIGNvcnJlc3BvbmRpbmcgZnJhbWUgdG8gdGhlIG1hdGNoaW5nCj4gKwljYXB0 dXJlIGJ1ZmZlci4KPiAgCj4gICAgICAqIC4uIF9WNEwyLVBJWC1GTVQtVlA5Ogo+ICAKPiBkaWZm IC0tZ2l0IGEvZHJpdmVycy9tZWRpYS92NGwyLWNvcmUvdjRsMi1pb2N0bC5jIGIvZHJpdmVycy9t ZWRpYS92NGwyLWNvcmUvdjRsMi1pb2N0bC5jCj4gaW5kZXggMDMwNGRhYTg0NzFkLi5lMmZmMDNk MGQ3NzMgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9tZWRpYS92NGwyLWNvcmUvdjRsMi1pb2N0bC5j Cj4gKysrIGIvZHJpdmVycy9tZWRpYS92NGwyLWNvcmUvdjRsMi1pb2N0bC5jCj4gQEAgLTE1MDEs NiArMTUwMSw3IEBAIHN0YXRpYyB2b2lkIHY0bF9maWxsX2ZtdGRlc2Moc3RydWN0IHY0bDJfZm10 ZGVzYyAqZm10KQo+ICAJCWNhc2UgVjRMMl9QSVhfRk1UX1ZDMV9BTk5FWF9MOglkZXNjciA9ICJW Qy0xIChTTVBURSA0MTJNIEFubmV4IEwpIjsgYnJlYWs7Cj4gIAkJY2FzZSBWNEwyX1BJWF9GTVRf VlA4OgkJZGVzY3IgPSAiVlA4IjsgYnJlYWs7Cj4gIAkJY2FzZSBWNEwyX1BJWF9GTVRfVlA4X0ZS QU1FOiAgICBkZXNjciA9ICJWUDggRnJhbWUiOyBicmVhazsKPiArCQljYXNlIFY0TDJfUElYX0ZN VF9XRUJQX0ZSQU1FOiAgICBkZXNjciA9ICJXRUJQIFZQOCBGcmFtZSI7IGJyZWFrOwo+ICAJCWNh c2UgVjRMMl9QSVhfRk1UX1ZQOToJCWRlc2NyID0gIlZQOSI7IGJyZWFrOwo+ICAJCWNhc2UgVjRM Ml9QSVhfRk1UX1ZQOV9GUkFNRTogICAgZGVzY3IgPSAiVlA5IEZyYW1lIjsgYnJlYWs7Cj4gIAkJ Y2FzZSBWNEwyX1BJWF9GTVRfSEVWQzoJCWRlc2NyID0gIkhFVkMiOyBicmVhazsgLyogYWthIEgu MjY1ICovCj4gZGlmZiAtLWdpdCBhL2luY2x1ZGUvdWFwaS9saW51eC92aWRlb2RldjIuaCBiL2lu Y2x1ZGUvdWFwaS9saW51eC92aWRlb2RldjIuaAo+IGluZGV4IGU3YzRkY2UzOTAwNy4uMDlmZmYy NjllODUyIDEwMDY0NAo+IC0tLSBhL2luY2x1ZGUvdWFwaS9saW51eC92aWRlb2RldjIuaAo+ICsr KyBiL2luY2x1ZGUvdWFwaS9saW51eC92aWRlb2RldjIuaAo+IEBAIC03NTcsNiArNzU3LDcgQEAg c3RydWN0IHY0bDJfcGl4X2Zvcm1hdCB7Cj4gICNkZWZpbmUgVjRMMl9QSVhfRk1UX1ZDMV9BTk5F WF9MIHY0bDJfZm91cmNjKCdWJywgJ0MnLCAnMScsICdMJykgLyogU01QVEUgNDIxTSBBbm5leCBM IGNvbXBsaWFudCBzdHJlYW0gKi8KPiAgI2RlZmluZSBWNEwyX1BJWF9GTVRfVlA4ICAgICAgdjRs Ml9mb3VyY2MoJ1YnLCAnUCcsICc4JywgJzAnKSAvKiBWUDggKi8KPiAgI2RlZmluZSBWNEwyX1BJ WF9GTVRfVlA4X0ZSQU1FIHY0bDJfZm91cmNjKCdWJywgJ1AnLCAnOCcsICdGJykgLyogVlA4IHBh cnNlZCBmcmFtZSAqLwo+ICsjZGVmaW5lIFY0TDJfUElYX0ZNVF9XRUJQX0ZSQU1FIHY0bDJfZm91 cmNjKCdXJywgJ0InLCAnUCcsICdGJykgLyogV0VCUCBWUDggcGFyc2VkIGZyYW1lICovCj4gICNk ZWZpbmUgVjRMMl9QSVhfRk1UX1ZQOSAgICAgIHY0bDJfZm91cmNjKCdWJywgJ1AnLCAnOScsICcw JykgLyogVlA5ICovCj4gICNkZWZpbmUgVjRMMl9QSVhfRk1UX1ZQOV9GUkFNRSB2NGwyX2ZvdXJj YygnVicsICdQJywgJzknLCAnRicpIC8qIFZQOSBwYXJzZWQgZnJhbWUgKi8KPiAgI2RlZmluZSBW NEwyX1BJWF9GTVRfSEVWQyAgICAgdjRsMl9mb3VyY2MoJ0gnLCAnRScsICdWJywgJ0MnKSAvKiBI RVZDIGFrYSBILjI2NSAqLwo+IC0tIAo+IDIuMjUuMQo+IAo+IAo+IF9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCj4gTGludXgtcm9ja2NoaXAgbWFpbGluZyBs aXN0Cj4gTGludXgtcm9ja2NoaXBAbGlzdHMuaW5mcmFkZWFkLm9yZwo+IGh0dHA6Ly9saXN0cy5p bmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtcm9ja2NoaXAKCi0tIApMaW5rIE1h dXZlCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpMaW51 eC1yb2NrY2hpcCBtYWlsaW5nIGxpc3QKTGludXgtcm9ja2NoaXBAbGlzdHMuaW5mcmFkZWFkLm9y ZwpodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LXJvY2tj aGlwCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from luna.linkmauve.fr (luna.linkmauve.fr [82.65.109.163]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 95B0A46B8; Wed, 20 Nov 2024 13:47:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=82.65.109.163 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732110444; cv=none; b=SXoVWxxTCSVy2iDAE7NBWis5zn/5xTrfPZ4VGMW9St5C2TqEAxWWzBKw4M7ihJg20jJTFM4WDYMkQJAUqs2K/SbzLP61aZSlG31wVbZBylU+Ce+1+Kr+hXcd5zLwGi2Gb3EhOdfXdQePf4ifWRu2m3dtJEscsd0Y+zVHYI0ibZI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732110444; c=relaxed/simple; bh=GP/o+Od12FOpsHcCB1ShPWRmit3YqkCpFr4TAa12ps4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=scO8cR0FuRpbVMX1eQbXAn4AklWyfYIcIHKm9cVLRqO9fkUQPt9mqFYa21AZu+wTbYqc6SEHqK0/cQXeUkohuaTVxwGN8+lUOZjrP4vR1YBop2NGqaRnTRIHQeHpKILOMcf0KdAQFacgV7mfnnFTJBA9TTDaLTq3w0nWRba2P90= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=linkmauve.fr; spf=pass smtp.mailfrom=linkmauve.fr; arc=none smtp.client-ip=82.65.109.163 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=linkmauve.fr Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linkmauve.fr Received: by luna.linkmauve.fr (Postfix, from userid 1000) id D522618559CF; Wed, 20 Nov 2024 14:42:03 +0100 (CET) Date: Wed, 20 Nov 2024 14:42:03 +0100 From: Link Mauve To: Hugues Fruchet Cc: Mauro Carvalho Chehab , Ezequiel Garcia , Philipp Zabel , Hans Verkuil , Fritz Koenig , Sebastian Fricke , Daniel Almeida , Andrzej Pietrasiewicz , Nicolas Dufresne , Benjamin Gaignard , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com Subject: Re: [PATCH v2 1/3] media: uapi: add WebP uAPI Message-ID: References: <20241120110105.244413-1-hugues.fruchet@foss.st.com> <20241120110105.244413-2-hugues.fruchet@foss.st.com> Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20241120110105.244413-2-hugues.fruchet@foss.st.com> Jabber-ID: linkmauve@linkmauve.fr Hi, On Wed, Nov 20, 2024 at 12:01:03PM +0100, Hugues Fruchet wrote: > This patch adds the WebP picture decoding kernel uAPI. > > This design is based on currently available VP8 API implementation and > aims to support the development of WebP stateless video codecs > on Linux. Why do you need this new uAPI exactly? The WebP format is more complex than the simple 'VP8 ' format, the 'VP8X' fourcc for instance is an animated format which may contain multiple VP8 keyframes, or an alpha side channel, and just like any other video container we queue each VP8 frame separately in V4L2 for decoding, not the whole file. In Onix[1] I parse the WebP header and pass the raw VP8 frame to V4L2 without the RIFF around it. So I’d rather NACK this patch, I don’t think it’s a good idea to hardcode the simplest version of the WebP container in the uAPI, to the detriment of all other possible WebP files. [1] git clone https://git.linkmauve.fr/onix.git/ > > Signed-off-by: Hugues Fruchet > --- > Documentation/userspace-api/media/v4l/biblio.rst | 9 +++++++++ > .../userspace-api/media/v4l/pixfmt-compressed.rst | 15 +++++++++++++++ > drivers/media/v4l2-core/v4l2-ioctl.c | 1 + > include/uapi/linux/videodev2.h | 1 + > 4 files changed, 26 insertions(+) > > diff --git a/Documentation/userspace-api/media/v4l/biblio.rst b/Documentation/userspace-api/media/v4l/biblio.rst > index 35674eeae20d..df3e963fc54f 100644 > --- a/Documentation/userspace-api/media/v4l/biblio.rst > +++ b/Documentation/userspace-api/media/v4l/biblio.rst > @@ -447,3 +447,12 @@ AV1 > :title: AV1 Bitstream & Decoding Process Specification > > :author: Peter de Rivaz, Argon Design Ltd, Jack Haughton, Argon Design Ltd > + > +.. _webp: > + > +WEBP > +==== > + > +:title: WEBP picture Bitstream & Decoding Process Specification > + > +:author: Google (https://developers.google.com/speed/webp) > diff --git a/Documentation/userspace-api/media/v4l/pixfmt-compressed.rst b/Documentation/userspace-api/media/v4l/pixfmt-compressed.rst > index 806ed73ac474..e664e70b0619 100644 > --- a/Documentation/userspace-api/media/v4l/pixfmt-compressed.rst > +++ b/Documentation/userspace-api/media/v4l/pixfmt-compressed.rst > @@ -169,6 +169,21 @@ Compressed Formats > this pixel format. The output buffer must contain the appropriate number > of macroblocks to decode a full corresponding frame to the matching > capture buffer. > + * .. _V4L2-PIX-FMT-WEBP-FRAME: > + > + - ``V4L2_PIX_FMT_WEBP_FRAME`` > + - 'WEBP' > + - WEBP VP8 parsed frame, excluding WEBP RIFF header, keeping only the VP8 > + bistream including the frame header, as extracted from the container. > + This format is adapted for stateless video decoders that implement a > + WEBP pipeline with the :ref:`stateless_decoder`. > + Metadata associated with the frame to decode is required to be passed > + through the ``V4L2_CID_STATELESS_VP8_FRAME`` control. > + See the :ref:`associated Codec Control IDs `. > + Exactly one output and one capture buffer must be provided for use with > + this pixel format. The output buffer must contain the appropriate number > + of macroblocks to decode a full corresponding frame to the matching > + capture buffer. > > * .. _V4L2-PIX-FMT-VP9: > > diff --git a/drivers/media/v4l2-core/v4l2-ioctl.c b/drivers/media/v4l2-core/v4l2-ioctl.c > index 0304daa8471d..e2ff03d0d773 100644 > --- a/drivers/media/v4l2-core/v4l2-ioctl.c > +++ b/drivers/media/v4l2-core/v4l2-ioctl.c > @@ -1501,6 +1501,7 @@ static void v4l_fill_fmtdesc(struct v4l2_fmtdesc *fmt) > case V4L2_PIX_FMT_VC1_ANNEX_L: descr = "VC-1 (SMPTE 412M Annex L)"; break; > case V4L2_PIX_FMT_VP8: descr = "VP8"; break; > case V4L2_PIX_FMT_VP8_FRAME: descr = "VP8 Frame"; break; > + case V4L2_PIX_FMT_WEBP_FRAME: descr = "WEBP VP8 Frame"; break; > case V4L2_PIX_FMT_VP9: descr = "VP9"; break; > case V4L2_PIX_FMT_VP9_FRAME: descr = "VP9 Frame"; break; > case V4L2_PIX_FMT_HEVC: descr = "HEVC"; break; /* aka H.265 */ > diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h > index e7c4dce39007..09fff269e852 100644 > --- a/include/uapi/linux/videodev2.h > +++ b/include/uapi/linux/videodev2.h > @@ -757,6 +757,7 @@ struct v4l2_pix_format { > #define V4L2_PIX_FMT_VC1_ANNEX_L v4l2_fourcc('V', 'C', '1', 'L') /* SMPTE 421M Annex L compliant stream */ > #define V4L2_PIX_FMT_VP8 v4l2_fourcc('V', 'P', '8', '0') /* VP8 */ > #define V4L2_PIX_FMT_VP8_FRAME v4l2_fourcc('V', 'P', '8', 'F') /* VP8 parsed frame */ > +#define V4L2_PIX_FMT_WEBP_FRAME v4l2_fourcc('W', 'B', 'P', 'F') /* WEBP VP8 parsed frame */ > #define V4L2_PIX_FMT_VP9 v4l2_fourcc('V', 'P', '9', '0') /* VP9 */ > #define V4L2_PIX_FMT_VP9_FRAME v4l2_fourcc('V', 'P', '9', 'F') /* VP9 parsed frame */ > #define V4L2_PIX_FMT_HEVC v4l2_fourcc('H', 'E', 'V', 'C') /* HEVC aka H.265 */ > -- > 2.25.1 > > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip -- Link Mauve