>-----Original Message-----
>From: U-Boot <u-boot-boun...@lists.denx.de> On Behalf Of Chris Packham
>Sent: Wednesday, March 3, 2021 2:30 AM
>To: u-boot@lists.denx.de
>Cc: York Sun <york....@nxp.com>; Rainer Boschung
><rainer.bosch...@keymile.com>; Chris Packham <judge.pack...@gmail.com>;
>Masahiro Yamada <masahi...@kernel.org>; Ovidiu Panait
><ovidiu.pan...@windriver.com>; Patrick Delaunay
><patrick.delau...@foss.st.com>; Simon Glass <s...@chromium.org>; Stefan
>Roese <s...@denx.de>; Stephen Warren <swar...@nvidia.com>
>Subject: [PATCH] common: board_f: Restore 85xx watchdog support
>
>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.pack...@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

Kindly fix below build error for non-fsl ppc boards
powerpc:  w+   mpc8308_p1m sbc8349 sbc8349_PCI_33 sbc8349_PCI_66 ve8313 caddy2 
vme8349 hrcon hrcon_dh strider_con strider_con_dp strider_cpu strider_cpu_dp 
ids8313 TQM834x sbc8548 sbc8548_PCI_33 sbc8548_PCI_33_PCIE sbc8548_PCI_66 
sbc8548_PCI_66_PCIE socrates UCP1020 kmcent2 kmcoge4 Cyrus_P5020 Cyrus_P5040 
xpedite520x xpedite537x xpedite550x sbc8641d xpedite517x MCR3000 +   
controlcenterd_36BIT_SDCARD controlcenterd_36BIT_SDCARD_DEVELOP 
controlcenterd_TRAILBLAZER controlcenterd_TRAILBLAZER_DEVELOP


+powerpc-linux-ld.bfd: common/built-in.o: in function `init_func_watchdog_init':
2021-03-23T11:09:10.9787128Z +common/board_f.c:115: undefined reference to 
`init_85xx_watchdog'
2021-03-23T11:09:10.9787600Z +make[1]: *** [u-boot] Error 1
2021-03-23T11:09:10.9787981Z +make: *** [sub-make] Error 2


Regards
Priyanka

Reply via email to