[U-Boot] [PATCH] at91: Don't initialize watchdog if CONFIG_SYS_WDTC_WDMR_VAL is undefined

[Alexander Stein]

Dear Wolfgang,

Am Donnerstag, 22. Juli 2010, 15:53:25 schrieben Sie:
> > Of course, if CONFIG_HW_WATCHDOG and CONFIG_AT91SAM9_WATCHDOG is set,
> > u-boot should use/trigger the watchdog. But (re-)programming the
> > watchdog on AT91 is not always necessary. Without programming (using
> > CONFIG_SYS_WDTC_WDMR_VAL), the watchdog just runs with default settings,
> > allowing e.g. Linux to reprogram it later.
> > This patch allows to program the watchdog from u-boot, if wanted, but
> > also allows to reprogram it later from Linux. While triggering the
> > watchdog from u- boot itself does still work, wrt. the default watchdog
> > settings.
> 
> I can see the intention, but this should not depend on "val" being
> defined or not - and it needs explicit documentation (in the README,
> for example).

So, you would be ok with an option e.g. CONFIG_SKIP_WATCHDOG_PROGRAM with a 
documentation in README.at91?

Best regards,
Alexander