From nobody Wed Oct 8 20:53:18 2025 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 B3B5329825B; Tue, 24 Jun 2025 12:29:44 +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=1750768186; cv=none; b=Znl3DVrjg5QkccFB2Xqwc5Fa2EPKmeKoH9dHVNFJZKqOV7a8l2Lkt/4l/0E0sroxe0XqvwNCkVjCoyxYkBXS/bmbvIkrVolQU5gyPrUdbWzPYDnuuFec7b+T/lzQtqs+TCpLdH1TrOtPezaCCzDzm907hGH+ZrKuibk1GPwi200= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750768186; c=relaxed/simple; bh=8WAPJMaT3RPf5H8MPe+6Y+/A6cCV8GHnSu9zGp/405I=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=o4JtKFoUs+4LPfwOKnie47BnZTDM9vV3qHX5ytI0+3qJIebJAgw/bMjGvntjxW+Kxg3s/HuHgXHECL5z0wQWSuFsDRo/RALY8lP1bYEHIcuvUUjb3CVebYwJGFUCgd7VgZDsbhoaqN5dyuexDVLL9rO/H6pUn/OQ4s0ZG5dVkK0= 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=qctFoLm4; 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="qctFoLm4" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1750768182; bh=8WAPJMaT3RPf5H8MPe+6Y+/A6cCV8GHnSu9zGp/405I=; h=From:To:Cc:Subject:Date:From; b=qctFoLm4S1HCsLWt+Nk4SlF4ZMzxRq0J4jEg964rH2UaOfmjM/0VmQirKWRj4WuOA huRpTjdECUX+l4o6iHyG+0OXYZdHwNFKJeAC9xD1euHK0orJ+VHKk7U9zyiNBRlyP6 UXzKdJq0WKtLNzcwNbqErR5o+SFtReDu6R1IJApUmYTVEX/7HR+2/8bQtBOz2tPw0n qu0DDoTIg5SOKyEUIW9+BQYniSd7/4R8Xo0qfqkk0SNe0cvt2/9Hgw8tAM7tN37r/c nPFyHNW2XQzDOoFPN6rFJZTy8Cs4pmbkWcl3L79XmsL0tUCiGxoK669wiYhG1P3kiI aJiy31f9T0lmw== Received: from benjamin-XPS-13-9310.. (unknown [IPv6:2a01:e0a:120:3210:ed49:23ab:f92:d8a4]) (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: benjamin.gaignard) by bali.collaboradmins.com (Postfix) with ESMTPSA id 8B6F517E0FDB; Tue, 24 Jun 2025 14:29:42 +0200 (CEST) From: Benjamin Gaignard To: nicolas.dufresne@collabora.com, p.zabel@pengutronix.de, mchehab@kernel.org, hverkuil@xs4all.nl Cc: linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, kernel@collabora.com, Benjamin Gaignard Subject: [PATCH] media: verisilicon: Export only needed pixels formats. Date: Tue, 24 Jun 2025 14:29:38 +0200 Message-ID: <20250624122938.62004-1-benjamin.gaignard@collabora.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" When enumerating the pixels formats check if the context request to only export post-processed pixels formats. The exception is when V4L2_FMTDESC_FLAG_ENUM_ALL is set, we need to export all pixels formats. Signed-off-by: Benjamin Gaignard Fixes: bcd4f091cf1e ("media: verisilicon: Use V4L2_FMTDESC_FLAG_ENUM_ALL fl= ag") Reviewed-by: Nicolas Dufresne --- drivers/media/platform/verisilicon/hantro_v4l2.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/media/platform/verisilicon/hantro_v4l2.c b/drivers/med= ia/platform/verisilicon/hantro_v4l2.c index 7c3515cf7d64..7869faf921f4 100644 --- a/drivers/media/platform/verisilicon/hantro_v4l2.c +++ b/drivers/media/platform/verisilicon/hantro_v4l2.c @@ -222,6 +222,7 @@ static int vidioc_enum_fmt(struct file *file, void *pri= v, unsigned int num_fmts, i, j =3D 0; bool skip_mode_none, enum_all_formats; u32 index =3D f->index & ~V4L2_FMTDESC_FLAG_ENUM_ALL; + bool need_postproc =3D ctx->need_postproc; =20 /* * If the V4L2_FMTDESC_FLAG_ENUM_ALL flag is set, we want to enumerate all @@ -230,6 +231,9 @@ static int vidioc_enum_fmt(struct file *file, void *pri= v, enum_all_formats =3D !!(f->index & V4L2_FMTDESC_FLAG_ENUM_ALL); f->index =3D index; =20 + if (enum_all_formats) + need_postproc =3D HANTRO_AUTO_POSTPROC; + /* * When dealing with an encoder: * - on the capture side we want to filter out all MODE_NONE formats. @@ -242,7 +246,7 @@ static int vidioc_enum_fmt(struct file *file, void *pri= v, */ skip_mode_none =3D capture =3D=3D ctx->is_encoder; =20 - formats =3D hantro_get_formats(ctx, &num_fmts, HANTRO_AUTO_POSTPROC); + formats =3D hantro_get_formats(ctx, &num_fmts, need_postproc); for (i =3D 0; i < num_fmts; i++) { bool mode_none =3D formats[i].codec_mode =3D=3D HANTRO_MODE_NONE; fmt =3D &formats[i]; --=20 2.43.0