1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
|
From 8279b4e4862b148b10e7d4d2967ac92eaae0f129 Mon Sep 17 00:00:00 2001
From: Holger Freyther <zecke@openmoko.org>
Date: Fri, 25 Jul 2008 23:06:18 +0100
Subject: [PATCH] From 3a32be40f78404d5f1185f0b3d6b5632381cb33f Mon Sep 17 00:00:00 2001
Subject: [PATCH] [neo1973 leds] Move from mutex to spinlock because we may not use mutexes
The led triggers may call set_brightness from atomic contexts. As
mutex_lock calls might_sleep and sleeping is not allowed in atomic contexts
we have to switch to spinlocks here.
Signed-Off-By: Holger Freyther <zecke@openmoko.org>
---
drivers/leds/leds-neo1973-gta02.c | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/leds/leds-neo1973-gta02.c b/drivers/leds/leds-neo1973-gta02.c
index 952ad69..8fe817b 100644
--- a/drivers/leds/leds-neo1973-gta02.c
+++ b/drivers/leds/leds-neo1973-gta02.c
@@ -27,7 +27,7 @@
struct gta02_led_priv
{
- struct mutex mutex;
+ spinlock_t lock;
struct led_classdev cdev;
struct s3c2410_pwm pwm;
unsigned int gpio;
@@ -53,6 +53,7 @@ static inline struct gta02_led_bundle *to_bundle(struct led_classdev *led_cdev)
static void gta02led_set(struct led_classdev *led_cdev,
enum led_brightness value)
{
+ unsigned long flags;
struct gta02_led_priv *lp = to_priv(led_cdev);
/*
@@ -60,7 +61,7 @@ static void gta02led_set(struct led_classdev *led_cdev,
* value == 128 -> 50% duty cycle (medium power)
* value == 0 -> 0% duty cycle (zero power)
*/
- mutex_lock(&lp->mutex);
+ spin_lock_irqsave(&lp->lock, flags);
if (lp->has_pwm) {
s3c2410_pwm_duty_cycle(value, &lp->pwm);
@@ -68,7 +69,7 @@ static void gta02led_set(struct led_classdev *led_cdev,
neo1973_gpb_setpin(lp->gpio, value ? 1 : 0);
}
- mutex_unlock(&lp->mutex);
+ spin_unlock_irqrestore(&lp->lock, flags);
}
#ifdef CONFIG_PM
@@ -170,7 +171,7 @@ static int __init gta02led_probe(struct platform_device *pdev)
break;
}
- mutex_init(&lp->mutex);
+ spin_lock_init(&lp->lock);
rc = led_classdev_register(&pdev->dev, &lp->cdev);
}
@@ -192,7 +193,6 @@ static int gta02led_remove(struct platform_device *pdev)
gta02led_set(&lp->cdev, 0);
led_classdev_unregister(&lp->cdev);
- mutex_destroy(&lp->mutex);
}
platform_set_drvdata(pdev, NULL);
--
1.5.6.3
|
