diff options
author | Naveen Gupta <ngupta@google.com> | 2005-08-17 03:10:10 -0400 |
---|---|---|
committer | Wim Van Sebroeck <wim@iguana.be> | 2005-09-11 15:48:46 -0400 |
commit | 28562af3d4b21d687dd57c44006aeeed1036c781 (patch) | |
tree | 46dd305650708846a09fbe41f3b9999803a8a7c4 /drivers/char | |
parent | c69af038cad5e6ea86e927a17b70b49af1f7cbfa (diff) |
[WATCHDOG] i6300esb.c-WDT_ENABLE-bug
This patch sets the WDT_ENABLE bit of the Lock Register to enable the
watchdog and WDT_LOCK bit only if nowayout is set. The old code always
sets the WDT_LOCK bit of watchdog timer for Intel 6300ESB chipset. So, we
end up locking the watchdog instead of enabling it.
Signed-off-by: Naveen Gupta <ngupta@google.com>
Signed-off-by: David Hardeman <david@2gen.com>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Diffstat (limited to 'drivers/char')
-rw-r--r-- | drivers/char/watchdog/i6300esb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/char/watchdog/i6300esb.c b/drivers/char/watchdog/i6300esb.c index 20ceb5fe28b6..f0e96fbd6489 100644 --- a/drivers/char/watchdog/i6300esb.c +++ b/drivers/char/watchdog/i6300esb.c | |||
@@ -97,7 +97,7 @@ static void esb_timer_start(void) | |||
97 | u8 val; | 97 | u8 val; |
98 | 98 | ||
99 | /* Enable or Enable + Lock? */ | 99 | /* Enable or Enable + Lock? */ |
100 | val = 0x02 | nowayout ? 0x01 : 0x00; | 100 | val = 0x02 | (nowayout ? 0x01 : 0x00); |
101 | 101 | ||
102 | pci_write_config_byte(esb_pci, ESB_LOCK_REG, val); | 102 | pci_write_config_byte(esb_pci, ESB_LOCK_REG, val); |
103 | } | 103 | } |