| Message ID | 20230915143304.477-1-quic_gkohli@quicinc.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v1] arm64: dts: qcom: msm8916: Fix iommu local address range | expand |
On 15.09.2023 16:33, Gaurav Kohli wrote: > Fix the apps iommu local address space range as per data sheet. > > Fixes: 6a6729f38436 ("arm64: dts: qcom: msm8916: Add IOMMU support") > Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > Tested-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > Signed-off-by: Gaurav Kohli <quic_gkohli@quicinc.com> > --- > Changes since v0: > -Update Fixes tag. Versions start from 1. When resending new revisions, it's expected you provide a link to the previous submission. Other than that: Acked-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad > > 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>;
On Fri, Sep 15, 2023 at 08:03:04PM +0530, Gaurav Kohli wrote: > Fix the apps iommu local address space range as per data sheet. > > Fixes: 6a6729f38436 ("arm64: dts: qcom: msm8916: Add IOMMU support") > Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > Tested-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > Signed-off-by: Gaurav Kohli <quic_gkohli@quicinc.com> > --- > Changes since v0: > -Update Fixes tag. > > 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>; Please also submit another patch to fix this in msm8939.dtsi. It has the same mistake. Thanks, Stephan
On 15/09/2023 15:42, Stephan Gerhold wrote: > On Fri, Sep 15, 2023 at 08:03:04PM +0530, Gaurav Kohli wrote: >> Fix the apps iommu local address space range as per data sheet. >> >> Fixes: 6a6729f38436 ("arm64: dts: qcom: msm8916: Add IOMMU support") >> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> >> Tested-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> >> Signed-off-by: Gaurav Kohli <quic_gkohli@quicinc.com> > > Aside from the minor things Konrad mentioned (v1 -> v2) FWIW: > > Reviewed-by: Stephan Gerhold <stephan@gerhold.net> > > Although I can kind of understand that starting to count at 0 often > feels more intuitive, especially for Linux kernel things. :-) Just put OPTION BASE 0 at the start of your cover letter ;) --- bod
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>;
Fix the apps iommu local address space range as per data sheet. Fixes: 6a6729f38436 ("arm64: dts: qcom: msm8916: Add IOMMU support") Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> Tested-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> Signed-off-by: Gaurav Kohli <quic_gkohli@quicinc.com> --- Changes since v0: -Update Fixes tag.