* [PATCH 1/3] arm64: dts: ti: k3-am62l: include WKUP_UART0 in wakeup peripheral window
2025-12-31 2:51 [PATCH 0/3] arm64: boot: dts: ti: k3-am62l: enable WKUP UART wakeup Kendall Willis
@ 2025-12-31 2:51 ` Kendall Willis
2026-01-05 13:22 ` Dhruva Gole
2025-12-31 2:51 ` [PATCH 2/3] arm64: boot: dts: ti: k3-am62l-wakeup: create label for target-module Kendall Willis
2025-12-31 2:51 ` [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM Kendall Willis
2 siblings, 1 reply; 13+ messages in thread
From: Kendall Willis @ 2025-12-31 2:51 UTC (permalink / raw)
To: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Dhruva Gole, Bryan Brattlof
Cc: vishalm, sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel, Kendall Willis
WKUP_UART0 is apart of the wakeup peripherals and has a range from
0x002B300000 to 0x002B3001FF. Expand the wakeup peripheral window to
include WKUP_UART0.
Fixes: 5f016758b0ab ("arm64: dts: ti: k3-am62l: add initial infrastructure")
Signed-off-by: Kendall Willis <k-willis@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62l.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62l.dtsi b/arch/arm64/boot/dts/ti/k3-am62l.dtsi
index 23acdbb301fe38e6045c0ef3ae340196009ed217..e01e342c26daaa06a72036cc3a9a7b13a60e6738 100644
--- a/arch/arm64/boot/dts/ti/k3-am62l.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62l.dtsi
@@ -92,7 +92,7 @@ cbass_main: bus@f0000 {
<0x00 0x00b00000 0x00 0x00b00000 0x00 0x00001400>, /* VTM */
<0x00 0x04080000 0x00 0x04080000 0x00 0x00008000>, /* PDCFG */
<0x00 0x04201000 0x00 0x04201000 0x00 0x00000100>, /* GPIO */
- <0x00 0x2b100000 0x00 0x2b100000 0x00 0x00100100>, /* Wakeup Peripheral Window */
+ <0x00 0x2b100000 0x00 0x2b100000 0x00 0x00200200>, /* Wakeup Peripheral Window */
<0x00 0x40800000 0x00 0x40800000 0x00 0x00014000>, /* DMA */
<0x00 0x43000000 0x00 0x43000000 0x00 0x00080000>; /* CTRL MMRs */
#address-cells = <2>;
@@ -104,7 +104,7 @@ cbass_wakeup: bus@a80000 {
<0x00 0x00b00000 0x00 0x00b00000 0x00 0x00001400>, /* VTM */
<0x00 0x04080000 0x00 0x04080000 0x00 0x00008000>, /* PDCFG */
<0x00 0x04201000 0x00 0x04201000 0x00 0x00000100>, /* GPIO */
- <0x00 0x2b100000 0x00 0x2b100000 0x00 0x00100100>, /* Wakeup Peripheral Window */
+ <0x00 0x2b100000 0x00 0x2b100000 0x00 0x00200200>, /* Wakeup Peripheral Window */
<0x00 0x40800000 0x00 0x40800000 0x00 0x00014000>, /* DMA */
<0x00 0x43000000 0x00 0x43000000 0x00 0x00080000>; /* CTRL MMRs */
#address-cells = <2>;
--
2.34.1
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 1/3] arm64: dts: ti: k3-am62l: include WKUP_UART0 in wakeup peripheral window
2025-12-31 2:51 ` [PATCH 1/3] arm64: dts: ti: k3-am62l: include WKUP_UART0 in wakeup peripheral window Kendall Willis
@ 2026-01-05 13:22 ` Dhruva Gole
0 siblings, 0 replies; 13+ messages in thread
From: Dhruva Gole @ 2026-01-05 13:22 UTC (permalink / raw)
To: Kendall Willis
Cc: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bryan Brattlof, vishalm,
sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel
On Dec 30, 2025 at 20:51:40 -0600, Kendall Willis wrote:
> WKUP_UART0 is apart of the wakeup peripherals and has a range from
Nit: a _space_ part
> 0x002B300000 to 0x002B3001FF. Expand the wakeup peripheral window to
> include WKUP_UART0.
>
> Fixes: 5f016758b0ab ("arm64: dts: ti: k3-am62l: add initial infrastructure")
> Signed-off-by: Kendall Willis <k-willis@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am62l.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62l.dtsi b/arch/arm64/boot/dts/ti/k3-am62l.dtsi
> index 23acdbb301fe38e6045c0ef3ae340196009ed217..e01e342c26daaa06a72036cc3a9a7b13a60e6738 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62l.dtsi
> @@ -92,7 +92,7 @@ cbass_main: bus@f0000 {
> <0x00 0x00b00000 0x00 0x00b00000 0x00 0x00001400>, /* VTM */
> <0x00 0x04080000 0x00 0x04080000 0x00 0x00008000>, /* PDCFG */
> <0x00 0x04201000 0x00 0x04201000 0x00 0x00000100>, /* GPIO */
> - <0x00 0x2b100000 0x00 0x2b100000 0x00 0x00100100>, /* Wakeup Peripheral Window */
> + <0x00 0x2b100000 0x00 0x2b100000 0x00 0x00200200>, /* Wakeup Peripheral Window */
> <0x00 0x40800000 0x00 0x40800000 0x00 0x00014000>, /* DMA */
> <0x00 0x43000000 0x00 0x43000000 0x00 0x00080000>; /* CTRL MMRs */
> #address-cells = <2>;
> @@ -104,7 +104,7 @@ cbass_wakeup: bus@a80000 {
Reviewed-by: Dhruva Gole <d-gole@ti.com>
--
Best regards,
Dhruva Gole
Texas Instruments Incorporated
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 2/3] arm64: boot: dts: ti: k3-am62l-wakeup: create label for target-module
2025-12-31 2:51 [PATCH 0/3] arm64: boot: dts: ti: k3-am62l: enable WKUP UART wakeup Kendall Willis
2025-12-31 2:51 ` [PATCH 1/3] arm64: dts: ti: k3-am62l: include WKUP_UART0 in wakeup peripheral window Kendall Willis
@ 2025-12-31 2:51 ` Kendall Willis
2026-01-05 13:24 ` Dhruva Gole
2025-12-31 2:51 ` [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM Kendall Willis
2 siblings, 1 reply; 13+ messages in thread
From: Kendall Willis @ 2025-12-31 2:51 UTC (permalink / raw)
To: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Dhruva Gole, Bryan Brattlof
Cc: vishalm, sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel, Kendall Willis
Add label to the target-module node so that it can easily be referenced.
The node specifically configures the SYSCONFIG related registers for WKUP
UART.
Signed-off-by: Kendall Willis <k-willis@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi
index 61bfcdcfc66ea8d802a36ed43cd01fbbf3decc70..e9d638d9ffd3a52aa6e0df70f6003879bc292358 100644
--- a/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi
@@ -81,7 +81,7 @@ wkup_i2c0: i2c@2b200000 {
status = "disabled";
};
- target-module@2b300050 {
+ wkup_uart0_interconnect: target-module@2b300050 {
compatible = "ti,sysc-omap2", "ti,sysc";
reg = <0x00 0x2b300050 0x00 0x4>,
<0x00 0x2b300054 0x00 0x4>,
--
2.34.1
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 2/3] arm64: boot: dts: ti: k3-am62l-wakeup: create label for target-module
2025-12-31 2:51 ` [PATCH 2/3] arm64: boot: dts: ti: k3-am62l-wakeup: create label for target-module Kendall Willis
@ 2026-01-05 13:24 ` Dhruva Gole
0 siblings, 0 replies; 13+ messages in thread
From: Dhruva Gole @ 2026-01-05 13:24 UTC (permalink / raw)
To: Kendall Willis
Cc: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bryan Brattlof, vishalm,
sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel
On Dec 30, 2025 at 20:51:41 -0600, Kendall Willis wrote:
> Add label to the target-module node so that it can easily be referenced.
> The node specifically configures the SYSCONFIG related registers for WKUP
> UART.
>
> Signed-off-by: Kendall Willis <k-willis@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi
> index 61bfcdcfc66ea8d802a36ed43cd01fbbf3decc70..e9d638d9ffd3a52aa6e0df70f6003879bc292358 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi
> @@ -81,7 +81,7 @@ wkup_i2c0: i2c@2b200000 {
> status = "disabled";
> };
>
> - target-module@2b300050 {
> + wkup_uart0_interconnect: target-module@2b300050 {
Reviewed-by: Dhruva Gole <d-gole@ti.com>
--
Best regards,
Dhruva Gole
Texas Instruments Incorporated
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM
2025-12-31 2:51 [PATCH 0/3] arm64: boot: dts: ti: k3-am62l: enable WKUP UART wakeup Kendall Willis
2025-12-31 2:51 ` [PATCH 1/3] arm64: dts: ti: k3-am62l: include WKUP_UART0 in wakeup peripheral window Kendall Willis
2025-12-31 2:51 ` [PATCH 2/3] arm64: boot: dts: ti: k3-am62l-wakeup: create label for target-module Kendall Willis
@ 2025-12-31 2:51 ` Kendall Willis
2026-01-05 13:25 ` Dhruva Gole
` (2 more replies)
2 siblings, 3 replies; 13+ messages in thread
From: Kendall Willis @ 2025-12-31 2:51 UTC (permalink / raw)
To: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Dhruva Gole, Bryan Brattlof
Cc: vishalm, sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel, Kendall Willis
Change the status of the wkup_uart0_interconnect node to enabled. The
target-module node sets the UART SYSC and SYSS registers to allow wakeup
from WKUP UART in DeepSleep low power mode.
Signed-off-by: Kendall Willis <k-willis@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
index cae04cce337366b50928d39b0c888550b14e43d3..02b1a1c76b2ee05d49cd69d199a6aacdb1973fa0 100644
--- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
@@ -359,3 +359,7 @@ &usb1 {
pinctrl-names = "default";
pinctrl-0 = <&usb1_default_pins>;
};
+
+&wkup_uart0_interconnect {
+ status = "okay";
+};
--
2.34.1
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM
2025-12-31 2:51 ` [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM Kendall Willis
@ 2026-01-05 13:25 ` Dhruva Gole
2026-01-05 13:55 ` Nishanth Menon
2026-01-05 14:01 ` Bryan Brattlof
2 siblings, 0 replies; 13+ messages in thread
From: Dhruva Gole @ 2026-01-05 13:25 UTC (permalink / raw)
To: Kendall Willis
Cc: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bryan Brattlof, vishalm,
sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel
On Dec 30, 2025 at 20:51:42 -0600, Kendall Willis wrote:
> Change the status of the wkup_uart0_interconnect node to enabled. The
> target-module node sets the UART SYSC and SYSS registers to allow wakeup
> from WKUP UART in DeepSleep low power mode.
>
> Signed-off-by: Kendall Willis <k-willis@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> index cae04cce337366b50928d39b0c888550b14e43d3..02b1a1c76b2ee05d49cd69d199a6aacdb1973fa0 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> @@ -359,3 +359,7 @@ &usb1 {
> pinctrl-names = "default";
> pinctrl-0 = <&usb1_default_pins>;
> };
> +
> +&wkup_uart0_interconnect {
> + status = "okay";
> +};
Reviewed-by: Dhruva Gole <d-gole@ti.com>
>
> --
> 2.34.1
>
--
Best regards,
Dhruva Gole
Texas Instruments Incorporated
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM
2025-12-31 2:51 ` [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM Kendall Willis
2026-01-05 13:25 ` Dhruva Gole
@ 2026-01-05 13:55 ` Nishanth Menon
2026-01-05 22:10 ` Kendall Willis
2026-01-05 14:01 ` Bryan Brattlof
2 siblings, 1 reply; 13+ messages in thread
From: Nishanth Menon @ 2026-01-05 13:55 UTC (permalink / raw)
To: Kendall Willis
Cc: Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Dhruva Gole, Bryan Brattlof,
vishalm, sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel
On 20:51-20251230, Kendall Willis wrote:
> Change the status of the wkup_uart0_interconnect node to enabled. The
> target-module node sets the UART SYSC and SYSS registers to allow wakeup
> from WKUP UART in DeepSleep low power mode.
>
> Signed-off-by: Kendall Willis <k-willis@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> index cae04cce337366b50928d39b0c888550b14e43d3..02b1a1c76b2ee05d49cd69d199a6aacdb1973fa0 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> @@ -359,3 +359,7 @@ &usb1 {
> pinctrl-names = "default";
> pinctrl-0 = <&usb1_default_pins>;
> };
> +
> +&wkup_uart0_interconnect {
> + status = "okay";
> +};
don't we need to set status okay and provide pinmux for wkup_uart?
>
> --
> 2.34.1
>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM
2026-01-05 13:55 ` Nishanth Menon
@ 2026-01-05 22:10 ` Kendall Willis
2026-01-06 11:56 ` Nishanth Menon
0 siblings, 1 reply; 13+ messages in thread
From: Kendall Willis @ 2026-01-05 22:10 UTC (permalink / raw)
To: Nishanth Menon
Cc: Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Dhruva Gole, Bryan Brattlof,
vishalm, sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel
On 1/5/26 07:55, Nishanth Menon wrote:
> On 20:51-20251230, Kendall Willis wrote:
>> Change the status of the wkup_uart0_interconnect node to enabled. The
>> target-module node sets the UART SYSC and SYSS registers to allow wakeup
>> from WKUP UART in DeepSleep low power mode.
>>
>> Signed-off-by: Kendall Willis <k-willis@ti.com>
>> ---
>> arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
>> index cae04cce337366b50928d39b0c888550b14e43d3..02b1a1c76b2ee05d49cd69d199a6aacdb1973fa0 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
>> @@ -359,3 +359,7 @@ &usb1 {
>> pinctrl-names = "default";
>> pinctrl-0 = <&usb1_default_pins>;
>> };
>> +
>> +&wkup_uart0_interconnect {
>> + status = "okay";
>> +};
>
> don't we need to set status okay and provide pinmux for wkup_uart?
>
No, only the interconnect target node needs to be enabled because it
manages the SYSC register. The SYSC register configures the WKUP_UART to
be able to wakeup the SoC from system suspend. This allows the WKUP_UART
to be wakeup capable even if it is reserved by firmware.
Best,
Kendall
>>
>> --
>> 2.34.1
>>
>
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM
2026-01-05 22:10 ` Kendall Willis
@ 2026-01-06 11:56 ` Nishanth Menon
2026-01-06 17:10 ` Kendall Willis
0 siblings, 1 reply; 13+ messages in thread
From: Nishanth Menon @ 2026-01-06 11:56 UTC (permalink / raw)
To: Kendall Willis
Cc: Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Dhruva Gole, Bryan Brattlof,
vishalm, sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel
On 16:10-20260105, Kendall Willis wrote:
> On 1/5/26 07:55, Nishanth Menon wrote:
> > On 20:51-20251230, Kendall Willis wrote:
> > > Change the status of the wkup_uart0_interconnect node to enabled. The
> > > target-module node sets the UART SYSC and SYSS registers to allow wakeup
> > > from WKUP UART in DeepSleep low power mode.
> > >
> > > Signed-off-by: Kendall Willis <k-willis@ti.com>
> > > ---
> > > arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++++
> > > 1 file changed, 4 insertions(+)
> > >
> > > diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> > > index cae04cce337366b50928d39b0c888550b14e43d3..02b1a1c76b2ee05d49cd69d199a6aacdb1973fa0 100644
> > > --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> > > +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> > > @@ -359,3 +359,7 @@ &usb1 {
> > > pinctrl-names = "default";
> > > pinctrl-0 = <&usb1_default_pins>;
> > > };
> > > +
> > > +&wkup_uart0_interconnect {
> > > + status = "okay";
> > > +};
> >
> > don't we need to set status okay and provide pinmux for wkup_uart?
> >
>
> No, only the interconnect target node needs to be enabled because it manages
> the SYSC register. The SYSC register configures the WKUP_UART to be able to
> wakeup the SoC from system suspend. This allows the WKUP_UART to be wakeup
> capable even if it is reserved by firmware.
>
If wkup_uart is controlled by firmware, it should be managed by
firmware. $subject claims that wkup_uart is being enabled, but it is
not.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM
2026-01-06 11:56 ` Nishanth Menon
@ 2026-01-06 17:10 ` Kendall Willis
0 siblings, 0 replies; 13+ messages in thread
From: Kendall Willis @ 2026-01-06 17:10 UTC (permalink / raw)
To: Nishanth Menon
Cc: Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Dhruva Gole, Bryan Brattlof,
vishalm, sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel
On 1/6/26 05:56, Nishanth Menon wrote:
> On 16:10-20260105, Kendall Willis wrote:
>> On 1/5/26 07:55, Nishanth Menon wrote:
>>> On 20:51-20251230, Kendall Willis wrote:
>>>> Change the status of the wkup_uart0_interconnect node to enabled. The
>>>> target-module node sets the UART SYSC and SYSS registers to allow wakeup
>>>> from WKUP UART in DeepSleep low power mode.
>>>>
>>>> Signed-off-by: Kendall Willis <k-willis@ti.com>
>>>> ---
>>>> arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++++
>>>> 1 file changed, 4 insertions(+)
>>>>
>>>> diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
>>>> index cae04cce337366b50928d39b0c888550b14e43d3..02b1a1c76b2ee05d49cd69d199a6aacdb1973fa0 100644
>>>> --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
>>>> +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
>>>> @@ -359,3 +359,7 @@ &usb1 {
>>>> pinctrl-names = "default";
>>>> pinctrl-0 = <&usb1_default_pins>;
>>>> };
>>>> +
>>>> +&wkup_uart0_interconnect {
>>>> + status = "okay";
>>>> +};
>>>
>>> don't we need to set status okay and provide pinmux for wkup_uart?
>>>
>>
>> No, only the interconnect target node needs to be enabled because it manages
>> the SYSC register. The SYSC register configures the WKUP_UART to be able to
>> wakeup the SoC from system suspend. This allows the WKUP_UART to be wakeup
>> capable even if it is reserved by firmware.
>>
>
> If wkup_uart is controlled by firmware, it should be managed by
> firmware. $subject claims that wkup_uart is being enabled, but it is
> not.
>
>
Okay, I will change the $subject to reflect the patch better in v2.
Best,
Kendall
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM
2025-12-31 2:51 ` [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM Kendall Willis
2026-01-05 13:25 ` Dhruva Gole
2026-01-05 13:55 ` Nishanth Menon
@ 2026-01-05 14:01 ` Bryan Brattlof
2026-01-05 22:14 ` Kendall Willis
2 siblings, 1 reply; 13+ messages in thread
From: Bryan Brattlof @ 2026-01-05 14:01 UTC (permalink / raw)
To: Kendall Willis
Cc: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Dhruva Gole, vishalm,
sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel
On December 30, 2025 thus sayeth Kendall Willis:
> Change the status of the wkup_uart0_interconnect node to enabled. The
> target-module node sets the UART SYSC and SYSS registers to allow wakeup
> from WKUP UART in DeepSleep low power mode.
>
> Signed-off-by: Kendall Willis <k-willis@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> index cae04cce337366b50928d39b0c888550b14e43d3..02b1a1c76b2ee05d49cd69d199a6aacdb1973fa0 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> @@ -359,3 +359,7 @@ &usb1 {
> pinctrl-names = "default";
> pinctrl-0 = <&usb1_default_pins>;
> };
> +
> +&wkup_uart0_interconnect {
> + status = "okay";
> +};
Does the UART need to be enabled as well?
~Bryan
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 3/3] arm64: boot: dts: ti: k3-am62l3-evm: enable WKUP UART wakeup from LPM
2026-01-05 14:01 ` Bryan Brattlof
@ 2026-01-05 22:14 ` Kendall Willis
0 siblings, 0 replies; 13+ messages in thread
From: Kendall Willis @ 2026-01-05 22:14 UTC (permalink / raw)
To: Bryan Brattlof
Cc: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Dhruva Gole, vishalm,
sebin.francis, msp, khilman, a-kaur, s-kochidanadu,
linux-arm-kernel, devicetree, linux-kernel
On 1/5/26 08:01, Bryan Brattlof wrote:
> On December 30, 2025 thus sayeth Kendall Willis:
>> Change the status of the wkup_uart0_interconnect node to enabled. The
>> target-module node sets the UART SYSC and SYSS registers to allow wakeup
>> from WKUP UART in DeepSleep low power mode.
>>
>> Signed-off-by: Kendall Willis <k-willis@ti.com>
>> ---
>> arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
>> index cae04cce337366b50928d39b0c888550b14e43d3..02b1a1c76b2ee05d49cd69d199a6aacdb1973fa0 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
>> @@ -359,3 +359,7 @@ &usb1 {
>> pinctrl-names = "default";
>> pinctrl-0 = <&usb1_default_pins>;
>> };
>> +
>> +&wkup_uart0_interconnect {
>> + status = "okay";
>> +};
>
> Does the UART need to be enabled as well?
>
> ~Bryan
The UART does not need to be enabled because the interconnect target
module manages the SYSC register which configures the WKUP_UART to be
able to wakeup the SoC from system suspend. This allows the WKUP_UART to
be wakeup capable for the SoC even if it is disabled by Linux.
Best,
Kendall
^ permalink raw reply [flat|nested] 13+ messages in thread