Messages in this thread | | | Date | Sun, 19 Jun 2011 11:03:28 +0100 | From | Alan Cox <> | Subject | Re: [PATCH 7/10 v2] Generic Watchdog Timer Driver |
| |
> > /* 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.
> 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.
Alan
| |