diff mbox series

[6/6] arm64: dts: ti: k3-am62p: Add partial-io wakeup sources

Message ID 20240523080225.1288617-7-msp@baylibre.com (mailing list archive)
State New, archived
Headers show
Series firmware: ti_sci: Partial-IO support | expand

Commit Message

Markus Schneider-Pargmann May 23, 2024, 8:02 a.m. UTC
From: Vibhore Vardhan <vibhore@ti.com>

In Partial-IO mode there are a number of possible wakeup sources. Add
the list of phandles to these wakeup sources. Based on the patch for
AM62a.

Signed-off-by: Vibhore Vardhan <vibhore@ti.com>
Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com>
---
 arch/arm64/boot/dts/ti/k3-am62p.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Nishanth Menon May 24, 2024, 5:54 a.m. UTC | #1
On 10:02-20240523, Markus Schneider-Pargmann wrote:
> From: Vibhore Vardhan <vibhore@ti.com>
> 
> In Partial-IO mode there are a number of possible wakeup sources. Add
> the list of phandles to these wakeup sources. Based on the patch for
> AM62a.
> 
> Signed-off-by: Vibhore Vardhan <vibhore@ti.com>
> Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am62p.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am62p.dtsi b/arch/arm64/boot/dts/ti/k3-am62p.dtsi
> index 94babc412575..4d43cc972056 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62p.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62p.dtsi
> @@ -123,3 +123,7 @@ cbass_wakeup: bus@b00000 {
>  #include "k3-am62p-main.dtsi"
>  #include "k3-am62p-mcu.dtsi"
>  #include "k3-am62p-wakeup.dtsi"
> +
> +&dmsc {
> +	ti,partial-io-wakeup-sources = <&mcu_mcan0>, <&mcu_mcan1>, <&mcu_uart0>, <&wkup_uart0>;
> +};

NOTE: https://software-dl.ti.com/tisci/esd/latest/2_tisci_msgs/pm/lpm.html#supported-low-power-modes
table of SoCs supported do not provide J722s or AM62p in
the list. I suspect it is an oversight, but seems to be a mismatch atm.
Dhruva Gole May 27, 2024, 8:48 a.m. UTC | #2
Hi Nishanth,

On May 24, 2024 at 00:54:58 -0500, Nishanth Menon wrote:
> On 10:02-20240523, Markus Schneider-Pargmann wrote:
> > From: Vibhore Vardhan <vibhore@ti.com>
> > 
> > In Partial-IO mode there are a number of possible wakeup sources. Add
> > the list of phandles to these wakeup sources. Based on the patch for
> > AM62a.
> > 
> > Signed-off-by: Vibhore Vardhan <vibhore@ti.com>
> > Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com>
> > ---
> >  arch/arm64/boot/dts/ti/k3-am62p.dtsi | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/ti/k3-am62p.dtsi b/arch/arm64/boot/dts/ti/k3-am62p.dtsi
> > index 94babc412575..4d43cc972056 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am62p.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-am62p.dtsi
> > @@ -123,3 +123,7 @@ cbass_wakeup: bus@b00000 {
> >  #include "k3-am62p-main.dtsi"
> >  #include "k3-am62p-mcu.dtsi"
> >  #include "k3-am62p-wakeup.dtsi"
> > +
> > +&dmsc {
> > +	ti,partial-io-wakeup-sources = <&mcu_mcan0>, <&mcu_mcan1>, <&mcu_uart0>, <&wkup_uart0>;
> > +};
> 
> NOTE: https://software-dl.ti.com/tisci/esd/latest/2_tisci_msgs/pm/lpm.html#supported-low-power-modes
> table of SoCs supported do not provide J722s or AM62p in
> the list. I suspect it is an oversight, but seems to be a mismatch atm.

Yes, you are right. We will fix this in the documentation soon.
AM62P will be added to that list. J722s does not have lpm support.
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/ti/k3-am62p.dtsi b/arch/arm64/boot/dts/ti/k3-am62p.dtsi
index 94babc412575..4d43cc972056 100644
--- a/arch/arm64/boot/dts/ti/k3-am62p.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62p.dtsi
@@ -123,3 +123,7 @@  cbass_wakeup: bus@b00000 {
 #include "k3-am62p-main.dtsi"
 #include "k3-am62p-mcu.dtsi"
 #include "k3-am62p-wakeup.dtsi"
+
+&dmsc {
+	ti,partial-io-wakeup-sources = <&mcu_mcan0>, <&mcu_mcan1>, <&mcu_uart0>, <&wkup_uart0>;
+};