* [PATCH 1/3] dt-bindings: interconnect: sdm660: Add missing a2noc qos clocks
2021-08-23 9:56 [PATCH 0/3] Add missing A2NoC QoS clocks for SDM660 interconnect driver Shawn Guo
@ 2021-08-23 9:56 ` Shawn Guo
2021-08-23 13:30 ` Rob Herring
2021-08-23 9:56 ` [PATCH 2/3] interconnect: qcom: " Shawn Guo
2021-08-23 9:56 ` [PATCH 3/3] arm64: dts: qcom: sdm630: " Shawn Guo
2 siblings, 1 reply; 8+ messages in thread
From: Shawn Guo @ 2021-08-23 9:56 UTC (permalink / raw)
To: Georgi Djakov
Cc: AngeloGioacchino Del Regno, Bjorn Andersson, Rob Herring,
devicetree, linux-arm-msm, linux-pm, Shawn Guo
It adds the missing a2noc clocks required for QoS registers programming
per downstream kernel[1].
[1] https://source.codeaurora.org/quic/la/kernel/msm-4.4/tree/arch/arm/boot/dts/qcom/sdm660-bus.dtsi?h=LA.UM.8.2.r1-04800-sdm660.0#n43
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
---
.../bindings/interconnect/qcom,sdm660.yaml | 41 ++++++++++++++++++-
1 file changed, 39 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/interconnect/qcom,sdm660.yaml b/Documentation/devicetree/bindings/interconnect/qcom,sdm660.yaml
index 29de7807df54..19bc3091ea40 100644
--- a/Documentation/devicetree/bindings/interconnect/qcom,sdm660.yaml
+++ b/Documentation/devicetree/bindings/interconnect/qcom,sdm660.yaml
@@ -72,6 +72,32 @@ allOf:
contains:
enum:
- qcom,sdm660-a2noc
+ then:
+ properties:
+ clocks:
+ items:
+ - description: Bus Clock.
+ - description: Bus A Clock.
+ - description: IPA Clock.
+ - description: UFS AXI Clock.
+ - description: Aggregate2 UFS AXI Clock.
+ - description: Aggregate2 USB3 AXI Clock.
+ - description: Config NoC USB2 AXI Clock.
+ clock-names:
+ items:
+ - const: bus
+ - const: bus_a
+ - const: ipa
+ - const: ufs_axi
+ - const: aggre2_ufs_axi
+ - const: aggre2_usb3_axi
+ - const: cfg_noc_usb2_axi
+
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
- qcom,sdm660-bimc
- qcom,sdm660-cnoc
- qcom,sdm660-gnoc
@@ -123,9 +149,20 @@ examples:
compatible = "qcom,sdm660-a2noc";
reg = <0x01704000 0xc100>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
+ clock-names = "bus",
+ "bus_a",
+ "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_AGGR2_NOC_A_CLK>,
+ <&rpmcc RPM_SMD_IPA_CLK>,
+ <&gcc GCC_UFS_AXI_CLK>,
+ <&gcc GCC_AGGRE2_UFS_AXI_CLK>,
+ <&gcc GCC_AGGRE2_USB3_AXI_CLK>,
+ <&gcc GCC_CFG_NOC_USB2_AXI_CLK>;
};
mnoc: interconnect@1745000 {
--
2.17.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 1/3] dt-bindings: interconnect: sdm660: Add missing a2noc qos clocks
2021-08-23 9:56 ` [PATCH 1/3] dt-bindings: interconnect: sdm660: Add missing a2noc qos clocks Shawn Guo
@ 2021-08-23 13:30 ` Rob Herring
0 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2021-08-23 13:30 UTC (permalink / raw)
To: Shawn Guo
Cc: AngeloGioacchino Del Regno, Bjorn Andersson, linux-arm-msm,
Rob Herring, devicetree, linux-pm, Georgi Djakov
On Mon, 23 Aug 2021 17:56:01 +0800, Shawn Guo wrote:
> It adds the missing a2noc clocks required for QoS registers programming
> per downstream kernel[1].
>
> [1] https://source.codeaurora.org/quic/la/kernel/msm-4.4/tree/arch/arm/boot/dts/qcom/sdm660-bus.dtsi?h=LA.UM.8.2.r1-04800-sdm660.0#n43
>
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
> ---
> .../bindings/interconnect/qcom,sdm660.yaml | 41 ++++++++++++++++++-
> 1 file changed, 39 insertions(+), 2 deletions(-)
>
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors:
Error: Documentation/devicetree/bindings/interconnect/qcom,sdm660.example.dts:63.32-33 syntax error
FATAL ERROR: Unable to parse input tree
make[1]: *** [scripts/Makefile.lib:380: Documentation/devicetree/bindings/interconnect/qcom,sdm660.example.dt.yaml] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:1419: dt_binding_check] Error 2
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/patch/1519617
This check can fail if there are any dependencies. The base for a patch
series is generally the most recent rc1.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/3] interconnect: qcom: sdm660: Add missing a2noc qos clocks
2021-08-23 9:56 [PATCH 0/3] Add missing A2NoC QoS clocks for SDM660 interconnect driver Shawn Guo
2021-08-23 9:56 ` [PATCH 1/3] dt-bindings: interconnect: sdm660: Add missing a2noc qos clocks Shawn Guo
@ 2021-08-23 9:56 ` Shawn Guo
2021-08-23 15:57 ` Bjorn Andersson
2021-08-23 9:56 ` [PATCH 3/3] arm64: dts: qcom: sdm630: " Shawn Guo
2 siblings, 1 reply; 8+ messages in thread
From: Shawn Guo @ 2021-08-23 9:56 UTC (permalink / raw)
To: Georgi Djakov
Cc: AngeloGioacchino Del Regno, Bjorn Andersson, Rob Herring,
devicetree, linux-arm-msm, linux-pm, Shawn Guo
It adds the missing a2noc clocks required for QoS registers programming
per downstream kernel[1]. Otherwise, qcom_icc_noc_set_qos_priority()
call on mas_ufs or mas_usb_hs node will simply result in a hardware hang
on SDM660 SoC.
[1] https://source.codeaurora.org/quic/la/kernel/msm-4.4/tree/arch/arm/boot/dts/qcom/sdm660-bus.dtsi?h=LA.UM.8.2.r1-04800-sdm660.0#n43
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
---
drivers/interconnect/qcom/sdm660.c | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/drivers/interconnect/qcom/sdm660.c b/drivers/interconnect/qcom/sdm660.c
index c89c991a80a0..661eb3635d21 100644
--- a/drivers/interconnect/qcom/sdm660.c
+++ b/drivers/interconnect/qcom/sdm660.c
@@ -174,6 +174,16 @@ static const struct clk_bulk_data bus_mm_clocks[] = {
{ .id = "iface" },
};
+static const struct clk_bulk_data bus_a2noc_clocks[] = {
+ { .id = "bus" },
+ { .id = "bus_a" },
+ { .id = "ipa" },
+ { .id = "ufs_axi" },
+ { .id = "aggre2_ufs_axi" },
+ { .id = "aggre2_usb3_axi" },
+ { .id = "cfg_noc_usb2_axi" },
+};
+
/**
* struct qcom_icc_provider - Qualcomm specific interconnect provider
* @provider: generic interconnect provider
@@ -811,6 +821,10 @@ static int qnoc_probe(struct platform_device *pdev)
qp->bus_clks = devm_kmemdup(dev, bus_mm_clocks,
sizeof(bus_mm_clocks), GFP_KERNEL);
qp->num_clks = ARRAY_SIZE(bus_mm_clocks);
+ } else if (of_device_is_compatible(dev->of_node, "qcom,sdm660-a2noc")) {
+ qp->bus_clks = devm_kmemdup(dev, bus_a2noc_clocks,
+ sizeof(bus_a2noc_clocks), GFP_KERNEL);
+ qp->num_clks = ARRAY_SIZE(bus_a2noc_clocks);
} else {
if (of_device_is_compatible(dev->of_node, "qcom,sdm660-bimc"))
qp->is_bimc_node = true;
--
2.17.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 2/3] interconnect: qcom: sdm660: Add missing a2noc qos clocks
2021-08-23 9:56 ` [PATCH 2/3] interconnect: qcom: " Shawn Guo
@ 2021-08-23 15:57 ` Bjorn Andersson
0 siblings, 0 replies; 8+ messages in thread
From: Bjorn Andersson @ 2021-08-23 15:57 UTC (permalink / raw)
To: Shawn Guo
Cc: Georgi Djakov, AngeloGioacchino Del Regno, Rob Herring,
devicetree, linux-arm-msm, linux-pm
On Mon 23 Aug 02:56 PDT 2021, Shawn Guo wrote:
> It adds the missing a2noc clocks required for QoS registers programming
> per downstream kernel[1]. Otherwise, qcom_icc_noc_set_qos_priority()
> call on mas_ufs or mas_usb_hs node will simply result in a hardware hang
> on SDM660 SoC.
>
> [1] https://source.codeaurora.org/quic/la/kernel/msm-4.4/tree/arch/arm/boot/dts/qcom/sdm660-bus.dtsi?h=LA.UM.8.2.r1-04800-sdm660.0#n43
>
Tested-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Regards,
Bjorn
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
> ---
> drivers/interconnect/qcom/sdm660.c | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/drivers/interconnect/qcom/sdm660.c b/drivers/interconnect/qcom/sdm660.c
> index c89c991a80a0..661eb3635d21 100644
> --- a/drivers/interconnect/qcom/sdm660.c
> +++ b/drivers/interconnect/qcom/sdm660.c
> @@ -174,6 +174,16 @@ static const struct clk_bulk_data bus_mm_clocks[] = {
> { .id = "iface" },
> };
>
> +static const struct clk_bulk_data bus_a2noc_clocks[] = {
> + { .id = "bus" },
> + { .id = "bus_a" },
> + { .id = "ipa" },
> + { .id = "ufs_axi" },
> + { .id = "aggre2_ufs_axi" },
> + { .id = "aggre2_usb3_axi" },
> + { .id = "cfg_noc_usb2_axi" },
> +};
> +
> /**
> * struct qcom_icc_provider - Qualcomm specific interconnect provider
> * @provider: generic interconnect provider
> @@ -811,6 +821,10 @@ static int qnoc_probe(struct platform_device *pdev)
> qp->bus_clks = devm_kmemdup(dev, bus_mm_clocks,
> sizeof(bus_mm_clocks), GFP_KERNEL);
> qp->num_clks = ARRAY_SIZE(bus_mm_clocks);
> + } else if (of_device_is_compatible(dev->of_node, "qcom,sdm660-a2noc")) {
> + qp->bus_clks = devm_kmemdup(dev, bus_a2noc_clocks,
> + sizeof(bus_a2noc_clocks), GFP_KERNEL);
> + qp->num_clks = ARRAY_SIZE(bus_a2noc_clocks);
> } else {
> if (of_device_is_compatible(dev->of_node, "qcom,sdm660-bimc"))
> qp->is_bimc_node = true;
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 3/3] arm64: dts: qcom: sdm630: Add missing a2noc qos clocks
2021-08-23 9:56 [PATCH 0/3] Add missing A2NoC QoS clocks for SDM660 interconnect driver Shawn Guo
2021-08-23 9:56 ` [PATCH 1/3] dt-bindings: interconnect: sdm660: Add missing a2noc qos clocks Shawn Guo
2021-08-23 9:56 ` [PATCH 2/3] interconnect: qcom: " Shawn Guo
@ 2021-08-23 9:56 ` Shawn Guo
2021-08-23 16:51 ` Dmitry Baryshkov
2 siblings, 1 reply; 8+ messages in thread
From: Shawn Guo @ 2021-08-23 9:56 UTC (permalink / raw)
To: Georgi Djakov
Cc: AngeloGioacchino Del Regno, Bjorn Andersson, Rob Herring,
devicetree, linux-arm-msm, linux-pm, Shawn Guo
It adds the missing a2noc clocks required for QoS registers programming
per downstream kernel[1].
[1] https://source.codeaurora.org/quic/la/kernel/msm-4.4/tree/arch/arm/boot/dts/qcom/sdm660-bus.dtsi?h=LA.UM.8.2.r1-04800-sdm660.0#n43
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm630.dtsi | 17 ++++++++++++++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index 9153e6616ba4..b3a7f3bf1560 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -652,11 +652,22 @@
a2noc: interconnect@1704000 {
compatible = "qcom,sdm660-a2noc";
- reg = <0x01704000 0xc100>;
+ reg = <0x01704000 0x1c000>;
#interconnect-cells = <1>;
- clock-names = "bus", "bus_a";
+ clock-names = "bus",
+ "bus_a",
+ "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_AGGR2_NOC_A_CLK>,
+ <&rpmcc RPM_SMD_IPA_CLK>,
+ <&gcc GCC_UFS_AXI_CLK>,
+ <&gcc GCC_AGGRE2_UFS_AXI_CLK>,
+ <&gcc GCC_AGGRE2_USB3_AXI_CLK>,
+ <&gcc GCC_CFG_NOC_USB2_AXI_CLK>;
};
mnoc: interconnect@1745000 {
--
2.17.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 3/3] arm64: dts: qcom: sdm630: Add missing a2noc qos clocks
2021-08-23 9:56 ` [PATCH 3/3] arm64: dts: qcom: sdm630: " Shawn Guo
@ 2021-08-23 16:51 ` Dmitry Baryshkov
2021-08-24 1:17 ` Shawn Guo
0 siblings, 1 reply; 8+ messages in thread
From: Dmitry Baryshkov @ 2021-08-23 16:51 UTC (permalink / raw)
To: Shawn Guo
Cc: Georgi Djakov, AngeloGioacchino Del Regno, Bjorn Andersson,
Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list:DRM DRIVER FOR MSM ADRENO GPU, Linux PM
On Mon, 23 Aug 2021 at 12:56, Shawn Guo <shawn.guo@linaro.org> wrote:
>
> It adds the missing a2noc clocks required for QoS registers programming
> per downstream kernel[1].
>
> [1] https://source.codeaurora.org/quic/la/kernel/msm-4.4/tree/arch/arm/boot/dts/qcom/sdm660-bus.dtsi?h=LA.UM.8.2.r1-04800-sdm660.0#n43
>
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/sdm630.dtsi | 17 ++++++++++++++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> index 9153e6616ba4..b3a7f3bf1560 100644
> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> @@ -652,11 +652,22 @@
>
> a2noc: interconnect@1704000 {
> compatible = "qcom,sdm660-a2noc";
> - reg = <0x01704000 0xc100>;
> + reg = <0x01704000 0x1c000>;
Shawn, as you are at it, do we want to keep these nocs shifted
compared to the downstream dtsi (so that the offset of QoS registers
is 0) or we'd better introduce QoS register offset and move noc start
address to the same address as in downstream?
> #interconnect-cells = <1>;
> - clock-names = "bus", "bus_a";
> + clock-names = "bus",
> + "bus_a",
> + "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_AGGR2_NOC_A_CLK>,
> + <&rpmcc RPM_SMD_IPA_CLK>,
> + <&gcc GCC_UFS_AXI_CLK>,
> + <&gcc GCC_AGGRE2_UFS_AXI_CLK>,
> + <&gcc GCC_AGGRE2_USB3_AXI_CLK>,
> + <&gcc GCC_CFG_NOC_USB2_AXI_CLK>;
> };
>
> mnoc: interconnect@1745000 {
> --
> 2.17.1
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH 3/3] arm64: dts: qcom: sdm630: Add missing a2noc qos clocks
2021-08-23 16:51 ` Dmitry Baryshkov
@ 2021-08-24 1:17 ` Shawn Guo
0 siblings, 0 replies; 8+ messages in thread
From: Shawn Guo @ 2021-08-24 1:17 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Georgi Djakov, AngeloGioacchino Del Regno, Bjorn Andersson,
Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list:DRM DRIVER FOR MSM ADRENO GPU, Linux PM
On Mon, Aug 23, 2021 at 07:51:32PM +0300, Dmitry Baryshkov wrote:
> On Mon, 23 Aug 2021 at 12:56, Shawn Guo <shawn.guo@linaro.org> wrote:
> >
> > It adds the missing a2noc clocks required for QoS registers programming
> > per downstream kernel[1].
> >
> > [1] https://source.codeaurora.org/quic/la/kernel/msm-4.4/tree/arch/arm/boot/dts/qcom/sdm660-bus.dtsi?h=LA.UM.8.2.r1-04800-sdm660.0#n43
> >
> > Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
> > ---
> > arch/arm64/boot/dts/qcom/sdm630.dtsi | 17 ++++++++++++++---
> > 1 file changed, 14 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> > index 9153e6616ba4..b3a7f3bf1560 100644
> > --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> > @@ -652,11 +652,22 @@
> >
> > a2noc: interconnect@1704000 {
> > compatible = "qcom,sdm660-a2noc";
> > - reg = <0x01704000 0xc100>;
> > + reg = <0x01704000 0x1c000>;
>
> Shawn, as you are at it, do we want to keep these nocs shifted
> compared to the downstream dtsi (so that the offset of QoS registers
> is 0) or we'd better introduce QoS register offset and move noc start
> address to the same address as in downstream?
Dmitry, thanks for spotting this! This is really an unintended leftover
from debugging. I will drop it in v2. For address alignment with
downstream, I do not really have a strong opinion and I can live with
either of them :)
Shawn
> > #interconnect-cells = <1>;
> > - clock-names = "bus", "bus_a";
> > + clock-names = "bus",
> > + "bus_a",
> > + "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_AGGR2_NOC_A_CLK>,
> > + <&rpmcc RPM_SMD_IPA_CLK>,
> > + <&gcc GCC_UFS_AXI_CLK>,
> > + <&gcc GCC_AGGRE2_UFS_AXI_CLK>,
> > + <&gcc GCC_AGGRE2_USB3_AXI_CLK>,
> > + <&gcc GCC_CFG_NOC_USB2_AXI_CLK>;
> > };
> >
> > mnoc: interconnect@1745000 {
> > --
> > 2.17.1
> >
>
>
> --
> With best wishes
> Dmitry
^ permalink raw reply [flat|nested] 8+ messages in thread