Re: [PATCH 1/2] arm64: dts: qcom: agatti: Fix IOMMU DT properties

[Akhil P Oommen]

On 12/29/2025 12:42 PM, Sumit Garg wrote:
> From: Sumit Garg <sumit.garg@xxxxxxxxxxxxxxxx>
> 
> Fix IOMMU DT propeties for GPU, display and video peripherals via
> dropping SMMU stream IDs which relates to secure context bank.
> 
> This problem only surfaced when the Gunyah based firmware stack is
> ported on Agatti replacing the legacy QHEE based firmware stack. Assigning
> Linux kernel (HLOS) VMID to secure context bank stream IDs is treated
> as a fault by Gunyah hypervisor which were previously ignored by QHEE
> hypervisor.
> 
> The DT changes should be backwards compatible with legacy QHEE based
> firmware stack too.
> 
> Suggested-by: Prakash Gupta <guptap@xxxxxxxxxxxxxxxx>
> Signed-off-by: Sumit Garg <sumit.garg@xxxxxxxxxxxxxxxx>

Reviewed-by: Akhil P Oommen <akhilpo@xxxxxxxxxxxxxxxx>

-Akhil

> ---
>  arch/arm64/boot/dts/qcom/agatti.dtsi | 11 +++--------
>  1 file changed, 3 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/agatti.dtsi b/arch/arm64/boot/dts/qcom/agatti.dtsi
> index 8bf5c5583fc2..e705eb24160a 100644
> --- a/arch/arm64/boot/dts/qcom/agatti.dtsi
> +++ b/arch/arm64/boot/dts/qcom/agatti.dtsi
> @@ -1613,8 +1613,7 @@ gpu: gpu@5900000 {
>  					 &bimc SLAVE_EBI1 RPM_ALWAYS_TAG>;
>  			interconnect-names = "gfx-mem";
>  
> -			iommus = <&adreno_smmu 0 1>,
> -				 <&adreno_smmu 2 0>;
> +			iommus = <&adreno_smmu 0 1>;
>  			operating-points-v2 = <&gpu_opp_table>;
>  			power-domains = <&rpmpd QCM2290_VDDCX>;
>  			qcom,gmu = <&gmu_wrapper>;
> @@ -1895,8 +1894,7 @@ mdss: display-subsystem@5e00000 {
>  
>  			power-domains = <&dispcc MDSS_GDSC>;
>  
> -			iommus = <&apps_smmu 0x420 0x2>,
> -				 <&apps_smmu 0x421 0x0>;
> +			iommus = <&apps_smmu 0x420 0x2>;
>  			interconnects = <&mmrt_virt MASTER_MDP0 RPM_ALWAYS_TAG
>  					 &bimc SLAVE_EBI1 RPM_ALWAYS_TAG>,
>  					<&bimc MASTER_APPSS_PROC RPM_ALWAYS_TAG
> @@ -2339,10 +2337,7 @@ venus: video-codec@5a00000 {
>  
>  			memory-region = <&pil_video_mem>;
>  			iommus = <&apps_smmu 0x860 0x0>,
> -				 <&apps_smmu 0x880 0x0>,
> -				 <&apps_smmu 0x861 0x04>,
> -				 <&apps_smmu 0x863 0x0>,
> -				 <&apps_smmu 0x804 0xe0>;
> +				 <&apps_smmu 0x880 0x0>;
>  
>  			interconnects = <&mmnrt_virt MASTER_VIDEO_P0 RPM_ALWAYS_TAG
>  					 &bimc SLAVE_EBI1 RPM_ALWAYS_TAG>,