On Sunday 19 June 2011 12:03:28 Alan Cox wrote: > > > /* Register the watchdog timer device */ > > > res = watchdog_register_device(&wdt_dev); > > > > Why is this a feature of the individual drivers and not of the core? > > It's a hardware dependant feature - some watchdogs cannot be stopped once > initialized. Ah, I see. OTOH, 80 of the 107 watchdogs in the kernel do provide it as a module option, which indicates that there is some room for consolidation, even if it might not be appropriate for every one of them. > > Maybe it can be in both, so existing drivers don't need to change the user > > interface, but new drivers don't have to provide the option individually > > when you can simply set it for the base module. > > Then you'd need an additional interface to specify which watchdog as soon > as we support multiple watchdogs. You can always have multiple ways of setting nowayout -- hardware requirements, global module option, local module option, and a new ioctl command -- but what is being used is then the logical OR of all of them. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
