Re: [PATCH RFC 00/10] media: qcom: venus: add MSM8939 support

Erikas Bitovtas
2026-04-16T17:00:56Z

>> 3. MSM8939 supports HEVC decoding, however, as the patchset is written
>> now, it does not work. It can be enabled, however, it will result in
>> breakage of Venus for faulty MSM8916 firmwares, because the code
>> disabling HEVC for HFI v1 needs to be removed, and as per commit
>> c50cc6dc6c48 ("media: venus: hfi_parser: Ignore HEVC encoding for V1"),
>> this would break support for some MSM8916 devices. What could be the
>> best way to work around this?
>
> if (!device_is_compatible(core->dev, "qcom,msm8939-venus"))?
>
> Also, you mentioned HEVC *de*coding, while the commit you pointed to
> disables *en*coding (decoding had been already disabled prior to that
> commit)
>
> Konrad

From the commit message I assumed HEVC decoding had already been
disabled for the same reasons encoding was - faulty firmware reporting
codecs it doesn't actually support.