diff options
| author | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2009-10-19 06:07:05 -0400 |
|---|---|---|
| committer | Samuel Ortiz <sameo@linux.intel.com> | 2009-10-26 19:20:32 -0400 |
| commit | 14f572fa38c55fbe369127b808a4a79b0bbec367 (patch) | |
| tree | 3fea2662caff0d56fa3e6234e5ad1a8d16ba67e3 | |
| parent | 964fe080d94db82a3268443e9b9ece4c60246414 (diff) | |
mfd: Always initialise WM831x IRQ mutex
This avoids crashes when running without interrupt support.
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
| -rw-r--r-- | drivers/mfd/wm831x-irq.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/mfd/wm831x-irq.c b/drivers/mfd/wm831x-irq.c index d3015dfb9134..ac056ea6b66e 100644 --- a/drivers/mfd/wm831x-irq.c +++ b/drivers/mfd/wm831x-irq.c | |||
| @@ -507,6 +507,8 @@ int wm831x_irq_init(struct wm831x *wm831x, int irq) | |||
| 507 | { | 507 | { |
| 508 | int i, ret; | 508 | int i, ret; |
| 509 | 509 | ||
| 510 | mutex_init(&wm831x->irq_lock); | ||
| 511 | |||
| 510 | if (!irq) { | 512 | if (!irq) { |
| 511 | dev_warn(wm831x->dev, | 513 | dev_warn(wm831x->dev, |
| 512 | "No interrupt specified - functionality limited\n"); | 514 | "No interrupt specified - functionality limited\n"); |
| @@ -521,7 +523,6 @@ int wm831x_irq_init(struct wm831x *wm831x, int irq) | |||
| 521 | } | 523 | } |
| 522 | 524 | ||
| 523 | wm831x->irq = irq; | 525 | wm831x->irq = irq; |
| 524 | mutex_init(&wm831x->irq_lock); | ||
| 525 | INIT_WORK(&wm831x->irq_work, wm831x_irq_worker); | 526 | INIT_WORK(&wm831x->irq_work, wm831x_irq_worker); |
| 526 | 527 | ||
| 527 | /* Mask the individual interrupt sources */ | 528 | /* Mask the individual interrupt sources */ |
