On Fri, Sep 15, 2023 at 10:36:11AM +0530, Gaurav Kohli wrote:
> Fix the apps iommu local address space range as per data sheet.
>
> Fixes: 327c0f5f2510 ("arm64: dts: qcom: msm8916: Sort nodes")
> Signed-off-by: Gaurav Kohli <quic_gkohli@xxxxxxxxxxx>
>
The above quoted commit it Fixes tag did not introduce the problem. This
is introduced in
6a6729f38436("arm64: dts: qcom: msm8916: Add IOMMU support")
with that fixed, the change looks good to me.
> diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
> index 33fb65d73104..3c934363368c 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
> @@ -1813,7 +1813,7 @@
> #size-cells = <1>;
> #iommu-cells = <1>;
> compatible = "qcom,msm8916-iommu", "qcom,msm-iommu-v1";
> - ranges = <0 0x01e20000 0x40000>;
> + ranges = <0 0x01e20000 0x20000>;
> reg = <0x01ef0000 0x3000>;
> clocks = <&gcc GCC_SMMU_CFG_CLK>,
> <&gcc GCC_APSS_TCU_CLK>;
Thanks,
Pavan
