| Message ID | 20240523080225.1288617-7-msp@baylibre.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | firmware: ti_sci: Partial-IO support | expand |
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.
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 --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>; +};