On Mon, 2011-09-12 at 05:57 -0400, Hans de Goede wrote: > Add support for the watchdog integrated into the (Fujitsu Theseus version of) > the sch5636 superio hwmon part. Using the new watchdog timer core. > > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > --- > drivers/hwmon/sch5636.c | 233 ++++++++++++++++++++++++++++++++++++++++++++++- > 1 files changed, 231 insertions(+), 2 deletions(-) > [ resent to proper thread ] Given that this is no longer a pure hwmon driver, it may make sense to split it into separate mfd/hwmon/watchdog drivers. But on the other side I notice that other drivers in the hwmon directory implement watchdog functionality as well. Bad precedent :(. Wonder what happens with those watchdogs if HWMON is disabled. Not really sure if we should let this happen, or start being more restrictive and enforce cleaner code. Jean, any thoughts/comments ? Guenter -- 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
