On Sat, Mar 30, 2013 at 06:59:08PM -0700, Tony Chung wrote: > Observed that the w83627hf watchdog timer start counting during reboot. > If the system load the driver after 5 minutes, it rebooted immediately > because of timer expired. > For example, fsck took more than 5 minutes to run, then reboot will occurred. > > Signed-off-by: Tony Chung <tonychun...@gmail.com> > --- > drivers/watchdog/w83627hf_wdt.c | 5 +++++ > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/drivers/watchdog/w83627hf_wdt.c b/drivers/watchdog/w83627hf_wdt.c > index 8f1111d..47eb233 100644 > --- a/drivers/watchdog/w83627hf_wdt.c > +++ b/drivers/watchdog/w83627hf_wdt.c > @@ -174,6 +174,11 @@ static void w83627hf_init(void) > > outb_p(0xF7, WDT_EFER); /* Select CRF7 */ > t = inb_p(WDT_EFDR); /* read CRF7 */ > + if (t & 0x10) { > + pr_info("Watchdog Timer timeout occurred!"); > + t &= ~0x10; /* clear the event */ > + pr_info("Event cleared\n"); > + } > t &= ~0xC0; /* disable keyboard & mouse turning off > watchdog */
I would suggest to simply clear it silently. I don't think an extra message provides any real value. t &= ~0xE0; /* clear watchdog, disable keyboard & mouse turning off watchdog */ Guenter > outb_p(t, WDT_EFDR); /* Write back to CRF7 */ > -- > 1.7.0.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/