From mboxrd@z Thu Jan  1 00:00:00 1970
From: David Updegraff <dave@cray.com>
Date: Thu, 09 Nov 2006 11:27:18 -0600
Subject: [U-Boot-Users] mpc83xx watchdog
In-Reply-To: <45536227.1080506@freescale.com>
References: <eivm9p$njd$2@sea.gmane.org> <45536227.1080506@freescale.com>
Message-ID: <455364F6.7050205@cray.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Timur

Good to hear you already have one; fine by me.

But.. I dont know how it can work w/o the tweek to start.S, since by my
reading of the PPC 603 asm.ref., the existing instruction there will not
accept full 16bit value.  Did you have another workaround for it?

> David Updegraff wrote:
>> Hi.
>>
>> Watchdog could work with this.
> 
> A patch that does something similar is already in Freescale's 83xx tree
> that is waiting for Wolfgang to pull.
> 
> Here's the relevant bit:
> 
> @@ -150,9 +213,21 @@ unsigned long get_tbclk(void)
>  #if defined(CONFIG_WATCHDOG)
>  void watchdog_reset (void)
>  {
> -    hang();        /* FIXME: implement watchdog_reset()? */
> +#ifdef CONFIG_MPC834X
> +    int re_enable = disable_interrupts();
> +
> +    /* Reset the 83xx watchdog */
> +    volatile immap_t *immr = (immap_t *) CFG_IMMRBAR;
> +    immr->wdt.swsrr = 0x556c;
> +    immr->wdt.swsrr = 0xaa39;
> +
> +    if (re_enable)
> +        enable_interrupts ();
> +#else
> +    hang();
> +#endif
>  }
> -#endif /* CONFIG_WATCHDOG */
> +#endif
> 
> 
> We don't have any changes for start.S, though.  Also, our tree renames
> CFG_IMMRBAR to CFG_IMMR.
> 
> I'm going to NACK your patch, simply because it's incompatible with
> ours.  I'm willing to work with you to fix that, though, if you like.
>