From: w.sang@pengutronix.de (Wolfram Sang)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC 0/6] watchdog drivers converted to the new framework
Date: Wed, 13 Jul 2011 22:26:00 +0200 [thread overview]
Message-ID: <1310588766-16638-1-git-send-email-w.sang@pengutronix.de> (raw)
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
next reply other threads:[~2011-07-13 20:26 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-13 20:26 Wolfram Sang [this message]
2011-07-14 17:23 ` [RFC 0/6] watchdog drivers converted to the new framework H Hartley Sweeten
2011-07-14 18:27 ` Wolfram Sang
2011-07-14 18:42 ` H Hartley Sweeten
2011-07-14 20:00 ` Wolfram Sang
2011-07-14 20:10 ` H Hartley Sweeten
2011-07-22 17:55 ` Wim Van Sebroeck
[not found] ` <1310588766-16638-2-git-send-email-w.sang@pengutronix.de>
2011-07-16 2:09 ` [RFC 1/6] Documentation: watchdog: add guide how to convert drivers to " Randy Dunlap
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1310588766-16638-1-git-send-email-w.sang@pengutronix.de \
--to=w.sang@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).