console: Use flush() before panic and reset
authorTony Dinh <mibodhi@gmail.com>
Wed, 15 Mar 2023 00:24:26 +0000 (17:24 -0700)
committerTom Rini <trini@konsulko.com>
Wed, 22 Mar 2023 16:51:08 +0000 (12:51 -0400)
To make sure the panic and the reset messages will go out, console flush() should be used.
Sleep periods do not work in early u-boot phase when timer driver is not initialized yet.

Reference: https://lists.denx.de/pipermail/u-boot/2023-March/512233.html

Signed-off-by: Tony Dinh <mibodhi@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Stefan Roese <sr@denx.de>
arch/arm/lib/reset.c
lib/panic.c

index 95169ba..3e051e3 100644 (file)
@@ -25,6 +25,7 @@
 #include <cpu_func.h>
 #include <irq_func.h>
 #include <linux/delay.h>
+#include <stdio.h>
 
 __weak void reset_misc(void)
 {
@@ -33,8 +34,7 @@ __weak void reset_misc(void)
 int do_reset(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
 {
        puts ("resetting ...\n");
-
-       mdelay(50);                             /* wait 50 ms */
+       flush();
 
        disable_interrupts();
 
index 58382ac..66ae17f 100644 (file)
@@ -15,6 +15,7 @@
 #include <command.h>
 #endif
 #include <linux/delay.h>
+#include <stdio.h>
 
 static void panic_finish(void) __attribute__ ((noreturn));
 
@@ -24,7 +25,8 @@ static void panic_finish(void)
 #if defined(CONFIG_PANIC_HANG)
        hang();
 #else
-       udelay(100000); /* allow messages to go out */
+       flush();  /* flush the panic message before reset */
+
        do_reset(NULL, 0, 0, NULL);
 #endif
        while (1)