[v2] ARM: dts: imx6ulz-bsh-smm-m2: fix resume via console
diff mbox series

Message ID 20250715141322.1305512-1-dario.binacchi@amarulasolutions.com
State New
Headers show
Series
  • [v2] ARM: dts: imx6ulz-bsh-smm-m2: fix resume via console
Related show

Commit Message

Dario Binacchi July 15, 2025, 2:13 p.m. UTC 
From: Wolfgang Birkner <wolfgang.birkner@bshg.com>

Despite the current configuration being compliant with the technical
reference manual (TRM), testing on the system showed that resuming from
suspend via UART4 (used as the console) fails unless any other UART
is also enabled.

In our use case, UART2 is enabled to ensure reliable resume when UART4
is used as the console.

Signed-off-by: Wolfgang Birkner <wolfgang.birkner@bshg.com>
Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>

---

Changes in v2:
- Improve commit message
- Drop patches:
  - 1/3 ARM: dts: imx6ulz-bsh-smm-m2: Enable hardware rng
  - 3/3 ARM: dts: imx6ulz-bsh-smm-m2: Update wifi/bluetooth pinctrl

 arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Peng Fan July 21, 2025, 9:36 a.m. UTC | #1 
> Subject: [PATCH v2] ARM: dts: imx6ulz-bsh-smm-m2: fix resume via
> console
> 
> From: Wolfgang Birkner <wolfgang.birkner@bshg.com>
> 
> Despite the current configuration being compliant with the technical
> reference manual (TRM), testing on the system showed that resuming
> from suspend via UART4 (used as the console) fails unless any other
> UART is also enabled.
> 
> In our use case, UART2 is enabled to ensure reliable resume when
> UART4 is used as the console.
> 
> Signed-off-by: Wolfgang Birkner <wolfgang.birkner@bshg.com>
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> 

Not sure why there is dependency here, but enable it
does not harm, so

Ack-by: Peng Fan <peng.fan@nxp.com>

To unsubscribe from this group and stop receiving emails from it, send an email to linux-amarula+unsubscribe@amarulasolutions.com.
Shawn Guo Aug. 21, 2025, 7:27 a.m. UTC | #2 
On Tue, Jul 15, 2025 at 04:13:10PM +0200, Dario Binacchi wrote:
> From: Wolfgang Birkner <wolfgang.birkner@bshg.com>
> 
> Despite the current configuration being compliant with the technical
> reference manual (TRM), testing on the system showed that resuming from
> suspend via UART4 (used as the console) fails unless any other UART
> is also enabled.
> 
> In our use case, UART2 is enabled to ensure reliable resume when UART4
> is used as the console.
> 
> Signed-off-by: Wolfgang Birkner <wolfgang.birkner@bshg.com>
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>

Weird, but applied.

To unsubscribe from this group and stop receiving emails from it, send an email to linux-amarula+unsubscribe@amarulasolutions.com.

Patch
diff mbox series 

diff --git a/arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts b/arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts
index 6159ed70d966..2d9f495660c9 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts
@@ -33,6 +33,10 @@  &snvs_poweroff {
 	status = "okay";
 };
 
+&uart2 {
+	status = "okay";
+};
+
 &uart3 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_uart3>;