* [PATCH v2 00/14] Clean up RPM bus clocks remnants
@ 2023-09-12 13:31 Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks Konrad Dybcio
` (15 more replies)
0 siblings, 16 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio, Krzysztof Kozlowski
After the recent cleanups ([1], [2]) some in-tree abusers that directly
accessed the RPM bus clocks, effectively circumventing and working
against the efforts of the interconnect framework, were found.
Patches 1-5 drop deprecated references and the rest attempt to stop
direct bus clock abuses.
Depends on [2].
8996 and 8998 remoteproc changes were not tested, they never worked on
my Sony phones.
[1] https://lore.kernel.org/linux-arm-msm/20230526-topic-smd_icc-v7-0-09c78c175546@linaro.org/
[2] https://lore.kernel.org/linux-arm-msm/20230721-topic-icc_bindings-v1-0-93e2bc728fb7@linaro.org/
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
Changes in v2:
- Incorporate [3] into the sdm630 patch, add required bindings fixes
- dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock: Merge entries (krzk)
- Pick up a-b (krzk)
- Add "sdm630: Fix USB2 clock-names order"
- Link to v1: https://lore.kernel.org/r/20230721-topic-rpm_clk_cleanup-v1-0-cf6cd5c621d5@linaro.org
[3] https://lore.kernel.org/linux-arm-msm/20230719073520.2644966-1-alexeymin@postmarketos.org/#t
---
Konrad Dybcio (14):
arm64: dts: qcom: msm8916: Drop RPM bus clocks
arm64: dts: qcom: msm8996: Drop RPM bus clocks
arm64: dts: qcom: qcs404: Drop RPM bus clocks
dt-bindings: arm-smmu: Fix SDM630 clocks description
dt-bindings: usb: qcom,dwc3: Fix SDM660 clock description
arm64: dts: qcom: sdm630: Drop RPM bus clocks
arm64: dts: qcom: msm8939: Drop RPM bus clocks
dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock
dt-bindings: remoteproc: qcom,msm8996-mss-pil: Remove PNoC clock
remoteproc: qcom: q6v5-mss: Remove PNoC clock from 8996 MSS
arm64: dts: qcom: msm8998: Remove AGGRE2 clock from SLPI
arm64: dts: qcom: msm8996: Remove AGGRE2 clock from SLPI
arm64: dts: qcom: msm8996: Remove PNoC clock from MSS
arm64: dts: qcom: sdm630: Fix USB2 clock-names order
.../devicetree/bindings/iommu/arm,smmu.yaml | 2 +-
.../devicetree/bindings/remoteproc/qcom,adsp.yaml | 20 +-------
.../bindings/remoteproc/qcom,msm8996-mss-pil.yaml | 2 -
.../devicetree/bindings/usb/qcom,dwc3.yaml | 6 +--
arch/arm64/boot/dts/qcom/msm8916.dtsi | 9 ----
arch/arm64/boot/dts/qcom/msm8939.dtsi | 12 -----
arch/arm64/boot/dts/qcom/msm8996.dtsi | 43 ++++++-----------
arch/arm64/boot/dts/qcom/msm8998.dtsi | 5 +-
arch/arm64/boot/dts/qcom/qcs404.dtsi | 9 ----
arch/arm64/boot/dts/qcom/sdm630.dtsi | 55 +++++-----------------
drivers/remoteproc/qcom_q6v5_mss.c | 1 -
11 files changed, 34 insertions(+), 130 deletions(-)
---
base-commit: 66d9573193967138cd12e232d4b5bc2b57e0d1ac
change-id: 20230721-topic-rpm_clk_cleanup-1b2f4a1acd01
Best regards,
--
Konrad Dybcio <konrad.dybcio@linaro.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-13 7:07 ` Krzysztof Kozlowski
2023-09-12 13:31 ` [PATCH v2 02/14] arm64: dts: qcom: msm8996: " Konrad Dybcio
` (14 subsequent siblings)
15 siblings, 1 reply; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
These clocks are now handled from within the icc framework and are
no longer registered from within the CCF. Remove them.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8916.dtsi | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
index 33fb65d73104..8c2ec09f57c4 100644
--- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
@@ -519,9 +519,6 @@ bimc: interconnect@400000 {
compatible = "qcom,msm8916-bimc";
reg = <0x00400000 0x62000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
- <&rpmcc RPM_SMD_BIMC_A_CLK>;
};
tsens: thermal-sensor@4a9000 {
@@ -554,18 +551,12 @@ pcnoc: interconnect@500000 {
compatible = "qcom,msm8916-pcnoc";
reg = <0x00500000 0x11000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_PCNOC_CLK>,
- <&rpmcc RPM_SMD_PCNOC_A_CLK>;
};
snoc: interconnect@580000 {
compatible = "qcom,msm8916-snoc";
reg = <0x00580000 0x14000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
- <&rpmcc RPM_SMD_SNOC_A_CLK>;
};
stm: stm@802000 {
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 02/14] arm64: dts: qcom: msm8996: Drop RPM bus clocks
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 03/14] arm64: dts: qcom: qcs404: " Konrad Dybcio
` (13 subsequent siblings)
15 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
These clocks are now handled from within the icc framework and are
no longer registered from within the CCF. Remove them.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8996.dtsi | 27 ++++-----------------------
1 file changed, 4 insertions(+), 23 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index c8e0986425ab..acef67ab0581 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -798,9 +798,6 @@ bimc: interconnect@408000 {
compatible = "qcom,msm8996-bimc";
reg = <0x00408000 0x5a000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
- <&rpmcc RPM_SMD_BIMC_A_CLK>;
};
tsens0: thermal-sensor@4a9000 {
@@ -851,18 +848,12 @@ cnoc: interconnect@500000 {
compatible = "qcom,msm8996-cnoc";
reg = <0x00500000 0x1000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_CNOC_CLK>,
- <&rpmcc RPM_SMD_CNOC_A_CLK>;
};
snoc: interconnect@524000 {
compatible = "qcom,msm8996-snoc";
reg = <0x00524000 0x1c000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
- <&rpmcc RPM_SMD_SNOC_A_CLK>;
};
a0noc: interconnect@543000 {
@@ -882,19 +873,14 @@ a1noc: interconnect@562000 {
compatible = "qcom,msm8996-a1noc";
reg = <0x00562000 0x5000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_AGGR1_NOC_CLK>,
- <&rpmcc RPM_SMD_AGGR1_NOC_A_CLK>;
};
a2noc: interconnect@583000 {
compatible = "qcom,msm8996-a2noc";
reg = <0x00583000 0x7000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a", "aggre2_ufs_axi", "ufs_axi";
- clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>,
- <&rpmcc RPM_SMD_AGGR2_NOC_A_CLK>,
- <&gcc GCC_AGGRE2_UFS_AXI_CLK>,
+ clock-names = "aggre2_ufs_axi", "ufs_axi";
+ clocks = <&gcc GCC_AGGRE2_UFS_AXI_CLK>,
<&gcc GCC_UFS_AXI_CLK>;
};
@@ -902,19 +888,14 @@ mnoc: interconnect@5a4000 {
compatible = "qcom,msm8996-mnoc";
reg = <0x005a4000 0x1c000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a", "iface";
- clocks = <&rpmcc RPM_SMD_MMAXI_CLK>,
- <&rpmcc RPM_SMD_MMAXI_A_CLK>,
- <&mmcc AHB_CLK_SRC>;
+ clock-names = "iface";
+ clocks = <&mmcc AHB_CLK_SRC>;
};
pnoc: interconnect@5c0000 {
compatible = "qcom,msm8996-pnoc";
reg = <0x005c0000 0x3000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_PCNOC_CLK>,
- <&rpmcc RPM_SMD_PCNOC_A_CLK>;
};
tcsr_mutex: hwlock@740000 {
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 03/14] arm64: dts: qcom: qcs404: Drop RPM bus clocks
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 02/14] arm64: dts: qcom: msm8996: " Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 04/14] dt-bindings: arm-smmu: Fix SDM630 clocks description Konrad Dybcio
` (12 subsequent siblings)
15 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
These clocks are now handled from within the icc framework and are
no longer registered from within the CCF. Remove them.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/qcs404.dtsi | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/qcs404.dtsi b/arch/arm64/boot/dts/qcom/qcs404.dtsi
index 2721f32dfb71..317a0df30b83 100644
--- a/arch/arm64/boot/dts/qcom/qcs404.dtsi
+++ b/arch/arm64/boot/dts/qcom/qcs404.dtsi
@@ -558,9 +558,6 @@ bimc: interconnect@400000 {
reg = <0x00400000 0x80000>;
compatible = "qcom,qcs404-bimc";
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
- <&rpmcc RPM_SMD_BIMC_A_CLK>;
};
tsens: thermal-sensor@4a9000 {
@@ -601,18 +598,12 @@ pcnoc: interconnect@500000 {
reg = <0x00500000 0x15080>;
compatible = "qcom,qcs404-pcnoc";
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_PNOC_CLK>,
- <&rpmcc RPM_SMD_PNOC_A_CLK>;
};
snoc: interconnect@580000 {
reg = <0x00580000 0x23080>;
compatible = "qcom,qcs404-snoc";
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
- <&rpmcc RPM_SMD_SNOC_A_CLK>;
};
remoteproc_cdsp: remoteproc@b00000 {
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 04/14] dt-bindings: arm-smmu: Fix SDM630 clocks description
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (2 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 03/14] arm64: dts: qcom: qcs404: " Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 05/14] dt-bindings: usb: qcom,dwc3: Fix SDM660 clock description Konrad Dybcio
` (11 subsequent siblings)
15 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
SDM630 was abusingly referencing one of the internal bus clocks, that
were recently dropped from Linux (because the original implementation
did not make much sense), circumventing the interconnect framework.
Fix it by dropping the bus-mm clock (which requires separating 630 from
similar entries) and keeping the rest as-is.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
Documentation/devicetree/bindings/iommu/arm,smmu.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/iommu/arm,smmu.yaml b/Documentation/devicetree/bindings/iommu/arm,smmu.yaml
index cf29ab10501c..b1b2cf81b42f 100644
--- a/Documentation/devicetree/bindings/iommu/arm,smmu.yaml
+++ b/Documentation/devicetree/bindings/iommu/arm,smmu.yaml
@@ -270,6 +270,7 @@ allOf:
contains:
enum:
- qcom,msm8998-smmu-v2
+ - qcom,sdm630-smmu-v2
then:
anyOf:
- properties:
@@ -311,7 +312,6 @@ allOf:
compatible:
contains:
enum:
- - qcom,sdm630-smmu-v2
- qcom,sm6375-smmu-v2
then:
anyOf:
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 05/14] dt-bindings: usb: qcom,dwc3: Fix SDM660 clock description
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (3 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 04/14] dt-bindings: arm-smmu: Fix SDM630 clocks description Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks Konrad Dybcio
` (10 subsequent siblings)
15 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
SDM660 was abusingly referencing one of the internal bus clocks, that
were recently dropped from Linux (because the original implementation
did not make much sense), circumventing the interconnect framework.
Drop it.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml b/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml
index 67591057f234..820878031d47 100644
--- a/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml
+++ b/Documentation/devicetree/bindings/usb/qcom,dwc3.yaml
@@ -283,8 +283,8 @@ allOf:
then:
properties:
clocks:
- minItems: 5
- maxItems: 6
+ minItems: 4
+ maxItems: 5
clock-names:
oneOf:
- items:
@@ -293,13 +293,11 @@ allOf:
- const: iface
- const: sleep
- const: mock_utmi
- - const: bus
- items:
- const: cfg_noc
- const: core
- const: sleep
- const: mock_utmi
- - const: bus
- if:
properties:
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (4 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 05/14] dt-bindings: usb: qcom,dwc3: Fix SDM660 clock description Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-13 7:13 ` Krzysztof Kozlowski
2024-01-02 21:17 ` Marijn Suijten
2023-09-12 13:31 ` [PATCH v2 07/14] arm64: dts: qcom: msm8939: " Konrad Dybcio
` (9 subsequent siblings)
15 siblings, 2 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
These clocks are now handled from within the icc framework and are
no longer registered from within the CCF. Remove them.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm630.dtsi | 49 +++++++-----------------------------
1 file changed, 9 insertions(+), 40 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index ec6003212c4d..f11d2a07508c 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -605,9 +605,6 @@ bimc: interconnect@1008000 {
compatible = "qcom,sdm660-bimc";
reg = <0x01008000 0x78000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
- <&rpmcc RPM_SMD_BIMC_A_CLK>;
};
restart@10ac000 {
@@ -619,28 +616,17 @@ cnoc: interconnect@1500000 {
compatible = "qcom,sdm660-cnoc";
reg = <0x01500000 0x10000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_CNOC_CLK>,
- <&rpmcc RPM_SMD_CNOC_A_CLK>;
};
snoc: interconnect@1626000 {
compatible = "qcom,sdm660-snoc";
reg = <0x01626000 0x7090>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
- <&rpmcc RPM_SMD_SNOC_A_CLK>;
};
anoc2_smmu: iommu@16c0000 {
compatible = "qcom,sdm630-smmu-v2", "qcom,smmu-v2";
reg = <0x016c0000 0x40000>;
-
- assigned-clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
- assigned-clock-rates = <1000>;
- clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
- clock-names = "bus";
#global-interrupts = <2>;
#iommu-cells = <1>;
@@ -685,16 +671,12 @@ a2noc: interconnect@1704000 {
compatible = "qcom,sdm660-a2noc";
reg = <0x01704000 0xc100>;
#interconnect-cells = <1>;
- clock-names = "bus",
- "bus_a",
- "ipa",
+ clock-names = "ipa",
"ufs_axi",
"aggre2_ufs_axi",
"aggre2_usb3_axi",
"cfg_noc_usb2_axi";
- clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>,
- <&rpmcc RPM_SMD_AGGR2_NOC_A_CLK>,
- <&rpmcc RPM_SMD_IPA_CLK>,
+ clocks = <&rpmcc RPM_SMD_IPA_CLK>,
<&gcc GCC_UFS_AXI_CLK>,
<&gcc GCC_AGGRE2_UFS_AXI_CLK>,
<&gcc GCC_AGGRE2_USB3_AXI_CLK>,
@@ -705,10 +687,8 @@ mnoc: interconnect@1745000 {
compatible = "qcom,sdm660-mnoc";
reg = <0x01745000 0xa010>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a", "iface";
- clocks = <&rpmcc RPM_SMD_MMSSNOC_AXI_CLK>,
- <&rpmcc RPM_SMD_MMSSNOC_AXI_CLK_A>,
- <&mmcc AHB_CLK_SRC>;
+ clock-names = "iface";
+ clocks = <&mmcc AHB_CLK_SRC>;
};
tsens: thermal-sensor@10ae000 {
@@ -1228,20 +1208,16 @@ usb3: usb@a8f8800 {
<&gcc GCC_USB30_MASTER_CLK>,
<&gcc GCC_AGGRE2_USB3_AXI_CLK>,
<&gcc GCC_USB30_SLEEP_CLK>,
- <&gcc GCC_USB30_MOCK_UTMI_CLK>,
- <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
+ <&gcc GCC_USB30_MOCK_UTMI_CLK>;
clock-names = "cfg_noc",
"core",
"iface",
"sleep",
- "mock_utmi",
- "bus";
+ "mock_utmi";
assigned-clocks = <&gcc GCC_USB30_MOCK_UTMI_CLK>,
- <&gcc GCC_USB30_MASTER_CLK>,
- <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
- assigned-clock-rates = <19200000>, <120000000>,
- <19200000>;
+ <&gcc GCC_USB30_MASTER_CLK>;
+ assigned-clock-rates = <19200000>, <120000000>;
interrupts = <GIC_SPI 347 IRQ_TYPE_LEVEL_HIGH>,
<GIC_SPI 243 IRQ_TYPE_LEVEL_HIGH>;
@@ -2144,10 +2120,9 @@ mmss_smmu: iommu@cd00000 {
clocks = <&mmcc MNOC_AHB_CLK>,
<&mmcc BIMC_SMMU_AHB_CLK>,
- <&rpmcc RPM_SMD_MMSSNOC_AXI_CLK>,
<&mmcc BIMC_SMMU_AXI_CLK>;
clock-names = "iface-mm", "iface-smmu",
- "bus-mm", "bus-smmu";
+ "bus-smmu";
#global-interrupts = <2>;
#iommu-cells = <1>;
@@ -2264,12 +2239,6 @@ gnoc: interconnect@17900000 {
compatible = "qcom,sdm660-gnoc";
reg = <0x17900000 0xe000>;
#interconnect-cells = <1>;
- /*
- * This one apparently features no clocks,
- * so let's not mess with the driver needlessly
- */
- clock-names = "bus", "bus_a";
- clocks = <&xo_board>, <&xo_board>;
};
apcs_glb: mailbox@17911000 {
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 07/14] arm64: dts: qcom: msm8939: Drop RPM bus clocks
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (5 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-13 7:15 ` Krzysztof Kozlowski
2023-09-12 13:31 ` [PATCH v2 08/14] dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock Konrad Dybcio
` (8 subsequent siblings)
15 siblings, 1 reply; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
These clocks are now handled from within the icc framework and are
no longer registered from within the CCF. Remove them.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8939.dtsi | 12 ------------
1 file changed, 12 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8939.dtsi b/arch/arm64/boot/dts/qcom/msm8939.dtsi
index 6e24f0f2374f..e3d1eb7dd289 100644
--- a/arch/arm64/boot/dts/qcom/msm8939.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8939.dtsi
@@ -581,9 +581,6 @@ rpm_msg_ram: sram@60000 {
bimc: interconnect@400000 {
compatible = "qcom,msm8939-bimc";
reg = <0x00400000 0x62000>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
- <&rpmcc RPM_SMD_BIMC_A_CLK>;
#interconnect-cells = <1>;
};
@@ -627,25 +624,16 @@ restart@4ab000 {
pcnoc: interconnect@500000 {
compatible = "qcom,msm8939-pcnoc";
reg = <0x00500000 0x11000>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_PCNOC_CLK>,
- <&rpmcc RPM_SMD_PCNOC_A_CLK>;
#interconnect-cells = <1>;
};
snoc: interconnect@580000 {
compatible = "qcom,msm8939-snoc";
reg = <0x00580000 0x14080>;
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
- <&rpmcc RPM_SMD_SNOC_A_CLK>;
#interconnect-cells = <1>;
snoc_mm: interconnect-snoc {
compatible = "qcom,msm8939-snoc-mm";
- clock-names = "bus", "bus_a";
- clocks = <&rpmcc RPM_SMD_SYSMMNOC_CLK>,
- <&rpmcc RPM_SMD_SYSMMNOC_A_CLK>;
#interconnect-cells = <1>;
};
};
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 08/14] dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (6 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 07/14] arm64: dts: qcom: msm8939: " Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-13 7:16 ` Krzysztof Kozlowski
2023-09-12 13:31 ` [PATCH v2 09/14] dt-bindings: remoteproc: qcom,msm8996-mss-pil: Remove PNoC clock Konrad Dybcio
` (7 subsequent siblings)
15 siblings, 1 reply; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
The AGGRE2 clock is a clock for the entire AGGRE2 bus, managed from
within the interconnect driver. Attaching it to SLPI was a total hack.
Get rid of it.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
.../devicetree/bindings/remoteproc/qcom,adsp.yaml | 20 ++------------------
1 file changed, 2 insertions(+), 18 deletions(-)
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
index a2b0079de039..661c2b425da3 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
@@ -66,7 +66,9 @@ allOf:
- qcom,msm8953-adsp-pil
- qcom,msm8974-adsp-pil
- qcom,msm8996-adsp-pil
+ - qcom,msm8996-slpi-pil
- qcom,msm8998-adsp-pas
+ - qcom,msm8998-slpi-pas
- qcom,sdm845-adsp-pas
- qcom,sdm845-cdsp-pas
- qcom,sdm845-slpi-pas
@@ -79,24 +81,6 @@ allOf:
items:
- const: xo
- - if:
- properties:
- compatible:
- contains:
- enum:
- - qcom,msm8996-slpi-pil
- - qcom,msm8998-slpi-pas
- then:
- properties:
- clocks:
- items:
- - description: XO clock
- - description: AGGRE2 clock
- clock-names:
- items:
- - const: xo
- - const: aggre2
-
- if:
properties:
compatible:
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 09/14] dt-bindings: remoteproc: qcom,msm8996-mss-pil: Remove PNoC clock
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (7 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 08/14] dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 10/14] remoteproc: qcom: q6v5-mss: Remove PNoC clock from 8996 MSS Konrad Dybcio
` (6 subsequent siblings)
15 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio, Krzysztof Kozlowski
The PNoC clock is a clock for the entire PNoC bus, managed from
within the interconnect driver. Attaching it to MSS was a total hack.
Get rid of it.
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
Documentation/devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml | 2 --
1 file changed, 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml
index 0643faae2c39..971734085d51 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml
@@ -220,7 +220,6 @@ allOf:
- description: GCC MSS GPLL0 clock
- description: GCC MSS SNOC_AXI clock
- description: GCC MSS MNOC_AXI clock
- - description: RPM PNOC clock
- description: RPM QDSS clock
clock-names:
items:
@@ -231,7 +230,6 @@ allOf:
- const: gpll0_mss
- const: snoc_axi
- const: mnoc_axi
- - const: pnoc
- const: qdss
glink-edge: false
required:
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 10/14] remoteproc: qcom: q6v5-mss: Remove PNoC clock from 8996 MSS
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (8 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 09/14] dt-bindings: remoteproc: qcom,msm8996-mss-pil: Remove PNoC clock Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 11/14] arm64: dts: qcom: msm8998: Remove AGGRE2 clock from SLPI Konrad Dybcio
` (5 subsequent siblings)
15 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
The PNoC clock is a clock for the entire PNoC bus, managed from
within the interconnect driver. Attaching it to MSS was a total hack.
Get rid of it.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
drivers/remoteproc/qcom_q6v5_mss.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/remoteproc/qcom_q6v5_mss.c b/drivers/remoteproc/qcom_q6v5_mss.c
index 22fe7b5f5236..394b2c1cb5e2 100644
--- a/drivers/remoteproc/qcom_q6v5_mss.c
+++ b/drivers/remoteproc/qcom_q6v5_mss.c
@@ -2322,7 +2322,6 @@ static const struct rproc_hexagon_res msm8996_mss = {
},
.proxy_clk_names = (char*[]){
"xo",
- "pnoc",
"qdss",
NULL
},
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 11/14] arm64: dts: qcom: msm8998: Remove AGGRE2 clock from SLPI
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (9 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 10/14] remoteproc: qcom: q6v5-mss: Remove PNoC clock from 8996 MSS Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 12/14] arm64: dts: qcom: msm8996: " Konrad Dybcio
` (4 subsequent siblings)
15 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
The AGGRE2 clock is a clock for the entire AGGRE2 bus, managed from
within the interconnect driver. Attaching it to SLPI was a total hack.
Get rid of it.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8998.dtsi | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi b/arch/arm64/boot/dts/qcom/msm8998.dtsi
index f180047cacb0..5667ffa83ce5 100644
--- a/arch/arm64/boot/dts/qcom/msm8998.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi
@@ -1618,9 +1618,8 @@ remoteproc_slpi: remoteproc@5800000 {
px-supply = <&vreg_lvs2a_1p8>;
- clocks = <&rpmcc RPM_SMD_XO_CLK_SRC>,
- <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
- clock-names = "xo", "aggre2";
+ clocks = <&rpmcc RPM_SMD_XO_CLK_SRC>;
+ clock-names = "xo";
memory-region = <&slpi_mem>;
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 12/14] arm64: dts: qcom: msm8996: Remove AGGRE2 clock from SLPI
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (10 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 11/14] arm64: dts: qcom: msm8998: Remove AGGRE2 clock from SLPI Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 13/14] arm64: dts: qcom: msm8996: Remove PNoC clock from MSS Konrad Dybcio
` (3 subsequent siblings)
15 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
The AGGRE2 clock is a clock for the entire AGGRE2 bus, managed from
within the interconnect driver. Attaching it to SLPI was a total hack.
Get rid of it.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8996.dtsi | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index acef67ab0581..7061a8e12c81 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -2433,9 +2433,8 @@ slpi_pil: remoteproc@1c00000 {
"handover",
"stop-ack";
- clocks = <&xo_board>,
- <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
- clock-names = "xo", "aggre2";
+ clocks = <&xo_board>;
+ clock-names = "xo";
memory-region = <&slpi_mem>;
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 13/14] arm64: dts: qcom: msm8996: Remove PNoC clock from MSS
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (11 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 12/14] arm64: dts: qcom: msm8996: " Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 14/14] arm64: dts: qcom: sdm630: Fix USB2 clock-names order Konrad Dybcio
` (2 subsequent siblings)
15 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
The PNoC clock is a clock for the entire PNoC bus, managed from
within the interconnect driver. Attaching it to MSS was a total hack.
Get rid of it and take the liberty to make the clock-names entries
more readable.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8996.dtsi | 11 ++++++++---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index 7061a8e12c81..51db5ddcef88 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -2479,10 +2479,15 @@ mss_pil: remoteproc@2080000 {
<&gcc GCC_MSS_GPLL0_DIV_CLK>,
<&gcc GCC_MSS_SNOC_AXI_CLK>,
<&gcc GCC_MSS_MNOC_BIMC_AXI_CLK>,
- <&rpmcc RPM_SMD_PCNOC_CLK>,
<&rpmcc RPM_SMD_QDSS_CLK>;
- clock-names = "iface", "bus", "mem", "xo", "gpll0_mss",
- "snoc_axi", "mnoc_axi", "pnoc", "qdss";
+ clock-names = "iface",
+ "bus",
+ "mem",
+ "xo",
+ "gpll0_mss",
+ "snoc_axi",
+ "mnoc_axi",
+ "qdss";
resets = <&gcc GCC_MSS_RESTART>;
reset-names = "mss_restart";
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PATCH v2 14/14] arm64: dts: qcom: sdm630: Fix USB2 clock-names order
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (12 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 13/14] arm64: dts: qcom: msm8996: Remove PNoC clock from MSS Konrad Dybcio
@ 2023-09-12 13:31 ` Konrad Dybcio
2023-09-13 7:17 ` Krzysztof Kozlowski
2023-09-13 7:19 ` Krzysztof Kozlowski
2023-09-18 11:05 ` [PATCH v2 00/14] Clean up RPM bus clocks remnants Will Deacon
2023-09-20 18:30 ` (subset) " Bjorn Andersson
15 siblings, 2 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-12 13:31 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Konrad Dybcio
The last 2 clock-names entries for the USB2 controller were swapped,
resulting in schema warnings:
['cfg_noc', 'core', 'mock_utmi', 'sleep'] is too short
'iface' was expected
'sleep' was expected
'mock_utmi' was expected
Fix it and take the liberty to make the clock-names entries more
readable.
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm630.dtsi | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index f11d2a07508c..316c8fd224e0 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -1394,8 +1394,10 @@ usb2: usb@c2f8800 {
<&gcc GCC_USB20_MASTER_CLK>,
<&gcc GCC_USB20_MOCK_UTMI_CLK>,
<&gcc GCC_USB20_SLEEP_CLK>;
- clock-names = "cfg_noc", "core",
- "mock_utmi", "sleep";
+ clock-names = "cfg_noc",
+ "core",
+ "sleep",
+ "mock_utmi";
assigned-clocks = <&gcc GCC_USB20_MOCK_UTMI_CLK>,
<&gcc GCC_USB20_MASTER_CLK>;
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* Re: [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks
2023-09-12 13:31 ` [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks Konrad Dybcio
@ 2023-09-13 7:07 ` Krzysztof Kozlowski
2023-09-13 8:47 ` Konrad Dybcio
0 siblings, 1 reply; 33+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-13 7:07 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 12/09/2023 15:31, Konrad Dybcio wrote:
> These clocks are now handled from within the icc framework and are
That's a driver behavior, not hardware.
> no longer registered from within the CCF. Remove them.
>
Changes in Linux clock drivers should not cause some clocks to disappear
from DTS...
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks
2023-09-12 13:31 ` [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks Konrad Dybcio
@ 2023-09-13 7:13 ` Krzysztof Kozlowski
2023-09-13 12:08 ` Konrad Dybcio
2024-01-02 21:17 ` Marijn Suijten
1 sibling, 1 reply; 33+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-13 7:13 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 12/09/2023 15:31, Konrad Dybcio wrote:
> These clocks are now handled from within the icc framework and are
> no longer registered from within the CCF. Remove them.
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/sdm630.dtsi | 49 +++++++-----------------------------
> 1 file changed, 9 insertions(+), 40 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> index ec6003212c4d..f11d2a07508c 100644
> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> @@ -605,9 +605,6 @@ bimc: interconnect@1008000 {
> compatible = "qcom,sdm660-bimc";
> reg = <0x01008000 0x78000>;
> #interconnect-cells = <1>;
> - clock-names = "bus", "bus_a";
> - clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
> - <&rpmcc RPM_SMD_BIMC_A_CLK>;
Bindings expect here two clocks, so you miss some bindings patches.
> };
>
> restart@10ac000 {
> @@ -619,28 +616,17 @@ cnoc: interconnect@1500000 {
> compatible = "qcom,sdm660-cnoc";
> reg = <0x01500000 0x10000>;
> #interconnect-cells = <1>;
> - clock-names = "bus", "bus_a";
> - clocks = <&rpmcc RPM_SMD_CNOC_CLK>,
> - <&rpmcc RPM_SMD_CNOC_A_CLK>;
> };
>
> snoc: interconnect@1626000 {
> compatible = "qcom,sdm660-snoc";
> reg = <0x01626000 0x7090>;
> #interconnect-cells = <1>;
> - clock-names = "bus", "bus_a";
> - clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
> - <&rpmcc RPM_SMD_SNOC_A_CLK>;
> };
>
> anoc2_smmu: iommu@16c0000 {
> compatible = "qcom,sdm630-smmu-v2", "qcom,smmu-v2";
> reg = <0x016c0000 0x40000>;
> -
> - assigned-clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
> - assigned-clock-rates = <1000>;
> - clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
> - clock-names = "bus";
This is also against bindings. After your patch #4, such bus clock (or
other combinations) is still required.
> #global-interrupts = <2>;
> #iommu-cells = <1>;
>
> @@ -685,16 +671,12 @@ a2noc: interconnect@1704000 {
> compatible = "qcom,sdm660-a2noc";
> reg = <0x01704000 0xc100>;
> #interconnect-cells = <1>;
> - clock-names = "bus",
> - "bus_a",
> - "ipa",
> + clock-names = "ipa",
And which bindings does this match?
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 07/14] arm64: dts: qcom: msm8939: Drop RPM bus clocks
2023-09-12 13:31 ` [PATCH v2 07/14] arm64: dts: qcom: msm8939: " Konrad Dybcio
@ 2023-09-13 7:15 ` Krzysztof Kozlowski
0 siblings, 0 replies; 33+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-13 7:15 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 12/09/2023 15:31, Konrad Dybcio wrote:
> These clocks are now handled from within the icc framework and are
> no longer registered from within the CCF. Remove them.
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 08/14] dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock
2023-09-12 13:31 ` [PATCH v2 08/14] dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock Konrad Dybcio
@ 2023-09-13 7:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 33+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-13 7:16 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 12/09/2023 15:31, Konrad Dybcio wrote:
> The AGGRE2 clock is a clock for the entire AGGRE2 bus, managed from
> within the interconnect driver. Attaching it to SLPI was a total hack.
> Get rid of it.
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 14/14] arm64: dts: qcom: sdm630: Fix USB2 clock-names order
2023-09-12 13:31 ` [PATCH v2 14/14] arm64: dts: qcom: sdm630: Fix USB2 clock-names order Konrad Dybcio
@ 2023-09-13 7:17 ` Krzysztof Kozlowski
2023-09-13 7:19 ` Krzysztof Kozlowski
1 sibling, 0 replies; 33+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-13 7:17 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 12/09/2023 15:31, Konrad Dybcio wrote:
> The last 2 clock-names entries for the USB2 controller were swapped,
> resulting in schema warnings:
>
> ['cfg_noc', 'core', 'mock_utmi', 'sleep'] is too short
> 'iface' was expected
> 'sleep' was expected
> 'mock_utmi' was expected
>
> Fix it and take the liberty to make the clock-names entries more
> readable.
This was already fixed:
https://lore.kernel.org/all/20230723141849.93078-2-krzysztof.kozlowski@linaro.org/
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 14/14] arm64: dts: qcom: sdm630: Fix USB2 clock-names order
2023-09-12 13:31 ` [PATCH v2 14/14] arm64: dts: qcom: sdm630: Fix USB2 clock-names order Konrad Dybcio
2023-09-13 7:17 ` Krzysztof Kozlowski
@ 2023-09-13 7:19 ` Krzysztof Kozlowski
1 sibling, 0 replies; 33+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-13 7:19 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 12/09/2023 15:31, Konrad Dybcio wrote:
> The last 2 clock-names entries for the USB2 controller were swapped,
> resulting in schema warnings:
>
> ['cfg_noc', 'core', 'mock_utmi', 'sleep'] is too short
> 'iface' was expected
> 'sleep' was expected
> 'mock_utmi' was expected
>
> Fix it and take the liberty to make the clock-names entries more
> readable.
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/sdm630.dtsi | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> index f11d2a07508c..316c8fd224e0 100644
> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> @@ -1394,8 +1394,10 @@ usb2: usb@c2f8800 {
> <&gcc GCC_USB20_MASTER_CLK>,
> <&gcc GCC_USB20_MOCK_UTMI_CLK>,
> <&gcc GCC_USB20_SLEEP_CLK>;
> - clock-names = "cfg_noc", "core",
> - "mock_utmi", "sleep";
> + clock-names = "cfg_noc",
> + "core",
> + "sleep",
> + "mock_utmi";
Plus this is just incorrect... :(
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks
2023-09-13 7:07 ` Krzysztof Kozlowski
@ 2023-09-13 8:47 ` Konrad Dybcio
2023-09-13 8:53 ` Krzysztof Kozlowski
0 siblings, 1 reply; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-13 8:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 13.09.2023 09:07, Krzysztof Kozlowski wrote:
> On 12/09/2023 15:31, Konrad Dybcio wrote:
>> These clocks are now handled from within the icc framework and are
>
> That's a driver behavior, not hardware.
I believe we've been over this already..
The rationale behind this change is: that hardware, which falls
under the "interconnect" class, was previously misrepresented as
a bunch of clocks. There are clocks underneath, but accessing them
directly would be equivalent to e.g. circumventing the PHY subsystem
and initializing your UFS PHY from within the UFS device.
Konrad
>
>> no longer registered from within the CCF. Remove them.
>>
>
> Changes in Linux clock drivers should not cause some clocks to disappear
> from DTS...
>
>
> Best regards,
> Krzysztof
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks
2023-09-13 8:47 ` Konrad Dybcio
@ 2023-09-13 8:53 ` Krzysztof Kozlowski
2023-09-13 10:48 ` Konrad Dybcio
0 siblings, 1 reply; 33+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-13 8:53 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 13/09/2023 10:47, Konrad Dybcio wrote:
> On 13.09.2023 09:07, Krzysztof Kozlowski wrote:
>> On 12/09/2023 15:31, Konrad Dybcio wrote:
>>> These clocks are now handled from within the icc framework and are
>>
>> That's a driver behavior, not hardware.
> I believe we've been over this already..
>
> The rationale behind this change is: that hardware, which falls
> under the "interconnect" class, was previously misrepresented as
> a bunch of clocks. There are clocks underneath, but accessing them
> directly would be equivalent to e.g. circumventing the PHY subsystem
> and initializing your UFS PHY from within the UFS device.
And every time one write such commit msg, how should we remember there
is some exception and actually it is about clock representation not CCF
or ICC framework.
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks
2023-09-13 8:53 ` Krzysztof Kozlowski
@ 2023-09-13 10:48 ` Konrad Dybcio
2023-09-13 11:14 ` Krzysztof Kozlowski
0 siblings, 1 reply; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-13 10:48 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 13.09.2023 10:53, Krzysztof Kozlowski wrote:
> On 13/09/2023 10:47, Konrad Dybcio wrote:
>> On 13.09.2023 09:07, Krzysztof Kozlowski wrote:
>>> On 12/09/2023 15:31, Konrad Dybcio wrote:
>>>> These clocks are now handled from within the icc framework and are
>>>
>>> That's a driver behavior, not hardware.
>> I believe we've been over this already..
>>
>> The rationale behind this change is: that hardware, which falls
>> under the "interconnect" class, was previously misrepresented as
>> a bunch of clocks. There are clocks underneath, but accessing them
>> directly would be equivalent to e.g. circumventing the PHY subsystem
>> and initializing your UFS PHY from within the UFS device.
>
> And every time one write such commit msg, how should we remember there
> is some exception and actually it is about clock representation not CCF
> or ICC framework.
So is your reply essentially "fine, but please make it clear in
each commit message"?
Konrad
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks
2023-09-13 10:48 ` Konrad Dybcio
@ 2023-09-13 11:14 ` Krzysztof Kozlowski
2023-09-13 11:14 ` Konrad Dybcio
0 siblings, 1 reply; 33+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-13 11:14 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 13/09/2023 12:48, Konrad Dybcio wrote:
> On 13.09.2023 10:53, Krzysztof Kozlowski wrote:
>> On 13/09/2023 10:47, Konrad Dybcio wrote:
>>> On 13.09.2023 09:07, Krzysztof Kozlowski wrote:
>>>> On 12/09/2023 15:31, Konrad Dybcio wrote:
>>>>> These clocks are now handled from within the icc framework and are
>>>>
>>>> That's a driver behavior, not hardware.
>>> I believe we've been over this already..
>>>
>>> The rationale behind this change is: that hardware, which falls
>>> under the "interconnect" class, was previously misrepresented as
>>> a bunch of clocks. There are clocks underneath, but accessing them
>>> directly would be equivalent to e.g. circumventing the PHY subsystem
>>> and initializing your UFS PHY from within the UFS device.
>>
>> And every time one write such commit msg, how should we remember there
>> is some exception and actually it is about clock representation not CCF
>> or ICC framework.
> So is your reply essentially "fine, but please make it clear in
> each commit message"?
I am fine with this change. If commit msg had such statement, I would
not have doubts :/
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks
2023-09-13 11:14 ` Krzysztof Kozlowski
@ 2023-09-13 11:14 ` Konrad Dybcio
2024-01-02 10:01 ` Aiqun Yu (Maria)
0 siblings, 1 reply; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-13 11:14 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 13.09.2023 13:14, Krzysztof Kozlowski wrote:
> On 13/09/2023 12:48, Konrad Dybcio wrote:
>> On 13.09.2023 10:53, Krzysztof Kozlowski wrote:
>>> On 13/09/2023 10:47, Konrad Dybcio wrote:
>>>> On 13.09.2023 09:07, Krzysztof Kozlowski wrote:
>>>>> On 12/09/2023 15:31, Konrad Dybcio wrote:
>>>>>> These clocks are now handled from within the icc framework and are
>>>>>
>>>>> That's a driver behavior, not hardware.
>>>> I believe we've been over this already..
>>>>
>>>> The rationale behind this change is: that hardware, which falls
>>>> under the "interconnect" class, was previously misrepresented as
>>>> a bunch of clocks. There are clocks underneath, but accessing them
>>>> directly would be equivalent to e.g. circumventing the PHY subsystem
>>>> and initializing your UFS PHY from within the UFS device.
>>>
>>> And every time one write such commit msg, how should we remember there
>>> is some exception and actually it is about clock representation not CCF
>>> or ICC framework.
>> So is your reply essentially "fine, but please make it clear in
>> each commit message"?
>
> I am fine with this change. If commit msg had such statement, I would
> not have doubts :/
Ok, I'll resend, thanks for confirming!
Konrad
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks
2023-09-13 7:13 ` Krzysztof Kozlowski
@ 2023-09-13 12:08 ` Konrad Dybcio
2023-09-14 6:26 ` Krzysztof Kozlowski
0 siblings, 1 reply; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-13 12:08 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 13.09.2023 09:13, Krzysztof Kozlowski wrote:
> On 12/09/2023 15:31, Konrad Dybcio wrote:
>> These clocks are now handled from within the icc framework and are
>> no longer registered from within the CCF. Remove them.
>>
>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>> ---
[...]
>> anoc2_smmu: iommu@16c0000 {
>> compatible = "qcom,sdm630-smmu-v2", "qcom,smmu-v2";
>> reg = <0x016c0000 0x40000>;
>> -
>> - assigned-clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
>> - assigned-clock-rates = <1000>;
>> - clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
>> - clock-names = "bus";
>
> This is also against bindings. After your patch #4, such bus clock (or
> other combinations) is still required.
So, we have 4 SMMU instances on this platform:
MMSS (described, iface, mem, mem_iface)
GPU (described, iface-mm, iface-smmu, bus-smmu)
ANOC2 (this one, no clocks after removing rpmcc bus)
LPASS (no clocks)
Should I then create a new entry in the bindings, replicating
what's there for msm8998[1] and dropping the entry with just "bus"
from anyOf?
Konrad
[1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/Documentation/devicetree/bindings/iommu/arm,smmu.yaml?h=next-20230913#n272
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks
2023-09-13 12:08 ` Konrad Dybcio
@ 2023-09-14 6:26 ` Krzysztof Kozlowski
2023-09-14 7:41 ` Konrad Dybcio
0 siblings, 1 reply; 33+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-14 6:26 UTC (permalink / raw)
To: Konrad Dybcio, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 13/09/2023 14:08, Konrad Dybcio wrote:
> On 13.09.2023 09:13, Krzysztof Kozlowski wrote:
>> On 12/09/2023 15:31, Konrad Dybcio wrote:
>>> These clocks are now handled from within the icc framework and are
>>> no longer registered from within the CCF. Remove them.
>>>
>>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>>> ---
> [...]
>
>>> anoc2_smmu: iommu@16c0000 {
>>> compatible = "qcom,sdm630-smmu-v2", "qcom,smmu-v2";
>>> reg = <0x016c0000 0x40000>;
>>> -
>>> - assigned-clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
>>> - assigned-clock-rates = <1000>;
>>> - clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
>>> - clock-names = "bus";
>>
>> This is also against bindings. After your patch #4, such bus clock (or
>> other combinations) is still required.
> So, we have 4 SMMU instances on this platform:
>
> MMSS (described, iface, mem, mem_iface)
> GPU (described, iface-mm, iface-smmu, bus-smmu)
>
> ANOC2 (this one, no clocks after removing rpmcc bus)
> LPASS (no clocks)
Ah, I did not notice it.
>
> Should I then create a new entry in the bindings, replicating
> what's there for msm8998[1] and dropping the entry with just "bus"
> from anyOf?
So this passes the bindings, right? anyOf: in the binding should allow
also no match, so this should be fine. However indeed we need to drop
the "bus" entry, because it is not valid anymore.
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks
2023-09-14 6:26 ` Krzysztof Kozlowski
@ 2023-09-14 7:41 ` Konrad Dybcio
0 siblings, 0 replies; 33+ messages in thread
From: Konrad Dybcio @ 2023-09-14 7:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Mathieu Poirier,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 14.09.2023 08:26, Krzysztof Kozlowski wrote:
> On 13/09/2023 14:08, Konrad Dybcio wrote:
>> On 13.09.2023 09:13, Krzysztof Kozlowski wrote:
>>> On 12/09/2023 15:31, Konrad Dybcio wrote:
>>>> These clocks are now handled from within the icc framework and are
>>>> no longer registered from within the CCF. Remove them.
>>>>
>>>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>>>> ---
>> [...]
>>
>>>> anoc2_smmu: iommu@16c0000 {
>>>> compatible = "qcom,sdm630-smmu-v2", "qcom,smmu-v2";
>>>> reg = <0x016c0000 0x40000>;
>>>> -
>>>> - assigned-clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
>>>> - assigned-clock-rates = <1000>;
>>>> - clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
>>>> - clock-names = "bus";
>>>
>>> This is also against bindings. After your patch #4, such bus clock (or
>>> other combinations) is still required.
>> So, we have 4 SMMU instances on this platform:
>>
>> MMSS (described, iface, mem, mem_iface)
>> GPU (described, iface-mm, iface-smmu, bus-smmu)
>>
>> ANOC2 (this one, no clocks after removing rpmcc bus)
>> LPASS (no clocks)
>
> Ah, I did not notice it.
>
>>
>> Should I then create a new entry in the bindings, replicating
>> what's there for msm8998[1] and dropping the entry with just "bus"
>> from anyOf?
>
> So this passes the bindings, right?
Yes
anyOf: in the binding should allow
> also no match, so this should be fine. However indeed we need to drop
> the "bus" entry, because it is not valid anymore.
Actually, looks like the LPASS smmu may require a single
clock. We can reuse that single-"bus"-clock entry for
HLOS1_VOTE_LPASS_ADSP_SMMU_CLK.
The device didn't crash when trying to access LPASS SMMU
with that clock absent, but I guess it may have just
been luck, things may change once more hardware is parked..
Konrad
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 00/14] Clean up RPM bus clocks remnants
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (13 preceding siblings ...)
2023-09-12 13:31 ` [PATCH v2 14/14] arm64: dts: qcom: sdm630: Fix USB2 clock-names order Konrad Dybcio
@ 2023-09-18 11:05 ` Will Deacon
2023-09-20 18:30 ` (subset) " Bjorn Andersson
15 siblings, 0 replies; 33+ messages in thread
From: Will Deacon @ 2023-09-18 11:05 UTC (permalink / raw)
To: Mathieu Poirier, Wesley Cheng, Manivannan Sadhasivam,
Joerg Roedel, Robin Murphy, Conor Dooley, Rob Herring,
Krzysztof Kozlowski, Sibi Sankar, Greg Kroah-Hartman,
Konrad Dybcio, Bjorn Andersson, Andy Gross
Cc: catalin.marinas, kernel-team, Will Deacon, Alexey Minnekhanov,
Krzysztof Kozlowski, iommu, linux-arm-msm, linux-kernel,
devicetree, linux-remoteproc, linux-arm-kernel, Marijn Suijten,
linux-usb
On Tue, 12 Sep 2023 15:31:38 +0200, Konrad Dybcio wrote:
> After the recent cleanups ([1], [2]) some in-tree abusers that directly
> accessed the RPM bus clocks, effectively circumventing and working
> against the efforts of the interconnect framework, were found.
>
> Patches 1-5 drop deprecated references and the rest attempt to stop
> direct bus clock abuses.
>
> [...]
Applied SMMU bindings fix to will (for-joerg/arm-smmu/fixes), thanks!
[04/14] dt-bindings: arm-smmu: Fix SDM630 clocks description
https://git.kernel.org/will/c/938ba2f252a5
Cheers,
--
Will
https://fixes.arm64.dev
https://next.arm64.dev
https://will.arm64.dev
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: (subset) [PATCH v2 00/14] Clean up RPM bus clocks remnants
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
` (14 preceding siblings ...)
2023-09-18 11:05 ` [PATCH v2 00/14] Clean up RPM bus clocks remnants Will Deacon
@ 2023-09-20 18:30 ` Bjorn Andersson
15 siblings, 0 replies; 33+ messages in thread
From: Bjorn Andersson @ 2023-09-20 18:30 UTC (permalink / raw)
To: Andy Gross, Mathieu Poirier, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Manivannan Sadhasivam, Sibi Sankar, Will Deacon,
Robin Murphy, Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng,
Konrad Dybcio
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb, Krzysztof Kozlowski
On Tue, 12 Sep 2023 15:31:38 +0200, Konrad Dybcio wrote:
> After the recent cleanups ([1], [2]) some in-tree abusers that directly
> accessed the RPM bus clocks, effectively circumventing and working
> against the efforts of the interconnect framework, were found.
>
> Patches 1-5 drop deprecated references and the rest attempt to stop
> direct bus clock abuses.
>
> [...]
Applied, thanks!
[08/14] dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock
commit: c4c5b47958529bc1de10260df0c583710853b516
[09/14] dt-bindings: remoteproc: qcom,msm8996-mss-pil: Remove PNoC clock
commit: e7781901449cbcff129d80a5d9021e9e96084ec4
[10/14] remoteproc: qcom: q6v5-mss: Remove PNoC clock from 8996 MSS
commit: e1592981c51bac38ea2041b642777b3ba30606a8
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks
2023-09-13 11:14 ` Konrad Dybcio
@ 2024-01-02 10:01 ` Aiqun Yu (Maria)
0 siblings, 0 replies; 33+ messages in thread
From: Aiqun Yu (Maria) @ 2024-01-02 10:01 UTC (permalink / raw)
To: Konrad Dybcio, Krzysztof Kozlowski, Andy Gross, Bjorn Andersson,
Mathieu Poirier, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Manivannan Sadhasivam, Sibi Sankar, Will Deacon, Robin Murphy,
Joerg Roedel, Greg Kroah-Hartman, Wesley Cheng
Cc: Marijn Suijten, Alexey Minnekhanov, linux-arm-msm,
linux-remoteproc, devicetree, linux-kernel, linux-arm-kernel,
iommu, linux-usb
On 9/13/2023 7:14 PM, Konrad Dybcio wrote:
> On 13.09.2023 13:14, Krzysztof Kozlowski wrote:
>> On 13/09/2023 12:48, Konrad Dybcio wrote:
>>> On 13.09.2023 10:53, Krzysztof Kozlowski wrote:
>>>> On 13/09/2023 10:47, Konrad Dybcio wrote:
>>>>> On 13.09.2023 09:07, Krzysztof Kozlowski wrote:
>>>>>> On 12/09/2023 15:31, Konrad Dybcio wrote:
>>>>>>> These clocks are now handled from within the icc framework and are
>>>>>>
>>>>>> That's a driver behavior, not hardware.
>>>>> I believe we've been over this already..
>>>>>
>>>>> The rationale behind this change is: that hardware, which falls
>>>>> under the "interconnect" class, was previously misrepresented as
>>>>> a bunch of clocks. There are clocks underneath, but accessing them
>>>>> directly would be equivalent to e.g. circumventing the PHY subsystem
>>>>> and initializing your UFS PHY from within the UFS device.
>>>>
>>>> And every time one write such commit msg, how should we remember there
>>>> is some exception and actually it is about clock representation not CCF
>>>> or ICC framework.
>>> So is your reply essentially "fine, but please make it clear in
>>> each commit message"?
>>
>> I am fine with this change. If commit msg had such statement, I would
>> not have doubts :/
> Ok, I'll resend, thanks for confirming!
Is there any one continue working on this?
The bindings already merged while the dtb is not consistent with current
binding files. So dt bindings checks are failed actually.
>
> Konrad
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Thx and BRs,
Aiqun(Maria) Yu
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks
2023-09-12 13:31 ` [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks Konrad Dybcio
2023-09-13 7:13 ` Krzysztof Kozlowski
@ 2024-01-02 21:17 ` Marijn Suijten
1 sibling, 0 replies; 33+ messages in thread
From: Marijn Suijten @ 2024-01-02 21:17 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Andy Gross, Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Manivannan Sadhasivam,
Sibi Sankar, Will Deacon, Robin Murphy, Joerg Roedel,
Greg Kroah-Hartman, Wesley Cheng, Alexey Minnekhanov,
linux-arm-msm, linux-remoteproc, devicetree, linux-kernel,
linux-arm-kernel, iommu, linux-usb, Dmitry Baryshkov
On 2023-09-12 15:31:44, Konrad Dybcio wrote:
> These clocks are now handled from within the icc framework and are
> no longer registered from within the CCF. Remove them.
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
This makes the USB and IOMMUs probe again on SDM630 devices like the Sony Xperia
XA2 Ultra. Thanks!
Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org>
+cc Dmitry who was talking to me about this SoC.
> ---
> arch/arm64/boot/dts/qcom/sdm630.dtsi | 49 +++++++-----------------------------
> 1 file changed, 9 insertions(+), 40 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> index ec6003212c4d..f11d2a07508c 100644
> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> @@ -605,9 +605,6 @@ bimc: interconnect@1008000 {
> compatible = "qcom,sdm660-bimc";
> reg = <0x01008000 0x78000>;
> #interconnect-cells = <1>;
> - clock-names = "bus", "bus_a";
> - clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
> - <&rpmcc RPM_SMD_BIMC_A_CLK>;
> };
>
> restart@10ac000 {
> @@ -619,28 +616,17 @@ cnoc: interconnect@1500000 {
> compatible = "qcom,sdm660-cnoc";
> reg = <0x01500000 0x10000>;
> #interconnect-cells = <1>;
> - clock-names = "bus", "bus_a";
> - clocks = <&rpmcc RPM_SMD_CNOC_CLK>,
> - <&rpmcc RPM_SMD_CNOC_A_CLK>;
> };
>
> snoc: interconnect@1626000 {
> compatible = "qcom,sdm660-snoc";
> reg = <0x01626000 0x7090>;
> #interconnect-cells = <1>;
> - clock-names = "bus", "bus_a";
> - clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
> - <&rpmcc RPM_SMD_SNOC_A_CLK>;
> };
>
> anoc2_smmu: iommu@16c0000 {
> compatible = "qcom,sdm630-smmu-v2", "qcom,smmu-v2";
> reg = <0x016c0000 0x40000>;
> -
> - assigned-clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
> - assigned-clock-rates = <1000>;
> - clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
> - clock-names = "bus";
> #global-interrupts = <2>;
> #iommu-cells = <1>;
>
> @@ -685,16 +671,12 @@ a2noc: interconnect@1704000 {
> compatible = "qcom,sdm660-a2noc";
> reg = <0x01704000 0xc100>;
> #interconnect-cells = <1>;
> - clock-names = "bus",
> - "bus_a",
> - "ipa",
> + clock-names = "ipa",
> "ufs_axi",
> "aggre2_ufs_axi",
> "aggre2_usb3_axi",
> "cfg_noc_usb2_axi";
> - clocks = <&rpmcc RPM_SMD_AGGR2_NOC_CLK>,
> - <&rpmcc RPM_SMD_AGGR2_NOC_A_CLK>,
> - <&rpmcc RPM_SMD_IPA_CLK>,
> + clocks = <&rpmcc RPM_SMD_IPA_CLK>,
> <&gcc GCC_UFS_AXI_CLK>,
> <&gcc GCC_AGGRE2_UFS_AXI_CLK>,
> <&gcc GCC_AGGRE2_USB3_AXI_CLK>,
> @@ -705,10 +687,8 @@ mnoc: interconnect@1745000 {
> compatible = "qcom,sdm660-mnoc";
> reg = <0x01745000 0xa010>;
> #interconnect-cells = <1>;
> - clock-names = "bus", "bus_a", "iface";
> - clocks = <&rpmcc RPM_SMD_MMSSNOC_AXI_CLK>,
> - <&rpmcc RPM_SMD_MMSSNOC_AXI_CLK_A>,
> - <&mmcc AHB_CLK_SRC>;
> + clock-names = "iface";
> + clocks = <&mmcc AHB_CLK_SRC>;
> };
>
> tsens: thermal-sensor@10ae000 {
> @@ -1228,20 +1208,16 @@ usb3: usb@a8f8800 {
> <&gcc GCC_USB30_MASTER_CLK>,
> <&gcc GCC_AGGRE2_USB3_AXI_CLK>,
> <&gcc GCC_USB30_SLEEP_CLK>,
> - <&gcc GCC_USB30_MOCK_UTMI_CLK>,
> - <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
> + <&gcc GCC_USB30_MOCK_UTMI_CLK>;
> clock-names = "cfg_noc",
> "core",
> "iface",
> "sleep",
> - "mock_utmi",
> - "bus";
> + "mock_utmi";
>
> assigned-clocks = <&gcc GCC_USB30_MOCK_UTMI_CLK>,
> - <&gcc GCC_USB30_MASTER_CLK>,
> - <&rpmcc RPM_SMD_AGGR2_NOC_CLK>;
> - assigned-clock-rates = <19200000>, <120000000>,
> - <19200000>;
> + <&gcc GCC_USB30_MASTER_CLK>;
> + assigned-clock-rates = <19200000>, <120000000>;
>
> interrupts = <GIC_SPI 347 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 243 IRQ_TYPE_LEVEL_HIGH>;
> @@ -2144,10 +2120,9 @@ mmss_smmu: iommu@cd00000 {
>
> clocks = <&mmcc MNOC_AHB_CLK>,
> <&mmcc BIMC_SMMU_AHB_CLK>,
> - <&rpmcc RPM_SMD_MMSSNOC_AXI_CLK>,
> <&mmcc BIMC_SMMU_AXI_CLK>;
> clock-names = "iface-mm", "iface-smmu",
> - "bus-mm", "bus-smmu";
> + "bus-smmu";
> #global-interrupts = <2>;
> #iommu-cells = <1>;
>
> @@ -2264,12 +2239,6 @@ gnoc: interconnect@17900000 {
> compatible = "qcom,sdm660-gnoc";
> reg = <0x17900000 0xe000>;
> #interconnect-cells = <1>;
> - /*
> - * This one apparently features no clocks,
> - * so let's not mess with the driver needlessly
> - */
> - clock-names = "bus", "bus_a";
> - clocks = <&xo_board>, <&xo_board>;
> };
>
> apcs_glb: mailbox@17911000 {
>
> --
> 2.42.0
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 33+ messages in thread
end of thread, other threads:[~2024-01-02 21:18 UTC | newest]
Thread overview: 33+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-09-12 13:31 [PATCH v2 00/14] Clean up RPM bus clocks remnants Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 01/14] arm64: dts: qcom: msm8916: Drop RPM bus clocks Konrad Dybcio
2023-09-13 7:07 ` Krzysztof Kozlowski
2023-09-13 8:47 ` Konrad Dybcio
2023-09-13 8:53 ` Krzysztof Kozlowski
2023-09-13 10:48 ` Konrad Dybcio
2023-09-13 11:14 ` Krzysztof Kozlowski
2023-09-13 11:14 ` Konrad Dybcio
2024-01-02 10:01 ` Aiqun Yu (Maria)
2023-09-12 13:31 ` [PATCH v2 02/14] arm64: dts: qcom: msm8996: " Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 03/14] arm64: dts: qcom: qcs404: " Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 04/14] dt-bindings: arm-smmu: Fix SDM630 clocks description Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 05/14] dt-bindings: usb: qcom,dwc3: Fix SDM660 clock description Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 06/14] arm64: dts: qcom: sdm630: Drop RPM bus clocks Konrad Dybcio
2023-09-13 7:13 ` Krzysztof Kozlowski
2023-09-13 12:08 ` Konrad Dybcio
2023-09-14 6:26 ` Krzysztof Kozlowski
2023-09-14 7:41 ` Konrad Dybcio
2024-01-02 21:17 ` Marijn Suijten
2023-09-12 13:31 ` [PATCH v2 07/14] arm64: dts: qcom: msm8939: " Konrad Dybcio
2023-09-13 7:15 ` Krzysztof Kozlowski
2023-09-12 13:31 ` [PATCH v2 08/14] dt-bindings: remoteproc: qcom,adsp: Remove AGGRE2 clock Konrad Dybcio
2023-09-13 7:16 ` Krzysztof Kozlowski
2023-09-12 13:31 ` [PATCH v2 09/14] dt-bindings: remoteproc: qcom,msm8996-mss-pil: Remove PNoC clock Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 10/14] remoteproc: qcom: q6v5-mss: Remove PNoC clock from 8996 MSS Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 11/14] arm64: dts: qcom: msm8998: Remove AGGRE2 clock from SLPI Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 12/14] arm64: dts: qcom: msm8996: " Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 13/14] arm64: dts: qcom: msm8996: Remove PNoC clock from MSS Konrad Dybcio
2023-09-12 13:31 ` [PATCH v2 14/14] arm64: dts: qcom: sdm630: Fix USB2 clock-names order Konrad Dybcio
2023-09-13 7:17 ` Krzysztof Kozlowski
2023-09-13 7:19 ` Krzysztof Kozlowski
2023-09-18 11:05 ` [PATCH v2 00/14] Clean up RPM bus clocks remnants Will Deacon
2023-09-20 18:30 ` (subset) " Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).