Dear Alexander Stein, In message <1279713664-20353-1-git-send-email-alexander.st...@systec-electronic.com> you wrote: > This allows Linux to initialize and use the watchdog with the included > driver. > CONFIG_AT91SAM9_WATCHDOG and CONFIG_HW_WATCHDOG should be defined to make > u-boot trigger the watchdog > > Signed-off-by: Alexander Stein <alexander.st...@systec-electronic.com> > --- > arch/arm/cpu/arm926ejs/at91/lowlevel_init.S | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/cpu/arm926ejs/at91/lowlevel_init.S > b/arch/arm/cpu/arm926ejs/at91/lowlevel_init.S > index 559c35c..21ebae9 100644 > --- a/arch/arm/cpu/arm926ejs/at91/lowlevel_init.S > +++ b/arch/arm/cpu/arm926ejs/at91/lowlevel_init.S > @@ -186,8 +186,10 @@ SDRAM_setup_end: > .ltorg > > SMRDATA: > +#if defined(CONFIG_SYS_WDTC_WDMR_VAL) > .word AT91_ASM_WDT_MR > .word CONFIG_SYS_WDTC_WDMR_VAL > +#endif
This seems the wrong approach to me - when CONFIG_HW_WATCHDOG is defined and requires CONFIG_SYS_WDTC_WDMR_VAL, but the latter is missing, than this should raise n error condition. We must not silently ignore errors. NAK. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de "The POP3 server service depends on the SMTP server service, which failed to start because of the following error: The operation comple- ted successfully." -- Windows NT Server v3.51 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
