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 69849D149DB 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=a/jetISU5AgjIrfGMmRmET0OHPgDh3JtFM97KiTv4ZY=; b=SUNcD8bFGLRESE v3aLFah9Fl5J7QLdnDcMecszp1t8xaVLmSQH0vo6qyIbex1eyxbnv/jUkGYsJAdvNn5sjEd6cdKEC IMmbuE85cK6kNXn/eddKqgB0e21gJQDlZrpe3ww9WZ1agOssyekY2N/HpwgASmgdvbkaV4T6ym6N8 PkjWVLzi67mkCTiG5XC90kgWMV3RhpFFs7di8TZZrEMWqTuTact2nddtkNLAKR5FSBQZoPtPUExdN eyTkCmjwA+sTbxqAqqqiWAZT7LUjf5C++q/h7lUe5hFb27C15NpTpkBbxjA9DfhHnOg7e9IwLPySo snG3k6iccwBeLxbyAZiA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t4P0E-00000004nPG-3CqM; Fri, 25 Oct 2024 18:25:38 +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 1t4OOq-00000004fbN-3f6K for linux-rockchip@lists.infradead.org; Fri, 25 Oct 2024 17:47:02 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1729878418; bh=ozBIpgyxnAgCBKNUleMXiK02jDu4q4LAhe4fTs4O04s=; h=Subject:From:To:Cc:Date:In-Reply-To:References:From; b=iUg44kWBzy9MSSIzXccTiM8CT+DmPeUOKRVz7wQSt6tXpAV8XHJyQsUgsBzcJMnrl H2wUZKvZqhgtu4gEjEq5KdV9Uu1DBV6gHCy04fMhzVm1of3mNa4NMGY+2HUbDULdW9 xFEXp2AoyMZUxDeTtHC4LByH0c2gdHO7XSutsFlHIRzODNIrcBNAWxjH3U1rkuwqye gujMmwRC7Ckl2G+l6HL4G9veMPSN9jM0QCowsOYc8GTEoqkKmo0RCqGQJQsuMRwLSy O1fctHQo8IOTsgglaa0sk2pQsQXhu1OaLnC0ZBS48/rbMPInMEPBB216adib9YQKFs dtLZ2T11X37QQ== 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) server-digest SHA256) (No client certificate requested) (Authenticated sender: nicolas) by bali.collaboradmins.com (Postfix) with ESMTPSA id B8C4817E36B5; Fri, 25 Oct 2024 19:46:56 +0200 (CEST) Message-ID: Subject: Re: [PATCH v6 11/11] media: rkvdec: Fix enumerate frame sizes From: Nicolas Dufresne To: Jonas Karlman , Sebastian Fricke , Ezequiel Garcia , Mauro Carvalho Chehab , Greg Kroah-Hartman , Hans Verkuil , Boris Brezillon Cc: 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, Mauro Carvalho Chehab Date: Fri, 25 Oct 2024 13:46:55 -0400 In-Reply-To: <20240909192522.1076704-12-jonas@kwiboo.se> References: <20240909192522.1076704-1-jonas@kwiboo.se> <20240909192522.1076704-12-jonas@kwiboo.se> 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_104701_101647_59A4FD11 X-CRM114-Status: GOOD ( 16.02 ) 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 TGUgbHVuZGkgMDkgc2VwdGVtYnJlIDIwMjQgw6AgMTk6MjUgKzAwMDAsIEpvbmFzIEthcmxtYW4g YSDDqWNyaXTCoDoKPiBUaGUgVklESU9DX0VOVU1fRlJBTUVTSVpFUyBpb2N0bCBzaG91bGQgcmV0 dXJuIGFsbCBmcmFtZSBzaXplcyAoaS4gZS4KPiB3aWR0aCBhbmQgaGVpZ2h0IGluIHBpeGVscykg dGhhdCB0aGUgZGV2aWNlIHN1cHBvcnRzIGZvciB0aGUgZ2l2ZW4gcGl4ZWwKPiBmb3JtYXQuCj4g Cj4gRm9yIGNvZGVkIGZvcm1hdCByZXR1cm5pbmcgdGhlIGZyYW1lIHNpemUgdXNlZCB0byBlbmZv cmNlIEhXIGFsaWdubWVudAo+IHJlcXVpcmVtZW50cyBmb3IgQ0FQVFVSRSBidWZmZXJzIGRvZXMg bm90IG1ha2UgZnVsbHkgc2Vuc2UuCj4gCj4gSW5zdGVhZCwgc2lnbmFsIGFwcGxpY2F0aW9ucyB3 aGF0IHRoZSBtYXhpbXVtIGZyYW1lIHNpemUgdGhhdCBpcwo+IHN1cHBvcnRlZCBieSB0aGUgSFcg ZGVjb2RlciB1c2luZyBhIGZyYW1lIHNpemUgb2YgY29udGludW91cyB0eXBlLgo+IAo+IEZpeGVz OiBjZDMzYzgzMDQ0OGIgKCJtZWRpYTogcmt2ZGVjOiBBZGQgdGhlIHJrdmRlYyBkcml2ZXIiKQo+ IFN1Z2dlc3RlZC1ieTogQWxleCBCZWUgPGtuYWVyemNoZUBnbWFpbC5jb20+Cj4gU2lnbmVkLW9m Zi1ieTogSm9uYXMgS2FybG1hbiA8am9uYXNAa3dpYm9vLnNlPgoKUmV2aWV3ZWQtYnk6IE5pY29s YXMgRHVmcmVzbmUgPG5pY29sYXMuZHVmcmVzbmVAY29sbGFib3JhLmNvbT4KCj4gLS0tCj4gdjY6 Cj4gLSBOZXcgcGF0Y2gKPiAKPiBXaXRoIHRoaXMgY2hhbmdlIEZGbXBlZyBWNEwyIFJlcXVlc3Qg QVBJIGh3YWNjZWxzIGNhbiBpbXBsZW1lbnQgYSBzdHJpY3QKPiBjaGVjayBpZiBmcmFtZSBzaXpl IGlzIHN1cHBvcnRlZCBieSB0aGUgdmlkZW8gZGV2aWNlOgo+IGh0dHBzOi8vZmZtcGVnLm9yZy9w aXBlcm1haWwvZmZtcGVnLWRldmVsLzIwMjQtQXVndXN0LzMzMjAzNy5odG1sCj4gLS0tCj4gIGRy aXZlcnMvc3RhZ2luZy9tZWRpYS9ya3ZkZWMvcmt2ZGVjLmMgfCAxMCArKysrKysrKy0tCj4gIDEg ZmlsZSBjaGFuZ2VkLCA4IGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCj4gCj4gZGlmZiAt LWdpdCBhL2RyaXZlcnMvc3RhZ2luZy9tZWRpYS9ya3ZkZWMvcmt2ZGVjLmMgYi9kcml2ZXJzL3N0 YWdpbmcvbWVkaWEvcmt2ZGVjL3JrdmRlYy5jCj4gaW5kZXggYzhjMTRmMzVhYzQ0Li45MDAyZWIz YTU5ZTUgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9zdGFnaW5nL21lZGlhL3JrdmRlYy9ya3ZkZWMu Ywo+ICsrKyBiL2RyaXZlcnMvc3RhZ2luZy9tZWRpYS9ya3ZkZWMvcmt2ZGVjLmMKPiBAQCAtMzM0 LDggKzMzNCwxNCBAQCBzdGF0aWMgaW50IHJrdmRlY19lbnVtX2ZyYW1lc2l6ZXMoc3RydWN0IGZp bGUgKmZpbGUsIHZvaWQgKnByaXYsCj4gIAlpZiAoIWZtdCkKPiAgCQlyZXR1cm4gLUVJTlZBTDsK PiAgCj4gLQlmc2l6ZS0+dHlwZSA9IFY0TDJfRlJNU0laRV9UWVBFX1NURVBXSVNFOwo+IC0JZnNp emUtPnN0ZXB3aXNlID0gZm10LT5mcm1zaXplOwo+ICsJZnNpemUtPnR5cGUgPSBWNEwyX0ZSTVNJ WkVfVFlQRV9DT05USU5VT1VTOwo+ICsJZnNpemUtPnN0ZXB3aXNlLm1pbl93aWR0aCA9IDE7Cj4g Kwlmc2l6ZS0+c3RlcHdpc2UubWF4X3dpZHRoID0gZm10LT5mcm1zaXplLm1heF93aWR0aDsKPiAr CWZzaXplLT5zdGVwd2lzZS5zdGVwX3dpZHRoID0gMTsKPiArCWZzaXplLT5zdGVwd2lzZS5taW5f aGVpZ2h0ID0gMTsKPiArCWZzaXplLT5zdGVwd2lzZS5tYXhfaGVpZ2h0ID0gZm10LT5mcm1zaXpl Lm1heF9oZWlnaHQ7Cj4gKwlmc2l6ZS0+c3RlcHdpc2Uuc3RlcF9oZWlnaHQgPSAxOwo+ICsKPiAg CXJldHVybiAwOwo+ICB9Cj4gIAoKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fCkxpbnV4LXJvY2tjaGlwIG1haWxpbmcgbGlzdApMaW51eC1yb2NrY2hpcEBs aXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlz dGluZm8vbGludXgtcm9ja2NoaXAK 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 988DC14D6EF for ; Fri, 25 Oct 2024 17:47:00 +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=1729878423; cv=none; b=b9xcpXQeG2CUMIEl9AYkBFNR1F+wheEy9mrd1a+vBYpCdO79BU3bRF31qxi/g+TbpIxrfySdy9uQJC6HFfQ7w5QKfYzaSxEbpOUT4KW6lnPd8giGU0hEpJqfLG6ylHLljleHRoOtvAeB05XE4ebVcW4N5xbDTZgvXxSdcyrl5n4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729878423; c=relaxed/simple; bh=ozBIpgyxnAgCBKNUleMXiK02jDu4q4LAhe4fTs4O04s=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=MpRx/qwNujLgFSUQIxk30w8I7UlM5U1bJznP78ABiEq5T+rzCwCdhbmK3LjAHDTeL07RXFBoEinZ1J5H5t3fe6mA0MQsrxVZqJGWOXQxxfaKHjIbeqvB1uCjjqtoMh9wcuj5ZJFMuDcvGj5PSn0FzQHvTrZ/QyqN1MXPBQ6AsKU= 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=iUg44kWB; 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="iUg44kWB" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1729878418; bh=ozBIpgyxnAgCBKNUleMXiK02jDu4q4LAhe4fTs4O04s=; h=Subject:From:To:Cc:Date:In-Reply-To:References:From; b=iUg44kWBzy9MSSIzXccTiM8CT+DmPeUOKRVz7wQSt6tXpAV8XHJyQsUgsBzcJMnrl H2wUZKvZqhgtu4gEjEq5KdV9Uu1DBV6gHCy04fMhzVm1of3mNa4NMGY+2HUbDULdW9 xFEXp2AoyMZUxDeTtHC4LByH0c2gdHO7XSutsFlHIRzODNIrcBNAWxjH3U1rkuwqye gujMmwRC7Ckl2G+l6HL4G9veMPSN9jM0QCowsOYc8GTEoqkKmo0RCqGQJQsuMRwLSy O1fctHQo8IOTsgglaa0sk2pQsQXhu1OaLnC0ZBS48/rbMPInMEPBB216adib9YQKFs dtLZ2T11X37QQ== 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) server-digest SHA256) (No client certificate requested) (Authenticated sender: nicolas) by bali.collaboradmins.com (Postfix) with ESMTPSA id B8C4817E36B5; Fri, 25 Oct 2024 19:46:56 +0200 (CEST) Message-ID: Subject: Re: [PATCH v6 11/11] media: rkvdec: Fix enumerate frame sizes From: Nicolas Dufresne To: Jonas Karlman , Sebastian Fricke , Ezequiel Garcia , Mauro Carvalho Chehab , Greg Kroah-Hartman , Hans Verkuil , Boris Brezillon Cc: 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, Mauro Carvalho Chehab Date: Fri, 25 Oct 2024 13:46:55 -0400 In-Reply-To: <20240909192522.1076704-12-jonas@kwiboo.se> References: <20240909192522.1076704-1-jonas@kwiboo.se> <20240909192522.1076704-12-jonas@kwiboo.se> 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 lundi 09 septembre 2024 à 19:25 +0000, Jonas Karlman a écrit : > The VIDIOC_ENUM_FRAMESIZES ioctl should return all frame sizes (i. e. > width and height in pixels) that the device supports for the given pixel > format. > > For coded format returning the frame size used to enforce HW alignment > requirements for CAPTURE buffers does not make fully sense. > > Instead, signal applications what the maximum frame size that is > supported by the HW decoder using a frame size of continuous type. > > Fixes: cd33c830448b ("media: rkvdec: Add the rkvdec driver") > Suggested-by: Alex Bee > Signed-off-by: Jonas Karlman Reviewed-by: Nicolas Dufresne > --- > v6: > - New patch > > With this change FFmpeg V4L2 Request API hwaccels can implement a strict > check if frame size is supported by the video device: > https://ffmpeg.org/pipermail/ffmpeg-devel/2024-August/332037.html > --- > drivers/staging/media/rkvdec/rkvdec.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/media/rkvdec/rkvdec.c b/drivers/staging/media/rkvdec/rkvdec.c > index c8c14f35ac44..9002eb3a59e5 100644 > --- a/drivers/staging/media/rkvdec/rkvdec.c > +++ b/drivers/staging/media/rkvdec/rkvdec.c > @@ -334,8 +334,14 @@ static int rkvdec_enum_framesizes(struct file *file, void *priv, > if (!fmt) > return -EINVAL; > > - fsize->type = V4L2_FRMSIZE_TYPE_STEPWISE; > - fsize->stepwise = fmt->frmsize; > + fsize->type = V4L2_FRMSIZE_TYPE_CONTINUOUS; > + fsize->stepwise.min_width = 1; > + fsize->stepwise.max_width = fmt->frmsize.max_width; > + fsize->stepwise.step_width = 1; > + fsize->stepwise.min_height = 1; > + fsize->stepwise.max_height = fmt->frmsize.max_height; > + fsize->stepwise.step_height = 1; > + > return 0; > } >