linux-watchdog.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 0/2] Make all it87 drivers SMP safe
@ 2011-04-11 22:05 Nat Gurumoorthy
  0 siblings, 0 replies; only message in thread
From: Nat Gurumoorthy @ 2011-04-11 22:05 UTC (permalink / raw)
  To: Jean Delvare, Guenter Roeck, Wim Van Sebroeck
  Cc: Mike Waychison, lm-sensors, linux-kernel, linux-watchdog,
	Nat Gurumoorthy

There are 3 different drivers that touch the it87 hardware registers.
The 3 drivers have been written independently and access the it87 hardware
registers assuming they are the only driver accessing it. This change
attempts to serialize access to the hardware by using
"request_muxed_region" macro defined by Alan Cox. Call to this macro
will hold off the requestor if the resource is currently busy.
The use of the above macro makes it possible to get rid of
spinlocks in it8712f_wdt.c and it87_wdt.c watchdog drivers.
This also greatly simplifies the implementation of it87_wdt.c driver.

01 - Changes to it87 watchdog driver to use "request_muxed_region"
 drivers/watchdog/it8712f_wdt.c
 drivers/watchdog/it87_wdt.c

02 - Chages to hwmon it87 driver to use "request_muxed_region"
 drivers/hwmon/it87.c

 drivers/hwmon/it87.c           |    4 ++++
 drivers/watchdog/it8712f_wdt.c |    9 ++++-----
 drivers/watchdog/it87_wdt.c    |   39 ++++-----------------------------------
 3 files changed, 12 insertions(+), 40 deletions(-)

Signed-off-by: Nat Gurumoorthy <natg@google.com>

Patch History:
v3:
- Totally abandon the spinlock based approach and use "request_muxed_region" to
  hold off requestors if the resource is busy.

v2:
- More verbose patch headers. Add In-Reply-To: field.


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-04-11 22:06 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-11 22:05 [PATCH v3 0/2] Make all it87 drivers SMP safe Nat Gurumoorthy

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).