[PATCH u-boot-mvebu 3/3] arm: mvebu: turris_omnia: disable MCU watchdog in SPL when booting over UART

Stefan Roese sr at denx.de
Mon Aug 16 15:07:33 CEST 2021


On 16.08.21 15:01, Marek Behún wrote:
> On Mon, 16 Aug 2021 14:50:15 +0200
> Stefan Roese <sr at denx.de> wrote:
> 
>>> -	disable_mcu_watchdog();
>>> +	/*
>>> +	 * If not booting from UART, MCU watchdog was not disabled
>>> in SPL,
>>> +	 * disable it now.
>>> +	 */
>>> +	if (get_boot_device() != BOOT_DEVICE_UART)
>>> +		disable_mcu_watchdog();
>>
>> Why do you disable the MCU watchdog here in U-Boot proper?
> 
> Hi Stefan,
> 
> this is just where it was before.
> But looking at the prompt
>     SoC:   MV88F6820-A0 at 1600 MHz
>     DRAM:  2 GiB (800 MHz, 32-bit, 2T, ECC not enabled)
>     Disabling MCU watchdog... disabled
>     WDT:   Started with servicing (60s timeout)
> 
> it looks like U-Boot's watchdog is being enabled after MCU watchdog.
> It would be better to enable U-Boot's WDT before disabling MCU watchdog.
> I will send another patch that moves it to another function,
> board_late_init for example.

So the MCU watchdog is some kind of very early WD and only used until
the SoC WD is used / enabled?

> Shall I resend this series with this new patch?

Yes, please do.

Thanks,
Stefan


More information about the U-Boot mailing list