From: Bryan O'Donoghue <bod@kernel.org>
To: "Erikas Bitovtas" <xerikasxx@gmail.com>,
"Vikash Garodia" <vikash.garodia@oss.qualcomm.com>,
"Dikshita Agarwal" <dikshita.agarwal@oss.qualcomm.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"André Apitzsch" <git@apitzsch.eu>,
"Bjorn Andersson" <andersson@kernel.org>,
"Konrad Dybcio" <konradybcio@kernel.org>,
"Michael Turquette" <mturquette@baylibre.com>,
"Stephen Boyd" <sboyd@kernel.org>,
"Brian Masney" <bmasney@redhat.com>
Cc: linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-clk@vger.kernel.org, ~postmarketos/upstreaming@lists.sr.ht,
phone-devel@vger.kernel.org
Subject: Re: [PATCH v7 7/8] media: qcom: venus: add codec blacklist mechanism
Date: Fri, 15 May 2026 11:12:16 +0100 [thread overview]
Message-ID: <4b097256-8037-42f6-b6ff-50c2b4d4934c@kernel.org> (raw)
In-Reply-To: <20260514-msm8939-venus-rfc-v7-7-33c6c6fb9285@gmail.com>
On 13/05/2026 22:24, Erikas Bitovtas wrote:
> case HFI_VIDEO_CODEC_H263:
> @@ -684,6 +686,8 @@ static const struct venus_resources msm8916_res = {
> .vmem_addr = 0,
> .dma_mask = 0xddc00000 - 1,
> .fwname = "qcom/venus-1.8/venus.mbn",
> + .dec_codec_blacklist = HFI_VIDEO_CODEC_HEVC | HFI_VIDEO_CODEC_SPARK,
> + .enc_codec_blacklist = HFI_VIDEO_CODEC_HEVC,
This should be separate from the addition of the mechanism.
This is enumeration of a particular plaform and you might want for
example to revert the 8916 stuff in isolation without yanking the logic
that goes with it.
--
bod