Re: [PATCH v8 00/14] Peripheral Image Loader support for Qualcomm SoCs running Linux host at EL2

[Bryan O'Donoghue <bod@kernel.org>]

On 21/11/2025 11:37, Mukesh Ojha wrote:
>> Sorry.
>>
>> Did we actually come up with a cogent reason to omit the video firmware
>> loading here ?
>>
>> AFAIU it is required for Lemans and Glymur - leaving it out is blocking
>> getting video stuff done and storing up trouble.
>>
>> What exactly is the blockage - is it something you want help with ?
> I replied to you here[1] and given my reason..till something concluded on
> "multi-cell IOMMU[2]", I can not add video and block what is working
> already.
> 
> [1]
> https://lore.kernel.org/lkml/20251105081421.f6j7ks5bd4dfgr67@hu-mojha- 
> hyd.qualcomm.com/

Why though ?

You are mixing together the issue of multiple SIDs and the original 
loading of firmware which could easily reuse the venus method of

&iris {
	video-firmware {
		iommus = <&apss_smmu hex>;
	};
};

That binding got dropped because it was unused in Iris.

https://lore.kernel.org/lkml/05d40a3b-cc13-b704-cac7-0ecbeea0e59d@quicinc.com/

I still fail to see why we are waiting for multi-cell IOMMU to land, 
when it is expected to and what the VPU enablement story is upstream in 
the meantime.

Blocked it seems.

---
bod