On 05/20/2012 08:59 PM, Mike Frysinger wrote:
On Sunday 20 May 2012 12:50:40 Stijn Souffriau wrote:
--- a/board/davinci/dm365evm/dm365evm.c +++ b/board/davinci/dm365evm/dm365evm.c
+#ifdef CONFIG_HW_WATCHDOG +static struct davinci_timer * const wdttimer =
(struct davinci_timer *)CONFIG_SYS_WDTTIMERBASE;+/* WDTCR bit definitions */ +#define WDEN (1<< 14) +#define WDFLAG (1<< 15) +#define WDKEY_SEQ0 (0xa5c6<< 16) +#define WDKEY_SEQ1 (0xda7e<< 16)
if this is an on-chip hardware block, this stuff should be in an SoC-specific header
Yes but as far as I know It only works for the davinci dm356evm.
--- a/include/configs/davinci_dm365evm.h +++ b/include/configs/davinci_dm365evm.h
+// Hardware watchdog config
don't use // comment blocks. use /* ... */.
ok
-mike