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