As promised, here is a RFC with two examples demonstrating how watchdog drivers can be converted to use the new watchdog framework (using the current version Wim posted two days ago). There is also conversion guide put to the documentation folder. Being RFC, all this is not final yet, but presentable, I hope. Although there are a few more consolidation options left, there is already a gain of ~100 lines per driver. Promising, but there are a few issues to be sorted out, too, yet nothing which can be dealt with. I have two other drivers in the making (stmp3xxx and imx2), but they need some more preparation; the first one needs some internal cleanups (like a lot of watchdog drivers); the latter one needs an addition to the framework (installing a timer for non-stoppable devices). I will also prepare a new driver (mx1) to show how small new drivers can be now :) The aim for all these driver conversions is inclusion in Linux 3.2. I still hope we can get the basic framework into Linux 3.1. Many thanks to CELF/LF for supporting this work and to Wim and Alan for making the framework! Looking forward to comments, Wolfram They are also available in the git repository at: git://git.pengutronix.de/git/wsa/linux-2.6.git generic-watchdog-with-drivers Wolfram Sang (6): Documentation: watchdog: add guide how to convert drivers to new framework watchdog: s3c2410: convert to use the watchdog framework watchdog: pnx4008: cleanup resource handling using managed devices watchdog: pnx4008: don't use __raw_-accessors watchdog: pnx4008: convert driver to use the watchdog framework watchdog: pnx4008: WIP refactor disabling device .../watchdog/convert_drivers_to_kernel_api.txt | 195 ++++++++++++++++ drivers/watchdog/Kconfig | 2 + drivers/watchdog/pnx4008_wdt.c | 246 +++++++------------- drivers/watchdog/s3c2410_wdt.c | 176 +++----------- 4 files changed, 313 insertions(+), 306 deletions(-) create mode 100644 Documentation/watchdog/convert_drivers_to_kernel_api.txt -- 1.7.2.5 -- 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
