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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
