From: Konrad Dybcio <konrad.dybcio@linaro.org>
To: Richard Acayan <mailingradian@gmail.com>,
Andy Gross <agross@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org
Cc: Odelu Kukatla <quic_okukatla@quicinc.com>, Luca Weiss <luca@z3ntu.xyz>
Subject: Re: [PATCH v4 2/2] arm64: dts: qcom: sdm670: add opps for peripherals
Date: Wed, 1 Feb 2023 19:51:31 +0100 [thread overview]
Message-ID: <6c4cf46d-e4cf-dc9c-193a-e5dd77723b9f@linaro.org> (raw)
In-Reply-To: <20230201010020.84586-3-mailingradian@gmail.com>
On 1.02.2023 02:00, Richard Acayan wrote:
> The interconnects are now in place. Add Operating Performance Points for
> them to allow the kernel to properly manage them.
You're really mostly adding interconnect paths in this patch.
>
> Signed-off-by: Richard Acayan <mailingradian@gmail.com>
> ---
For the contents though:
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
> arch/arm64/boot/dts/qcom/sdm670.dtsi | 109 +++++++++++++++++++++++++++
> 1 file changed, 109 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm670.dtsi b/arch/arm64/boot/dts/qcom/sdm670.dtsi
> index 02f14692dd9d..c5f839dd1c6e 100644
> --- a/arch/arm64/boot/dts/qcom/sdm670.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm670.dtsi
> @@ -10,6 +10,7 @@
> #include <dt-bindings/clock/qcom,rpmh.h>
> #include <dt-bindings/dma/qcom-gpi.h>
> #include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/interconnect/qcom,sdm670-rpmh.h>
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> #include <dt-bindings/phy/phy-qcom-qusb2.h>
> #include <dt-bindings/power/qcom-rpmpd.h>
> @@ -430,6 +431,10 @@ sdhc_1: mmc@7c4000 {
> <&gcc GCC_SDCC1_ICE_CORE_CLK>,
> <&gcc GCC_AGGRE_UFS_PHY_AXI_CLK>;
> clock-names = "iface", "core", "xo", "ice", "bus";
> + interconnects = <&aggre1_noc MASTER_EMMC 0 &aggre1_noc SLAVE_A1NOC_SNOC 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_EMMC_CFG 0>;
> + interconnect-names = "sdhc-ddr", "cpu-sdhc";
> + operating-points-v2 = <&sdhc1_opp_table>;
>
> iommus = <&apps_smmu 0x140 0xf>;
>
> @@ -442,6 +447,38 @@ sdhc_1: mmc@7c4000 {
> non-removable;
>
> status = "disabled";
> +
> + sdhc1_opp_table: opp-table {
> + compatible = "operating-points-v2";
> +
> + opp-20000000 {
> + opp-hz = /bits/ 64 <20000000>;
> + required-opps = <&rpmhpd_opp_min_svs>;
> + opp-peak-kBps = <80000 80000>;
> + opp-avg-kBps = <52286 80000>;
> + };
> +
> + opp-50000000 {
> + opp-hz = /bits/ 64 <50000000>;
> + required-opps = <&rpmhpd_opp_low_svs>;
> + opp-peak-kBps = <200000 100000>;
> + opp-avg-kBps = <130718 100000>;
> + };
> +
> + opp-100000000 {
> + opp-hz = /bits/ 64 <100000000>;
> + required-opps = <&rpmhpd_opp_svs>;
> + opp-peak-kBps = <200000 130000>;
> + opp-avg-kBps = <130718 130000>;
> + };
> +
> + opp-384000000 {
> + opp-hz = /bits/ 64 <384000000>;
> + required-opps = <&rpmhpd_opp_nom>;
> + opp-peak-kBps = <4096000 4096000>;
> + opp-avg-kBps = <1338562 1338562>;
> + };
> + };
> };
>
> gpi_dma0: dma-controller@800000 {
> @@ -477,6 +514,8 @@ qupv3_id_0: geniqup@8c0000 {
> #address-cells = <2>;
> #size-cells = <2>;
> ranges;
> + interconnects = <&aggre1_noc MASTER_BLSP_1 0 &config_noc SLAVE_BLSP_1 0>;
> + interconnect-names = "qup-core";
> status = "disabled";
>
> i2c0: i2c@880000 {
> @@ -490,6 +529,10 @@ i2c0: i2c@880000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre1_noc MASTER_BLSP_1 0 &config_noc SLAVE_BLSP_1 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_1 0>,
> + <&aggre1_noc MASTER_BLSP_1 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma0 0 0 QCOM_GPI_I2C>,
> <&gpi_dma0 1 0 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -507,6 +550,10 @@ i2c1: i2c@884000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre1_noc MASTER_BLSP_1 0 &config_noc SLAVE_BLSP_1 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_1 0>,
> + <&aggre1_noc MASTER_BLSP_1 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma0 0 1 QCOM_GPI_I2C>,
> <&gpi_dma0 1 1 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -524,6 +571,10 @@ i2c2: i2c@888000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre1_noc MASTER_BLSP_1 0 &config_noc SLAVE_BLSP_1 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_1 0>,
> + <&aggre1_noc MASTER_BLSP_1 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma0 0 2 QCOM_GPI_I2C>,
> <&gpi_dma0 1 2 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -541,6 +592,10 @@ i2c3: i2c@88c000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre1_noc MASTER_BLSP_1 0 &config_noc SLAVE_BLSP_1 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_1 0>,
> + <&aggre1_noc MASTER_BLSP_1 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma0 0 3 QCOM_GPI_I2C>,
> <&gpi_dma0 1 3 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -558,6 +613,10 @@ i2c4: i2c@890000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre1_noc MASTER_BLSP_1 0 &config_noc SLAVE_BLSP_1 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_1 0>,
> + <&aggre1_noc MASTER_BLSP_1 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma0 0 4 QCOM_GPI_I2C>,
> <&gpi_dma0 1 4 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -575,6 +634,10 @@ i2c5: i2c@894000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre1_noc MASTER_BLSP_1 0 &config_noc SLAVE_BLSP_1 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_1 0>,
> + <&aggre1_noc MASTER_BLSP_1 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma0 0 5 QCOM_GPI_I2C>,
> <&gpi_dma0 1 5 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -592,6 +655,10 @@ i2c6: i2c@898000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre1_noc MASTER_BLSP_1 0 &config_noc SLAVE_BLSP_1 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_1 0>,
> + <&aggre1_noc MASTER_BLSP_1 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma0 0 6 QCOM_GPI_I2C>,
> <&gpi_dma0 1 6 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -609,6 +676,10 @@ i2c7: i2c@89c000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre1_noc MASTER_BLSP_1 0 &config_noc SLAVE_BLSP_1 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_1 0>,
> + <&aggre1_noc MASTER_BLSP_1 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma0 0 7 QCOM_GPI_I2C>,
> <&gpi_dma0 1 7 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -649,6 +720,8 @@ qupv3_id_1: geniqup@ac0000 {
> #address-cells = <2>;
> #size-cells = <2>;
> ranges;
> + interconnects = <&aggre2_noc MASTER_BLSP_2 0 &config_noc SLAVE_BLSP_2 0>;
> + interconnect-names = "qup-core";
> status = "disabled";
>
> i2c8: i2c@a80000 {
> @@ -662,6 +735,10 @@ i2c8: i2c@a80000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre2_noc MASTER_BLSP_2 0 &config_noc SLAVE_BLSP_2 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_2 0>,
> + <&aggre2_noc MASTER_BLSP_2 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma1 0 0 QCOM_GPI_I2C>,
> <&gpi_dma1 1 0 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -679,6 +756,10 @@ i2c9: i2c@a84000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre2_noc MASTER_BLSP_2 0 &config_noc SLAVE_BLSP_2 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_2 0>,
> + <&aggre2_noc MASTER_BLSP_2 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma1 0 1 QCOM_GPI_I2C>,
> <&gpi_dma1 1 1 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -696,6 +777,10 @@ i2c10: i2c@a88000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre2_noc MASTER_BLSP_2 0 &config_noc SLAVE_BLSP_2 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_2 0>,
> + <&aggre2_noc MASTER_BLSP_2 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma1 0 2 QCOM_GPI_I2C>,
> <&gpi_dma1 1 2 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -713,6 +798,10 @@ i2c11: i2c@a8c000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre2_noc MASTER_BLSP_2 0 &config_noc SLAVE_BLSP_2 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_2 0>,
> + <&aggre2_noc MASTER_BLSP_2 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma1 0 3 QCOM_GPI_I2C>,
> <&gpi_dma1 1 3 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -730,6 +819,10 @@ i2c12: i2c@a90000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre2_noc MASTER_BLSP_2 0 &config_noc SLAVE_BLSP_2 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_2 0>,
> + <&aggre2_noc MASTER_BLSP_2 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma1 0 4 QCOM_GPI_I2C>,
> <&gpi_dma1 1 4 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -747,6 +840,10 @@ i2c13: i2c@a94000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre2_noc MASTER_BLSP_2 0 &config_noc SLAVE_BLSP_2 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_2 0>,
> + <&aggre2_noc MASTER_BLSP_2 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma1 0 5 QCOM_GPI_I2C>,
> <&gpi_dma1 1 5 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -764,6 +861,10 @@ i2c14: i2c@a98000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre2_noc MASTER_BLSP_2 0 &config_noc SLAVE_BLSP_2 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_2 0>,
> + <&aggre2_noc MASTER_BLSP_2 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma1 0 6 QCOM_GPI_I2C>,
> <&gpi_dma1 1 6 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -781,6 +882,10 @@ i2c15: i2c@a9c000 {
> #address-cells = <1>;
> #size-cells = <0>;
> power-domains = <&rpmhpd SDM670_CX>;
> + interconnects = <&aggre2_noc MASTER_BLSP_2 0 &config_noc SLAVE_BLSP_2 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_BLSP_2 0>,
> + <&aggre2_noc MASTER_BLSP_2 0 &mem_noc SLAVE_EBI_CH0 0>;
> + interconnect-names = "qup-core", "qup-config", "qup-memory";
> dmas = <&gpi_dma1 0 7 QCOM_GPI_I2C>,
> <&gpi_dma1 1 7 QCOM_GPI_I2C>;
> dma-names = "tx", "rx";
> @@ -1028,6 +1133,10 @@ usb_1: usb@a6f8800 {
>
> resets = <&gcc GCC_USB30_PRIM_BCR>;
>
> + interconnects = <&aggre2_noc MASTER_USB3 0 &mem_noc SLAVE_EBI_CH0 0>,
> + <&gladiator_noc MASTER_AMPSS_M0 0 &config_noc SLAVE_USB3 0>;
> + interconnect-names = "usb-ddr", "apps-usb";
> +
> status = "disabled";
>
> usb_1_dwc3: usb@a600000 {
next prev parent reply other threads:[~2023-02-01 18:51 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-01 1:00 [PATCH v4 0/2] SDM670 Interconnects (now with dts patches only) Richard Acayan
2023-02-01 1:00 ` [PATCH v4 1/2] arm64: dts: qcom: sdm670: add interconnects Richard Acayan
2023-02-01 1:00 ` [PATCH v4 2/2] arm64: dts: qcom: sdm670: add opps for peripherals Richard Acayan
2023-02-01 18:51 ` Konrad Dybcio [this message]
2023-02-09 4:23 ` [PATCH v4 0/2] SDM670 Interconnects (now with dts patches only) Bjorn Andersson
2023-03-15 23:35 ` (subset) " Bjorn Andersson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6c4cf46d-e4cf-dc9c-193a-e5dd77723b9f@linaro.org \
--to=konrad.dybcio@linaro.org \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=luca@z3ntu.xyz \
--cc=mailingradian@gmail.com \
--cc=quic_okukatla@quicinc.com \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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 NNTP newsgroup(s).