* [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node @ 2018-06-18 20:56 Douglas Anderson 2018-06-18 20:56 ` [PATCH 2/2] arm64: dts: sdm845: Add rpmh-clk node Douglas Anderson 2018-06-18 21:24 ` [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node Lina Iyer 0 siblings, 2 replies; 4+ messages in thread From: Douglas Anderson @ 2018-06-18 20:56 UTC (permalink / raw) To: linux-arm-kernel This adds the rpmh-rsc node to sdm845 based on the examples in the bindings. Signed-off-by: Douglas Anderson <dianders@chromium.org> --- arch/arm64/boot/dts/qcom/sdm845.dtsi | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi index cd308b84bed7..19b006293d3b 100644 --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi @@ -7,6 +7,7 @@ #include <dt-bindings/clock/qcom,gcc-sdm845.h> #include <dt-bindings/interrupt-controller/arm-gic.h> +#include <dt-bindings/soc/qcom,rpmh-rsc.h> / { interrupt-parent = <&intc>; @@ -984,6 +985,24 @@ #mbox-cells = <1>; }; + apps_rsc: rsc at 179c0000 { + label = "apps_rsc"; + compatible = "qcom,rpmh-rsc"; + reg = <0x179c0000 0x10000>, + <0x179d0000 0x10000>, + <0x179e0000 0x10000>; + reg-names = "drv-0", "drv-1", "drv-2"; + interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 4 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>; + qcom,tcs-offset = <0xd00>; + qcom,drv-id = <2>; + qcom,tcs-config = <SLEEP_TCS 3>, + <WAKE_TCS 3>, + <ACTIVE_TCS 2>, + <CONTROL_TCS 1>; + }; + intc: interrupt-controller at 17a00000 { compatible = "arm,gic-v3"; #address-cells = <1>; -- 2.18.0.rc1.244.gcf134e6275-goog ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 2/2] arm64: dts: sdm845: Add rpmh-clk node 2018-06-18 20:56 [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node Douglas Anderson @ 2018-06-18 20:56 ` Douglas Anderson 2018-06-18 21:24 ` [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node Lina Iyer 1 sibling, 0 replies; 4+ messages in thread From: Douglas Anderson @ 2018-06-18 20:56 UTC (permalink / raw) To: linux-arm-kernel This adds the rpmh-clk node to sdm845 based on the examples in the bindings. Signed-off-by: Douglas Anderson <dianders@chromium.org> --- NOTE: to apply this patch cleanly, apply it atop: arm64: dts: qcom: sdm845: Add I2C, SPI, and UART9 nodes https://patchwork.kernel.org/patch/10462691/ arch/arm64/boot/dts/qcom/sdm845.dtsi | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi index 19b006293d3b..c61ae815a697 100644 --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi @@ -6,6 +6,7 @@ */ #include <dt-bindings/clock/qcom,gcc-sdm845.h> +#include <dt-bindings/clock/qcom,rpmh.h> #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/soc/qcom,rpmh-rsc.h> @@ -1001,6 +1002,11 @@ <WAKE_TCS 3>, <ACTIVE_TCS 2>, <CONTROL_TCS 1>; + + rpmhcc: clock-controller { + compatible = "qcom,sdm845-rpmh-clk"; + #clock-cells = <1>; + }; }; intc: interrupt-controller at 17a00000 { -- 2.18.0.rc1.244.gcf134e6275-goog ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node 2018-06-18 20:56 [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node Douglas Anderson 2018-06-18 20:56 ` [PATCH 2/2] arm64: dts: sdm845: Add rpmh-clk node Douglas Anderson @ 2018-06-18 21:24 ` Lina Iyer 2018-06-18 21:52 ` Doug Anderson 1 sibling, 1 reply; 4+ messages in thread From: Lina Iyer @ 2018-06-18 21:24 UTC (permalink / raw) To: linux-arm-kernel On Mon, Jun 18 2018 at 14:56 -0600, Douglas Anderson wrote: >This adds the rpmh-rsc node to sdm845 based on the examples in the >bindings. > >Signed-off-by: Douglas Anderson <dianders@chromium.org> >--- > > arch/arm64/boot/dts/qcom/sdm845.dtsi | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > >diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi >index cd308b84bed7..19b006293d3b 100644 >--- a/arch/arm64/boot/dts/qcom/sdm845.dtsi >+++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi >@@ -7,6 +7,7 @@ > > #include <dt-bindings/clock/qcom,gcc-sdm845.h> > #include <dt-bindings/interrupt-controller/arm-gic.h> >+#include <dt-bindings/soc/qcom,rpmh-rsc.h> > > / { > interrupt-parent = <&intc>; >@@ -984,6 +985,24 @@ > #mbox-cells = <1>; > }; > >+ apps_rsc: rsc at 179c0000 { >+ label = "apps_rsc"; >+ compatible = "qcom,rpmh-rsc"; >+ reg = <0x179c0000 0x10000>, >+ <0x179d0000 0x10000>, >+ <0x179e0000 0x10000>; >+ reg-names = "drv-0", "drv-1", "drv-2"; >+ interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>, >+ <GIC_SPI 4 IRQ_TYPE_LEVEL_HIGH>, >+ <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>; >+ qcom,tcs-offset = <0xd00>; >+ qcom,drv-id = <2>; >+ qcom,tcs-config = <SLEEP_TCS 3>, >+ <WAKE_TCS 3>, >+ <ACTIVE_TCS 2>, >+ <CONTROL_TCS 1>; Sorry, my example had this incorrect order and I just noticed this. We will need to fix the example as well. The first TCS should be ACTIVE_TCS, then followed by SLEEP_TCS and WAKE_TCS. This order is important and should match what is set in the firmware. qcom,tcs-config = <ACTIVE_TCS 2>, <SLEEP_TCS 3>, <WAKE_TCS 3>, <CONTROL_TCS 1>; While the above configuration would work for now, it would fail, when we enable system low power modes, which would use TCSes 2-7 for sleep and wake set transitions from the firmware. Thanks, Lina >+ }; >+ > intc: interrupt-controller at 17a00000 { > compatible = "arm,gic-v3"; > #address-cells = <1>; >-- >2.18.0.rc1.244.gcf134e6275-goog > ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node 2018-06-18 21:24 ` [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node Lina Iyer @ 2018-06-18 21:52 ` Doug Anderson 0 siblings, 0 replies; 4+ messages in thread From: Doug Anderson @ 2018-06-18 21:52 UTC (permalink / raw) To: linux-arm-kernel Hi, On Mon, Jun 18, 2018 at 2:24 PM, Lina Iyer <ilina@codeaurora.org> wrote: > On Mon, Jun 18 2018 at 14:56 -0600, Douglas Anderson wrote: >> >> This adds the rpmh-rsc node to sdm845 based on the examples in the >> bindings. >> >> Signed-off-by: Douglas Anderson <dianders@chromium.org> >> --- >> >> arch/arm64/boot/dts/qcom/sdm845.dtsi | 19 +++++++++++++++++++ >> 1 file changed, 19 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi >> b/arch/arm64/boot/dts/qcom/sdm845.dtsi >> index cd308b84bed7..19b006293d3b 100644 >> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi >> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi >> @@ -7,6 +7,7 @@ >> >> #include <dt-bindings/clock/qcom,gcc-sdm845.h> >> #include <dt-bindings/interrupt-controller/arm-gic.h> >> +#include <dt-bindings/soc/qcom,rpmh-rsc.h> >> >> / { >> interrupt-parent = <&intc>; >> @@ -984,6 +985,24 @@ >> #mbox-cells = <1>; >> }; >> >> + apps_rsc: rsc at 179c0000 { >> + label = "apps_rsc"; >> + compatible = "qcom,rpmh-rsc"; >> + reg = <0x179c0000 0x10000>, >> + <0x179d0000 0x10000>, >> + <0x179e0000 0x10000>; >> + reg-names = "drv-0", "drv-1", "drv-2"; >> + interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 4 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>; >> + qcom,tcs-offset = <0xd00>; >> + qcom,drv-id = <2>; >> + qcom,tcs-config = <SLEEP_TCS 3>, >> + <WAKE_TCS 3>, >> + <ACTIVE_TCS 2>, >> + <CONTROL_TCS 1>; > > Sorry, my example had this incorrect order and I just noticed this. We > will need to fix the example as well. > > The first TCS should be ACTIVE_TCS, then followed by SLEEP_TCS and > WAKE_TCS. This order is important and should match what is set in the > firmware. > > qcom,tcs-config = <ACTIVE_TCS 2>, > <SLEEP_TCS 3>, > <WAKE_TCS 3>, > <CONTROL_TCS 1>; > > While the above configuration would work for now, it would fail, when we > enable system low power modes, which would use TCSes 2-7 for sleep and > wake set transitions from the firmware. Since I'm not expecting lots more feedback, I've gone ahead and sent v2 with this fix. -Doug ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-06-18 21:52 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-06-18 20:56 [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node Douglas Anderson 2018-06-18 20:56 ` [PATCH 2/2] arm64: dts: sdm845: Add rpmh-clk node Douglas Anderson 2018-06-18 21:24 ` [PATCH 1/2] arm64: dts: sdm845: Add rpmh-rsc node Lina Iyer 2018-06-18 21:52 ` Doug Anderson
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).