* [PATCH 1/3] dt-bindings: arm-smmu: Fix binding for SDX55 and SDX65
2023-01-23 13:19 [PATCH 0/3] Fix SMMU on SDX55 and SDX65 Manivannan Sadhasivam
@ 2023-01-23 13:19 ` Manivannan Sadhasivam
2023-01-23 15:49 ` Krzysztof Kozlowski
2023-01-23 13:19 ` [PATCH 2/3] ARM: dts: qcom: sdx55: Add Qcom SMMU-500 as the fallback for IOMMU node Manivannan Sadhasivam
` (3 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Manivannan Sadhasivam @ 2023-01-23 13:19 UTC (permalink / raw)
To: andersson, will, joro, robh+dt, krzysztof.kozlowski+dt
Cc: konrad.dybcio, linux-arm-msm, devicetree, linux-kernel, iommu,
dmitry.baryshkov, Manivannan Sadhasivam
Both SDX55 and SDX66 SoCs are using the Qualcomm version of the SMMU-500
IP. But the binding lists them under the non-qcom implementation which is
not correct.
So fix the binding by moving these two SoCs under "qcom,smmu-500"
implementation.
Fixes: 6c84bbd103d8 ("dt-bindings: arm-smmu: Add generic qcom,smmu-500 bindings")
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
---
Documentation/devicetree/bindings/iommu/arm,smmu.yaml | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/Documentation/devicetree/bindings/iommu/arm,smmu.yaml b/Documentation/devicetree/bindings/iommu/arm,smmu.yaml
index b28c5c2b0ff2..25fd10d39b3c 100644
--- a/Documentation/devicetree/bindings/iommu/arm,smmu.yaml
+++ b/Documentation/devicetree/bindings/iommu/arm,smmu.yaml
@@ -42,6 +42,8 @@ properties:
- qcom,sc8280xp-smmu-500
- qcom,sdm670-smmu-500
- qcom,sdm845-smmu-500
+ - qcom,sdx55-smmu-500
+ - qcom,sdx65-smmu-500
- qcom,sm6115-smmu-500
- qcom,sm6350-smmu-500
- qcom,sm6375-smmu-500
@@ -52,14 +54,6 @@ properties:
- const: qcom,smmu-500
- const: arm,mmu-500
- - description: Qcom SoCs implementing "arm,mmu-500" (non-qcom implementation)
- deprecated: true
- items:
- - enum:
- - qcom,sdx55-smmu-500
- - qcom,sdx65-smmu-500
- - const: arm,mmu-500
-
- description: Qcom SoCs implementing "arm,mmu-500" (legacy binding)
deprecated: true
items:
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 1/3] dt-bindings: arm-smmu: Fix binding for SDX55 and SDX65
2023-01-23 13:19 ` [PATCH 1/3] dt-bindings: arm-smmu: Fix binding for " Manivannan Sadhasivam
@ 2023-01-23 15:49 ` Krzysztof Kozlowski
0 siblings, 0 replies; 9+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-23 15:49 UTC (permalink / raw)
To: Manivannan Sadhasivam, andersson, will, joro, robh+dt,
krzysztof.kozlowski+dt
Cc: konrad.dybcio, linux-arm-msm, devicetree, linux-kernel, iommu,
dmitry.baryshkov
On 23/01/2023 14:19, Manivannan Sadhasivam wrote:
> Both SDX55 and SDX66 SoCs are using the Qualcomm version of the SMMU-500
> IP. But the binding lists them under the non-qcom implementation which is
> not correct.
>
> So fix the binding by moving these two SoCs under "qcom,smmu-500"
> implementation.
>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/3] ARM: dts: qcom: sdx55: Add Qcom SMMU-500 as the fallback for IOMMU node
2023-01-23 13:19 [PATCH 0/3] Fix SMMU on SDX55 and SDX65 Manivannan Sadhasivam
2023-01-23 13:19 ` [PATCH 1/3] dt-bindings: arm-smmu: Fix binding for " Manivannan Sadhasivam
@ 2023-01-23 13:19 ` Manivannan Sadhasivam
2023-01-23 13:21 ` Konrad Dybcio
2023-01-23 13:19 ` [PATCH 3/3] ARM: dts: qcom: sdx65: " Manivannan Sadhasivam
` (2 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Manivannan Sadhasivam @ 2023-01-23 13:19 UTC (permalink / raw)
To: andersson, will, joro, robh+dt, krzysztof.kozlowski+dt
Cc: konrad.dybcio, linux-arm-msm, devicetree, linux-kernel, iommu,
dmitry.baryshkov, Manivannan Sadhasivam, stable
SDX55 uses the Qcom version of the SMMU-500 IP. So use "qcom,smmu-500"
compatible as the fallback to the SoC specific compatible.
Cc: <stable@vger.kernel.org> # 5.12
Fixes: a2bdfdfba2af ("ARM: dts: qcom: sdx55: Enable ARM SMMU")
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
---
arch/arm/boot/dts/qcom-sdx55.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-sdx55.dtsi b/arch/arm/boot/dts/qcom-sdx55.dtsi
index f1c0dab40992..93d71aff3fab 100644
--- a/arch/arm/boot/dts/qcom-sdx55.dtsi
+++ b/arch/arm/boot/dts/qcom-sdx55.dtsi
@@ -578,7 +578,7 @@ pil-reloc@94c {
};
apps_smmu: iommu@15000000 {
- compatible = "qcom,sdx55-smmu-500", "arm,mmu-500";
+ compatible = "qcom,sdx55-smmu-500", "qcom,smmu-500", "arm,mmu-500";
reg = <0x15000000 0x20000>;
#iommu-cells = <2>;
#global-interrupts = <1>;
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 2/3] ARM: dts: qcom: sdx55: Add Qcom SMMU-500 as the fallback for IOMMU node
2023-01-23 13:19 ` [PATCH 2/3] ARM: dts: qcom: sdx55: Add Qcom SMMU-500 as the fallback for IOMMU node Manivannan Sadhasivam
@ 2023-01-23 13:21 ` Konrad Dybcio
0 siblings, 0 replies; 9+ messages in thread
From: Konrad Dybcio @ 2023-01-23 13:21 UTC (permalink / raw)
To: Manivannan Sadhasivam, andersson, will, joro, robh+dt,
krzysztof.kozlowski+dt
Cc: linux-arm-msm, devicetree, linux-kernel, iommu, dmitry.baryshkov,
stable
On 23.01.2023 14:19, Manivannan Sadhasivam wrote:
> SDX55 uses the Qcom version of the SMMU-500 IP. So use "qcom,smmu-500"
> compatible as the fallback to the SoC specific compatible.
>
> Cc: <stable@vger.kernel.org> # 5.12
> Fixes: a2bdfdfba2af ("ARM: dts: qcom: sdx55: Enable ARM SMMU")
> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm/boot/dts/qcom-sdx55.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-sdx55.dtsi b/arch/arm/boot/dts/qcom-sdx55.dtsi
> index f1c0dab40992..93d71aff3fab 100644
> --- a/arch/arm/boot/dts/qcom-sdx55.dtsi
> +++ b/arch/arm/boot/dts/qcom-sdx55.dtsi
> @@ -578,7 +578,7 @@ pil-reloc@94c {
> };
>
> apps_smmu: iommu@15000000 {
> - compatible = "qcom,sdx55-smmu-500", "arm,mmu-500";
> + compatible = "qcom,sdx55-smmu-500", "qcom,smmu-500", "arm,mmu-500";
> reg = <0x15000000 0x20000>;
> #iommu-cells = <2>;
> #global-interrupts = <1>;
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 3/3] ARM: dts: qcom: sdx65: Add Qcom SMMU-500 as the fallback for IOMMU node
2023-01-23 13:19 [PATCH 0/3] Fix SMMU on SDX55 and SDX65 Manivannan Sadhasivam
2023-01-23 13:19 ` [PATCH 1/3] dt-bindings: arm-smmu: Fix binding for " Manivannan Sadhasivam
2023-01-23 13:19 ` [PATCH 2/3] ARM: dts: qcom: sdx55: Add Qcom SMMU-500 as the fallback for IOMMU node Manivannan Sadhasivam
@ 2023-01-23 13:19 ` Manivannan Sadhasivam
2023-01-23 13:21 ` Konrad Dybcio
2023-01-24 12:10 ` [PATCH 0/3] Fix SMMU on SDX55 and SDX65 Will Deacon
2023-02-09 4:23 ` (subset) " Bjorn Andersson
4 siblings, 1 reply; 9+ messages in thread
From: Manivannan Sadhasivam @ 2023-01-23 13:19 UTC (permalink / raw)
To: andersson, will, joro, robh+dt, krzysztof.kozlowski+dt
Cc: konrad.dybcio, linux-arm-msm, devicetree, linux-kernel, iommu,
dmitry.baryshkov, Manivannan Sadhasivam, stable
SDX65 uses the Qcom version of the SMMU-500 IP. So use "qcom,smmu-500"
compatible as the fallback to the SoC specific compatible.
Cc: <stable@vger.kernel.org> # 5.19
Fixes: 98187f7b74bf ("ARM: dts: qcom: sdx65: Enable ARM SMMU")
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
---
arch/arm/boot/dts/qcom-sdx65.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-sdx65.dtsi b/arch/arm/boot/dts/qcom-sdx65.dtsi
index b073e0c63df4..408c4b87d44b 100644
--- a/arch/arm/boot/dts/qcom-sdx65.dtsi
+++ b/arch/arm/boot/dts/qcom-sdx65.dtsi
@@ -455,7 +455,7 @@ pil-reloc@94c {
};
apps_smmu: iommu@15000000 {
- compatible = "qcom,sdx65-smmu-500", "arm,mmu-500";
+ compatible = "qcom,sdx65-smmu-500", "qcom,smmu-500", "arm,mmu-500";
reg = <0x15000000 0x40000>;
#iommu-cells = <2>;
#global-interrupts = <1>;
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 3/3] ARM: dts: qcom: sdx65: Add Qcom SMMU-500 as the fallback for IOMMU node
2023-01-23 13:19 ` [PATCH 3/3] ARM: dts: qcom: sdx65: " Manivannan Sadhasivam
@ 2023-01-23 13:21 ` Konrad Dybcio
0 siblings, 0 replies; 9+ messages in thread
From: Konrad Dybcio @ 2023-01-23 13:21 UTC (permalink / raw)
To: Manivannan Sadhasivam, andersson, will, joro, robh+dt,
krzysztof.kozlowski+dt
Cc: linux-arm-msm, devicetree, linux-kernel, iommu, dmitry.baryshkov,
stable
On 23.01.2023 14:19, Manivannan Sadhasivam wrote:
> SDX65 uses the Qcom version of the SMMU-500 IP. So use "qcom,smmu-500"
> compatible as the fallback to the SoC specific compatible.
>
> Cc: <stable@vger.kernel.org> # 5.19
> Fixes: 98187f7b74bf ("ARM: dts: qcom: sdx65: Enable ARM SMMU")
> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm/boot/dts/qcom-sdx65.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-sdx65.dtsi b/arch/arm/boot/dts/qcom-sdx65.dtsi
> index b073e0c63df4..408c4b87d44b 100644
> --- a/arch/arm/boot/dts/qcom-sdx65.dtsi
> +++ b/arch/arm/boot/dts/qcom-sdx65.dtsi
> @@ -455,7 +455,7 @@ pil-reloc@94c {
> };
>
> apps_smmu: iommu@15000000 {
> - compatible = "qcom,sdx65-smmu-500", "arm,mmu-500";
> + compatible = "qcom,sdx65-smmu-500", "qcom,smmu-500", "arm,mmu-500";
> reg = <0x15000000 0x40000>;
> #iommu-cells = <2>;
> #global-interrupts = <1>;
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 0/3] Fix SMMU on SDX55 and SDX65
2023-01-23 13:19 [PATCH 0/3] Fix SMMU on SDX55 and SDX65 Manivannan Sadhasivam
` (2 preceding siblings ...)
2023-01-23 13:19 ` [PATCH 3/3] ARM: dts: qcom: sdx65: " Manivannan Sadhasivam
@ 2023-01-24 12:10 ` Will Deacon
2023-02-09 4:23 ` (subset) " Bjorn Andersson
4 siblings, 0 replies; 9+ messages in thread
From: Will Deacon @ 2023-01-24 12:10 UTC (permalink / raw)
To: krzysztof.kozlowski+dt, andersson, joro, Manivannan Sadhasivam,
robh+dt
Cc: catalin.marinas, kernel-team, Will Deacon, iommu, linux-kernel,
dmitry.baryshkov, linux-arm-msm, konrad.dybcio, devicetree
On Mon, 23 Jan 2023 18:49:28 +0530, Manivannan Sadhasivam wrote:
> Both SDX55 and SDX65 SoCs are using the Qcom version of the ARM SMMU-500
> IP. Even though the SoC specfic compatibles were being used in devicetree,
> the compatibles were not added to the arm-smmu-qcom driver. So they end up
> using the generic ARM SMMU-500 driver instead.
>
> Spotting this discrepancy, the recent arm,smmu bindings change moved these
> SoCs under the non-qcom implementation of the SMMU.
>
> [...]
Applied bindings change to will (for-joerg/arm-smmu/bindings), thanks!
[1/3] dt-bindings: arm-smmu: Fix binding for SDX55 and SDX65
https://git.kernel.org/will/c/eb9181a3ae60
Cheers,
--
Will
https://fixes.arm64.dev
https://next.arm64.dev
https://will.arm64.dev
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: (subset) [PATCH 0/3] Fix SMMU on SDX55 and SDX65
2023-01-23 13:19 [PATCH 0/3] Fix SMMU on SDX55 and SDX65 Manivannan Sadhasivam
` (3 preceding siblings ...)
2023-01-24 12:10 ` [PATCH 0/3] Fix SMMU on SDX55 and SDX65 Will Deacon
@ 2023-02-09 4:23 ` Bjorn Andersson
4 siblings, 0 replies; 9+ messages in thread
From: Bjorn Andersson @ 2023-02-09 4:23 UTC (permalink / raw)
To: Manivannan Sadhasivam, robh+dt, joro, krzysztof.kozlowski+dt,
will
Cc: konrad.dybcio, devicetree, dmitry.baryshkov, iommu, linux-kernel,
linux-arm-msm
On Mon, 23 Jan 2023 18:49:28 +0530, Manivannan Sadhasivam wrote:
> Both SDX55 and SDX65 SoCs are using the Qcom version of the ARM SMMU-500
> IP. Even though the SoC specfic compatibles were being used in devicetree,
> the compatibles were not added to the arm-smmu-qcom driver. So they end up
> using the generic ARM SMMU-500 driver instead.
>
> Spotting this discrepancy, the recent arm,smmu bindings change moved these
> SoCs under the non-qcom implementation of the SMMU.
>
> [...]
Applied, thanks!
[2/3] ARM: dts: qcom: sdx55: Add Qcom SMMU-500 as the fallback for IOMMU node
commit: af4ab377543853b690cc85b4c46cf976ab560dc2
[3/3] ARM: dts: qcom: sdx65: Add Qcom SMMU-500 as the fallback for IOMMU node
commit: 157178a7912e00a0aa0371dc9041952c1a21d112
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 9+ messages in thread