* [PATCH] arm64: dts: qcom: sm6115: Add interconnect nodes
@ 2022-11-30 10:45 Bhupesh Sharma
2023-01-18 23:40 ` Bjorn Andersson
2023-01-19 13:06 ` Dmitry Baryshkov
0 siblings, 2 replies; 5+ messages in thread
From: Bhupesh Sharma @ 2022-11-30 10:45 UTC (permalink / raw)
To: linux-arm-msm
Cc: devicetree, agross, bhupesh.sharma, bhupesh.linux, linux-kernel,
robh+dt, krzysztof.kozlowski, konrad.dybcio, a39.skl, andersson
Add the interconnect nodes inside SM6115 dtsi.
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>
Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
---
- Based on linux-next/master
- Depends on the SM6115 dt-binding and driver patchset, which can be
seen here: https://lore.kernel.org/linux-arm-msm/20221130103841.2266464-1-bhupesh.sharma@linaro.org/
arch/arm64/boot/dts/qcom/sm6115.dtsi | 51 ++++++++++++++++++++++++++++
1 file changed, 51 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/arm64/boot/dts/qcom/sm6115.dtsi
index e4a2440ce544..dad5ab3edf0e 100644
--- a/arch/arm64/boot/dts/qcom/sm6115.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm6115.dtsi
@@ -485,6 +485,57 @@ usb_1_hsphy: phy@1613000 {
status = "disabled";
};
+ snoc: interconnect@1880000 {
+ compatible = "qcom,sm6115-snoc";
+ reg = <0x01880000 0x60200>;
+ #interconnect-cells = <1>;
+ clock-names = "bus", "bus_a";
+ clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
+ <&rpmcc RPM_SMD_SNOC_A_CLK>;
+
+ clk_virt: interconnect-clk {
+ compatible = "qcom,sm6115-clk-virt";
+ #interconnect-cells = <1>;
+ clock-names = "bus", "bus_a";
+ clocks = <&rpmcc RPM_SMD_QUP_CLK>,
+ <&rpmcc RPM_SMD_QUP_A_CLK>;
+ };
+
+ mmnrt_virt: interconnect-mmnrt {
+ compatible = "qcom,sm6115-mmnrt-virt";
+ #interconnect-cells = <1>;
+ clock-names = "bus", "bus_a";
+ clocks = <&rpmcc RPM_SMD_MMNRT_CLK>,
+ <&rpmcc RPM_SMD_MMNRT_A_CLK>;
+ };
+
+ mmrt_virt: interconnect-mmrt {
+ compatible = "qcom,sm6115-mmrt-virt";
+ #interconnect-cells = <1>;
+ clock-names = "bus", "bus_a";
+ clocks = <&rpmcc RPM_SMD_MMRT_CLK>,
+ <&rpmcc RPM_SMD_MMRT_A_CLK>;
+ };
+ };
+
+ cnoc: interconnect@1900000 {
+ compatible = "qcom,sm6115-cnoc";
+ reg = <0x01900000 0x8200>;
+ #interconnect-cells = <1>;
+ clock-names = "bus", "bus_a";
+ clocks = <&rpmcc RPM_SMD_CNOC_CLK>,
+ <&rpmcc RPM_SMD_CNOC_A_CLK>;
+ };
+
+ bimc: interconnect@4480000 {
+ compatible = "qcom,sm6115-bimc";
+ reg = <0x04480000 0x80000>;
+ #interconnect-cells = <1>;
+ clock-names = "bus", "bus_a";
+ clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
+ <&rpmcc RPM_SMD_BIMC_A_CLK>;
+ };
+
qfprom@1b40000 {
compatible = "qcom,sm6115-qfprom", "qcom,qfprom";
reg = <0x01b40000 0x7000>;
--
2.38.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] arm64: dts: qcom: sm6115: Add interconnect nodes
2022-11-30 10:45 [PATCH] arm64: dts: qcom: sm6115: Add interconnect nodes Bhupesh Sharma
@ 2023-01-18 23:40 ` Bjorn Andersson
2023-01-19 6:11 ` Bhupesh Sharma
2023-01-19 13:06 ` Dmitry Baryshkov
1 sibling, 1 reply; 5+ messages in thread
From: Bjorn Andersson @ 2023-01-18 23:40 UTC (permalink / raw)
To: Bhupesh Sharma
Cc: linux-arm-msm, devicetree, agross, bhupesh.linux, linux-kernel,
robh+dt, krzysztof.kozlowski, konrad.dybcio, a39.skl
On Wed, Nov 30, 2022 at 04:15:19PM +0530, Bhupesh Sharma wrote:
> Add the interconnect nodes inside SM6115 dtsi.
>
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
Seems the driver series is waiting for a v2.
I'll drop this from my queue for now, please resubmit once the DT
binding has landed.
Thanks,
Bjorn
> ---
> - Based on linux-next/master
> - Depends on the SM6115 dt-binding and driver patchset, which can be
> seen here: https://lore.kernel.org/linux-arm-msm/20221130103841.2266464-1-bhupesh.sharma@linaro.org/
>
> arch/arm64/boot/dts/qcom/sm6115.dtsi | 51 ++++++++++++++++++++++++++++
> 1 file changed, 51 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> index e4a2440ce544..dad5ab3edf0e 100644
> --- a/arch/arm64/boot/dts/qcom/sm6115.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> @@ -485,6 +485,57 @@ usb_1_hsphy: phy@1613000 {
> status = "disabled";
> };
>
> + snoc: interconnect@1880000 {
> + compatible = "qcom,sm6115-snoc";
> + reg = <0x01880000 0x60200>;
> + #interconnect-cells = <1>;
> + clock-names = "bus", "bus_a";
> + clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
> + <&rpmcc RPM_SMD_SNOC_A_CLK>;
> +
> + clk_virt: interconnect-clk {
> + compatible = "qcom,sm6115-clk-virt";
> + #interconnect-cells = <1>;
> + clock-names = "bus", "bus_a";
> + clocks = <&rpmcc RPM_SMD_QUP_CLK>,
> + <&rpmcc RPM_SMD_QUP_A_CLK>;
> + };
> +
> + mmnrt_virt: interconnect-mmnrt {
> + compatible = "qcom,sm6115-mmnrt-virt";
> + #interconnect-cells = <1>;
> + clock-names = "bus", "bus_a";
> + clocks = <&rpmcc RPM_SMD_MMNRT_CLK>,
> + <&rpmcc RPM_SMD_MMNRT_A_CLK>;
> + };
> +
> + mmrt_virt: interconnect-mmrt {
> + compatible = "qcom,sm6115-mmrt-virt";
> + #interconnect-cells = <1>;
> + clock-names = "bus", "bus_a";
> + clocks = <&rpmcc RPM_SMD_MMRT_CLK>,
> + <&rpmcc RPM_SMD_MMRT_A_CLK>;
> + };
> + };
> +
> + cnoc: interconnect@1900000 {
> + compatible = "qcom,sm6115-cnoc";
> + reg = <0x01900000 0x8200>;
> + #interconnect-cells = <1>;
> + clock-names = "bus", "bus_a";
> + clocks = <&rpmcc RPM_SMD_CNOC_CLK>,
> + <&rpmcc RPM_SMD_CNOC_A_CLK>;
> + };
> +
> + bimc: interconnect@4480000 {
> + compatible = "qcom,sm6115-bimc";
> + reg = <0x04480000 0x80000>;
> + #interconnect-cells = <1>;
> + clock-names = "bus", "bus_a";
> + clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
> + <&rpmcc RPM_SMD_BIMC_A_CLK>;
> + };
> +
> qfprom@1b40000 {
> compatible = "qcom,sm6115-qfprom", "qcom,qfprom";
> reg = <0x01b40000 0x7000>;
> --
> 2.38.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] arm64: dts: qcom: sm6115: Add interconnect nodes
2023-01-18 23:40 ` Bjorn Andersson
@ 2023-01-19 6:11 ` Bhupesh Sharma
0 siblings, 0 replies; 5+ messages in thread
From: Bhupesh Sharma @ 2023-01-19 6:11 UTC (permalink / raw)
To: Bjorn Andersson
Cc: linux-arm-msm, devicetree, agross, bhupesh.linux, linux-kernel,
robh+dt, krzysztof.kozlowski, konrad.dybcio, a39.skl
On 1/19/23 5:10 AM, Bjorn Andersson wrote:
> On Wed, Nov 30, 2022 at 04:15:19PM +0530, Bhupesh Sharma wrote:
>> Add the interconnect nodes inside SM6115 dtsi.
>>
>> Cc: Bjorn Andersson <andersson@kernel.org>
>> Cc: Rob Herring <robh+dt@kernel.org>
>> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
>
> Seems the driver series is waiting for a v2.
>
> I'll drop this from my queue for now, please resubmit once the DT
> binding has landed.
Sure Bjorn. I will send a v2 shortly.
Thanks,
Bhupesh
>> ---
>> - Based on linux-next/master
>> - Depends on the SM6115 dt-binding and driver patchset, which can be
>> seen here: https://lore.kernel.org/linux-arm-msm/20221130103841.2266464-1-bhupesh.sharma@linaro.org/
>>
>> arch/arm64/boot/dts/qcom/sm6115.dtsi | 51 ++++++++++++++++++++++++++++
>> 1 file changed, 51 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/arm64/boot/dts/qcom/sm6115.dtsi
>> index e4a2440ce544..dad5ab3edf0e 100644
>> --- a/arch/arm64/boot/dts/qcom/sm6115.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sm6115.dtsi
>> @@ -485,6 +485,57 @@ usb_1_hsphy: phy@1613000 {
>> status = "disabled";
>> };
>>
>> + snoc: interconnect@1880000 {
>> + compatible = "qcom,sm6115-snoc";
>> + reg = <0x01880000 0x60200>;
>> + #interconnect-cells = <1>;
>> + clock-names = "bus", "bus_a";
>> + clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
>> + <&rpmcc RPM_SMD_SNOC_A_CLK>;
>> +
>> + clk_virt: interconnect-clk {
>> + compatible = "qcom,sm6115-clk-virt";
>> + #interconnect-cells = <1>;
>> + clock-names = "bus", "bus_a";
>> + clocks = <&rpmcc RPM_SMD_QUP_CLK>,
>> + <&rpmcc RPM_SMD_QUP_A_CLK>;
>> + };
>> +
>> + mmnrt_virt: interconnect-mmnrt {
>> + compatible = "qcom,sm6115-mmnrt-virt";
>> + #interconnect-cells = <1>;
>> + clock-names = "bus", "bus_a";
>> + clocks = <&rpmcc RPM_SMD_MMNRT_CLK>,
>> + <&rpmcc RPM_SMD_MMNRT_A_CLK>;
>> + };
>> +
>> + mmrt_virt: interconnect-mmrt {
>> + compatible = "qcom,sm6115-mmrt-virt";
>> + #interconnect-cells = <1>;
>> + clock-names = "bus", "bus_a";
>> + clocks = <&rpmcc RPM_SMD_MMRT_CLK>,
>> + <&rpmcc RPM_SMD_MMRT_A_CLK>;
>> + };
>> + };
>> +
>> + cnoc: interconnect@1900000 {
>> + compatible = "qcom,sm6115-cnoc";
>> + reg = <0x01900000 0x8200>;
>> + #interconnect-cells = <1>;
>> + clock-names = "bus", "bus_a";
>> + clocks = <&rpmcc RPM_SMD_CNOC_CLK>,
>> + <&rpmcc RPM_SMD_CNOC_A_CLK>;
>> + };
>> +
>> + bimc: interconnect@4480000 {
>> + compatible = "qcom,sm6115-bimc";
>> + reg = <0x04480000 0x80000>;
>> + #interconnect-cells = <1>;
>> + clock-names = "bus", "bus_a";
>> + clocks = <&rpmcc RPM_SMD_BIMC_CLK>,
>> + <&rpmcc RPM_SMD_BIMC_A_CLK>;
>> + };
>> +
>> qfprom@1b40000 {
>> compatible = "qcom,sm6115-qfprom", "qcom,qfprom";
>> reg = <0x01b40000 0x7000>;
>> --
>> 2.38.1
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sm6115: Add interconnect nodes
2022-11-30 10:45 [PATCH] arm64: dts: qcom: sm6115: Add interconnect nodes Bhupesh Sharma
2023-01-18 23:40 ` Bjorn Andersson
@ 2023-01-19 13:06 ` Dmitry Baryshkov
2023-01-19 14:23 ` Bhupesh Sharma
1 sibling, 1 reply; 5+ messages in thread
From: Dmitry Baryshkov @ 2023-01-19 13:06 UTC (permalink / raw)
To: Bhupesh Sharma, linux-arm-msm
Cc: devicetree, agross, bhupesh.linux, linux-kernel, robh+dt,
krzysztof.kozlowski, konrad.dybcio, a39.skl, andersson
On 30/11/2022 12:45, Bhupesh Sharma wrote:
> Add the interconnect nodes inside SM6115 dtsi.
>
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
> ---
> - Based on linux-next/master
> - Depends on the SM6115 dt-binding and driver patchset, which can be
> seen here: https://lore.kernel.org/linux-arm-msm/20221130103841.2266464-1-bhupesh.sharma@linaro.org/
>
> arch/arm64/boot/dts/qcom/sm6115.dtsi | 51 ++++++++++++++++++++++++++++
> 1 file changed, 51 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> index e4a2440ce544..dad5ab3edf0e 100644
> --- a/arch/arm64/boot/dts/qcom/sm6115.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> @@ -485,6 +485,57 @@ usb_1_hsphy: phy@1613000 {
> status = "disabled";
> };
>
> + snoc: interconnect@1880000 {
> + compatible = "qcom,sm6115-snoc";
> + reg = <0x01880000 0x60200>;
> + #interconnect-cells = <1>;
Should we use 2 here as we do now for most of interconnect drivers?
> + clock-names = "bus", "bus_a";
> + clocks = <&rpmcc RPM_SMD_SNOC_CLK>,
> + <&rpmcc RPM_SMD_SNOC_A_CLK>;
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] arm64: dts: qcom: sm6115: Add interconnect nodes
2023-01-19 13:06 ` Dmitry Baryshkov
@ 2023-01-19 14:23 ` Bhupesh Sharma
0 siblings, 0 replies; 5+ messages in thread
From: Bhupesh Sharma @ 2023-01-19 14:23 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: linux-arm-msm, devicetree, agross, bhupesh.linux, linux-kernel,
robh+dt, krzysztof.kozlowski, konrad.dybcio, a39.skl, andersson
Hi Dmitry,
On Thu, 19 Jan 2023 at 18:36, Dmitry Baryshkov
<dmitry.baryshkov@linaro.org> wrote:
>
> On 30/11/2022 12:45, Bhupesh Sharma wrote:
> > Add the interconnect nodes inside SM6115 dtsi.
> >
> > Cc: Bjorn Andersson <andersson@kernel.org>
> > Cc: Rob Herring <robh+dt@kernel.org>
> > Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
> > ---
> > - Based on linux-next/master
> > - Depends on the SM6115 dt-binding and driver patchset, which can be
> > seen here: https://lore.kernel.org/linux-arm-msm/20221130103841.2266464-1-bhupesh.sharma@linaro.org/
> >
> > arch/arm64/boot/dts/qcom/sm6115.dtsi | 51 ++++++++++++++++++++++++++++
> > 1 file changed, 51 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> > index e4a2440ce544..dad5ab3edf0e 100644
> > --- a/arch/arm64/boot/dts/qcom/sm6115.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sm6115.dtsi
> > @@ -485,6 +485,57 @@ usb_1_hsphy: phy@1613000 {
> > status = "disabled";
> > };
> >
> > + snoc: interconnect@1880000 {
> > + compatible = "qcom,sm6115-snoc";
> > + reg = <0x01880000 0x60200>;
> > + #interconnect-cells = <1>;
>
> Should we use 2 here as we do now for most of interconnect drivers?
Right now, we are using the value present in upstream qcs404.dtsi
which is the nearest in terms of similarity of the interconnect blocks
present on this SoC. But let me try and make this change in v2.
BTW the 'path tag' is optional, so I don't think that would be a
mandatory change.
Thanks,
Bhupesh
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-01-19 14:23 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-30 10:45 [PATCH] arm64: dts: qcom: sm6115: Add interconnect nodes Bhupesh Sharma
2023-01-18 23:40 ` Bjorn Andersson
2023-01-19 6:11 ` Bhupesh Sharma
2023-01-19 13:06 ` Dmitry Baryshkov
2023-01-19 14:23 ` Bhupesh Sharma
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).