RE: [CFT][PATCH] watchdog nowayout and timeout module parameters

Matt_Domsch@Dell.com
Tue, 18 Dec 2001 14:08:08 -0600


> Timeout has been moved to an ioctl more by other diffs so Im not sure
> timeout= is too important

Of the seven drivers I added this parm to, six do not offer such a method
via ioctl:
eurotechwdt.c provides WDIOC_SETTIMEOUT.
acquirewdt.c, advantechwdt.c, ib700wdt.c, pcwd.c, wdt.c, wdt_pci.c do not
provide a set timeout ioctl.

These already offer a timeout parm, but no set timeout ioctl:
i810-tco.c, softdog.c, wdt285.o

The other drivers I didn't add timeout parm to anyhow, but they don't have a
set timeout ioctl either:
machzwd.c, mixcomwd.c, shwdt.c, wdt977.c

Unless there are strong objections or a push to make it an ioctl everywhere
instead/also, I'd like to leave it in.

> Rest looks good

Thanks for reviewing and approving. 14 drivers modified, 5 drivers
approved, one investigating, two email addresses invalid, leaving 6 to hear
from.

-Matt

--
Matt Domsch
Sr. Software Engineer
Dell Linux Solutions www.dell.com/linux
#1 US Linux Server provider with 24.5% (IDC Dec 2001)
#2 Worldwide Linux Server provider with 18.2% (IDC Dec 2001)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/