cns3xxx: Fix laguna arm11mpcore watchdog
authorFelix Fietkau <nbd@openwrt.org>
Wed, 10 Oct 2012 11:38:58 +0000 (11:38 +0000)
committerFelix Fietkau <nbd@openwrt.org>
Wed, 10 Oct 2012 11:38:58 +0000 (11:38 +0000)
The ARM11MPCore Timer/Watchdog registers start at offset 0x600 which is where
all mpcore-wdt boards point the driver base too.  I believe this is wrong
because 0x600 is aliased to the timer/watchdog of the 'current CPU' where
0x700 is CPU0's timer/watchdog, and 0x800 is CPU1's timer/watchdog.  Thus
if your timer/watchdog application is switching between CPU's it can end up
writing to the wrong CPU's registers which results in random board resets
from watchdog timeouts etc.

This patch forces the timer/watchdog driver to use CPU0's registers always.
Its my opinion that other mpcore-wdt boards should be doing the same thing.

Signed-off-by: Tim Harvey <tharvey@gateworks.com>
SVN-Revision: 33683

target/linux/cns3xxx/patches-3.3/300-laguna_support.patch

index 7fe970a..a4602f9 100644 (file)
 +
 +static struct resource laguna_watchdog_resources[] = {
 +      [0] = {
-+              .start  = CNS3XXX_TC11MP_TWD_BASE,
++              .start  = CNS3XXX_TC11MP_TWD_BASE + 0x100, // CPU0 watchdog
 +              .end    = CNS3XXX_TC11MP_TWD_BASE + SZ_4K - 1,
 +              .flags  = IORESOURCE_MEM,
 +      },