summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNico Huber <nico.h@gmx.de>2017-09-09 19:46:44 +0200
committerNico Huber <nico.h@gmx.de>2017-09-10 17:49:27 +0000
commit0b80bd1cf4510b3a4f028b5bb5570a2806c22320 (patch)
treee7d8854664e59ee3193a6c4e30c37713a04de50b
parent3b4b069a0c8bf255c81a798c138b413f02415a5c (diff)
downloadcoreboot-0b80bd1cf4510b3a4f028b5bb5570a2806c22320.tar.xz
nb/intel/i945: Clear timeout bits after disabling watchdog
Even with the watchdog disabled, these bits influence other hardware blocks (e.g. SECOND_TO_STS stops SMBus block transfers, possibly yet before they started). Change-Id: If9f93fcc96827bb192148a80b4476796c9358a7a Signed-off-by: Nico Huber <nico.h@gmx.de> Reviewed-on: https://review.coreboot.org/21471 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin Roth <martinroth@google.com> Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
-rw-r--r--src/northbridge/intel/i945/early_init.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/northbridge/intel/i945/early_init.c b/src/northbridge/intel/i945/early_init.c
index 1d473d3104..0abd42eec8 100644
--- a/src/northbridge/intel/i945/early_init.c
+++ b/src/northbridge/intel/i945/early_init.c
@@ -170,6 +170,8 @@ static void i945_setup_bars(void)
printk(BIOS_DEBUG, "Disabling Watchdog reboot...");
RCBA32(GCS) = RCBA32(GCS) | (1 << 5); /* No reset */
outw((1 << 11), DEFAULT_PMBASE | 0x60 | 0x08); /* halt timer */
+ outw((1 << 3), DEFAULT_PMBASE | 0x60 | 0x04); /* clear timeout */
+ outw((1 << 1), DEFAULT_PMBASE | 0x60 | 0x06); /* clear 2nd timeout */
printk(BIOS_DEBUG, " done.\n");
/* Enable upper 128bytes of CMOS */