[PATCH] common: board_f: Restore 85xx watchdog support
Chris Packham
judge.packham at gmail.com
Tue Mar 2 22:00:03 CET 2021
In commit 75918afa649b ("powerpc: Drop old non-generic-board code") we
lost the call to init_85xx_watchdog() which had the effect of disabling
support for the watchdog on 85xx and similar SoCs (i.e. the QorIQ P
Series and T Series).
Signed-off-by: Chris Packham <judge.packham at gmail.com>
---
Admittedly this is a bit ugly but it's the most literal reinstatement of
the code that was lost. At the very least I should probably rename
init_85xx_watchdog() to hw_watchdog_init() so it fits with the rest of
the code.
The other question is how has this gone unnoticed for ~5 years. I think
the answer is because only the keymile boards were using it.
Finally I am wondering if this should be converted to a DM driver. But
given the fact that the watchdog is part of the core and not a
peripheral on the SoC I don't know how that would look (e.g. what would
I put in the DTS?).
So I thought I'd run this up the flag pole as-is and see what feedback I
get.
common/board_f.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/common/board_f.c b/common/board_f.c
index 0cddf0359dca..3778571a7196 100644
--- a/common/board_f.c
+++ b/common/board_f.c
@@ -111,6 +111,11 @@ static int init_func_watchdog_init(void)
hw_watchdog_init();
puts(" Watchdog enabled\n");
# endif
+# ifdef CONFIG_MPC85xx
+ init_85xx_watchdog();
+ puts(" Watchdog enabled\n");
+# endif
+
WATCHDOG_RESET();
return 0;
--
2.30.1
More information about the U-Boot
mailing list