summaryrefslogtreecommitdiff
path: root/target/linux/s3c24xx/patches/0206-From-3a32be40f78404d5f1185f0b3d6b5632381cb33f-Mon-Se.patch
blob: dfb6078b54b0a269782ca9d4fbaa47688e787078 (plain)
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