[U-Boot] [PATCH v2 0/1] Fix U-Boot Prompt on CM-FX6 with enabled watchdog

Andreas J. Reichel Andreas.Reichel at tngtech.com
Mon Aug 1 15:32:13 CEST 2016


This patch fixes unwanted watchdog resets while the user enters
a command at the U-Boot prompt.

As found on the CM-FX6 board from Compulab, when having enabled the
watchdog, a missing WATCHDOG_RESET call in _serial_tstc() in
(/drivers/serial/serial-uclass.c) causes this and alike boards to
reset when the watchdog's timeout has elapsed while waiting at the
U-Boot prompt.

Despite the user could press several keys within the watchdog
timeout limit, the while loop in cli_readline.c, line 261, does only
call WATCHDOG_RESET if first == 1, which gets set to 0 in the 1st
loop iteration. This leads to a watchdog timeout no matter if the
user presses keys or not.

The problem is solved with a call to WATCHDOG_RESET in _serial_tstc,
defined in drivers/serial/serial-uclass.c.

Since the macro WATCHDOG_RESET expands to {} if watchdog support
isn't configured, there's no need to surround it by #ifdef in this
case.

Changes in v2:
 - Move WATCHDOG_RESET() call from common/console.c to
   drivers/serial/serial-uclass.c.

Andreas J. Reichel (1):
  watchdog: Fix Watchdog Reset while in U-Boot Prompt

 drivers/serial/serial-uclass.c | 1 +
 1 file changed, 1 insertion(+)

-- 
2.8.2



More information about the U-Boot mailing list