]> git.karo-electronics.de Git - mv-sheeva.git/commit
OMAP2+: hwmod: upgrade per-hwmod mutex to a spinlock
authorPaul Walmsley <paul@pwsan.com>
Tue, 14 Dec 2010 19:42:35 +0000 (12:42 -0700)
committerPaul Walmsley <paul@pwsan.com>
Wed, 22 Dec 2010 02:55:12 +0000 (19:55 -0700)
commitdc6d1cda044b24c3d9f8e4af0431887ebe3488ef
tree475596354e1a9e3d861ee02e50e0b82575377cb3
parentbd36179eec2827cd60b4a8c6e180cc030c74a4ec
OMAP2+: hwmod: upgrade per-hwmod mutex to a spinlock

Change the per-hwmod mutex to a spinlock.  (The per-hwmod lock
serializes most post-initialization hwmod operations such as enable,
idle, and shutdown.)  Spinlocks are needed, because in some cases,
hwmods must be enabled from timer interrupt disabled-context, such as
an ISR.  The current use-case that is driving this is the OMAP GPIO
block ISR: it can trigger interrupts even with its clocks disabled,
but these clocks are needed for register accesses in the ISR to succeed.

This patch also effectively reverts commit
848240223c35fcc71c424ad51a8e8aef42d3879c - this patch makes
_omap_hwmod_enable() and _omap_hwmod_init() static, renames them back
to _enable() and _idle(), and changes their callers to call the
spinlocking versions.  Previously, since omap_hwmod_{enable,init}()
attempted to take mutexes, these functions could not be called while
the timer interrupt was disabled; but now that the functions use
spinlocks and save and restore the IRQ state, it is appropriate to
call them directly.

Kevin Hilman <khilman@deeprootsystems.com> originally proposed this
patch - thanks Kevin.

Signed-off-by: Paul Walmsley <paul@pwsan.com>
Cc: Kevin Hilman <khilman@deeprootsystems.com>
Cc: BenoƮt Cousson <b-cousson@ti.com>
arch/arm/mach-omap2/omap_hwmod.c
arch/arm/mach-omap2/serial.c
arch/arm/plat-omap/include/plat/omap_hwmod.h