[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <b2b68cc6-1dd9-25ed-c65b-c4bf3a5a4f16@linaro.org>
Date: Sat, 22 Jul 2023 11:41:57 +0200
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Konrad Dybcio <konrad.dybcio@...aro.org>,
Andy Gross <agross@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Georgi Djakov <djakov@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>,
Shawn Guo <shawn.guo@...aro.org>
Cc: Marijn Suijten <marijn.suijten@...ainline.org>,
Konrad Dybcio <konradybcio@...nel.org>,
linux-arm-msm@...r.kernel.org, linux-pm@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 5/7] dt-bindings: interconnect: qcom: Fix and separate out
SDM660
On 21/07/2023 15:54, Konrad Dybcio wrote:
> Separate out SDM660 icc bindings from the common file and fix the
> clocks description by removing the wrong internal RPM bus clock
> representation that we've been carrying for years.
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@...aro.org>
...
> +
> +examples:
> + - |
> + #include <dt-bindings/clock/qcom,gcc-sdm660.h>
> + #include <dt-bindings/clock/qcom,mmcc-sdm660.h>
> + #include <dt-bindings/clock/qcom,rpmcc.h>
> +
> + bimc: interconnect@...8000 {
> + compatible = "qcom,sdm660-bimc";
> + reg = <0x01008000 0x78000>;
> + #interconnect-cells = <1>;
> + };
> +
> + cnoc: interconnect@...0000 {
> + compatible = "qcom,sdm660-cnoc";
> + reg = <0x01500000 0x10000>;
> + #interconnect-cells = <1>;
> + };
> +
> + snoc: interconnect@...6000 {
> + compatible = "qcom,sdm660-snoc";
> + reg = <0x01626000 0x7090>;
> + #interconnect-cells = <1>;
> + };
> +
> + a2noc: interconnect@...4000 {
> + compatible = "qcom,sdm660-a2noc";
> + reg = <0x01704000 0xc100>;
> + #interconnect-cells = <1>;
> + clocks = <&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>;
> + clock-names = "ipa",
> + "ufs_axi",
> + "aggre2_ufs_axi",
> + "aggre2_usb3_axi",
> + "cfg_noc_usb2_axi";
> + };
Keep only two examples.
Best regards,
Krzysztof
Powered by blists - more mailing lists