On Fri, Feb 03, 2023 at 01:26:33PM +0100, Jan Kiszka wrote: > From: Jan Kiszka <[email protected]> > > Allows run-time control over watchdog auto-start and the timeout via > setting the environment variable watchdog_timeout_ms. A value of zero > means "do not start". Use CONFIG_WATCHDOG_TIMEOUT_MSECS as initial value > and this to zero by default. Users can then enable the watchdog once the > use and OS which picks it up during boot. > > Signed-off-by: Jan Kiszka <[email protected]> [snip] > diff --git a/include/configs/iot2050.h b/include/configs/iot2050.h > index 7d087413362..5186dfd8ff8 100644 > --- a/include/configs/iot2050.h > +++ b/include/configs/iot2050.h > @@ -15,6 +15,14 @@ > > /* SPL Loader Configuration */ > > +#define WATCHDOG_ENV \ > + "watchdog_timeout_ms=" __stringify(CONFIG_WATCHDOG_TIMEOUT_MSECS) "\0" \ > + "start_watchdog=if test ${watchdog_timeout_ms} -gt 0; then " \ > + "wdt dev watchdog@40610000; " \ > + "wdt start ${watchdog_timeout_ms}; " \ > + "echo Watchdog started, timeout ${watchdog_timeout_ms} ms; " \ > + "fi\0" > + > /* U-Boot general configuration */ > #define EXTRA_ENV_IOT2050_BOARD_SETTINGS \ > "usb_pgood_delay=900\0" > @@ -43,6 +51,7 @@ > #define CFG_EXTRA_ENV_SETTINGS \ > DEFAULT_LINUX_BOOT_ENV \ > BOOTENV \ > + WATCHDOG_ENV \ > EXTRA_ENV_IOT2050_BOARD_SETTINGS
As a follow-up I would like to see all of this migrated to the plain text environment, see board/*/*/*.env for various more and less complex examples of this kind of conversion. -- Tom
signature.asc
Description: PGP signature
