iTCO_wdt: fix typo when setting TCO_EN bit
authorLinus Torvalds <>
Thu, 4 Dec 2008 00:20:19 +0000 (16:20 -0800)
committerLinus Torvalds <>
Thu, 4 Dec 2008 00:20:19 +0000 (16:20 -0800)
The code used '&= 0x00002000' when it tried to set the TCO_EN bit, which
obviously didn't set that bit at all, but instead just reset all the
other bits in the SMI_EN register.

This bug seemingly caused various random behavior, with Frans Pop
reporting that just silently hung at startup and Rafael Wysocki
reports the fan spinning with full speed.


The problem seems to have been triggered by "[WATCHDOG] iTCO_wdt :
problem with rebooting on new ICH9 based motherboards" (commit
7cd5b08be3c489df11b559fef210b81133764ad4), but the bogus code existed
before that too (in the "supermicro_old_pre_stop()" function), it just
apparently never showed up due to different logic.

In that commit the broken code got moved around and now gets executed
much more.

Reported-by: Rafael J. Wysocki <>
Tested-by: Frans Pop <>
Cc: Wim Van Sebroeck <>
Signed-off-by: Linus Torvalds <>

index 26173a270e944c14503e7b4acc7f55b334eb861d..5b395a4ddfdfec67dbd4324acbc521258977a855 100644 (file)
@@ -392,7 +392,7 @@ static int iTCO_wdt_stop(void)
        /* Bit 13: TCO_EN -> 1 = Enables the TCO logic to generate SMI# */
        val32 = inl(SMI_EN);
-       val32 &= 0x00002000;
+       val32 |= 0x00002000;
        outl(val32, SMI_EN);
        /* Set the NO_REBOOT bit to prevent later reboots, just for sure */