From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stijn Souffriau Date: Sat, 02 Jun 2012 17:32:14 +0200 Subject: [U-Boot] [PATCH] Added watchdog support for davinchi_dm365evm In-Reply-To: <201205201459.28436.vapier@gentoo.org> References: <1337532640-1310-1-git-send-email-stijn.souffriau@essensium.com> <201205201459.28436.vapier@gentoo.org> Message-ID: <4FCA31FE.70408@essensium.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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