aboutsummaryrefslogtreecommitdiffstats
path: root/target/linux/generic-2.6
diff options
context:
space:
mode:
authorLars-Peter Clausen <lars@metafoo.de>2009-07-31 23:58:29 +0000
committerLars-Peter Clausen <lars@metafoo.de>2009-07-31 23:58:29 +0000
commit1da701c04e2c07f7377658adaf13e9c87ee91eb5 (patch)
tree4b8586b28859c46c1f32913f08ab11a01d994450 /target/linux/generic-2.6
parent930bb629e38db6ec060656ad5cc6e5ea193c344c (diff)
downloadupstream-1da701c04e2c07f7377658adaf13e9c87ee91eb5.tar.gz
upstream-1da701c04e2c07f7377658adaf13e9c87ee91eb5.tar.bz2
upstream-1da701c04e2c07f7377658adaf13e9c87ee91eb5.zip
ASoC: jack: Fix race in snd_soc_jack_add_gpios
SVN-Revision: 17071
Diffstat (limited to 'target/linux/generic-2.6')
-rw-r--r--target/linux/generic-2.6/patches-2.6.30/780-fix-race-in-snd_soc_jack_add_gpios.patch43
1 files changed, 43 insertions, 0 deletions
diff --git a/target/linux/generic-2.6/patches-2.6.30/780-fix-race-in-snd_soc_jack_add_gpios.patch b/target/linux/generic-2.6/patches-2.6.30/780-fix-race-in-snd_soc_jack_add_gpios.patch
new file mode 100644
index 0000000000..4185a8cb75
--- /dev/null
+++ b/target/linux/generic-2.6/patches-2.6.30/780-fix-race-in-snd_soc_jack_add_gpios.patch
@@ -0,0 +1,43 @@
+From d8d0421cddc6ecdec166612d914b8edbcdb96ed0 Mon Sep 17 00:00:00 2001
+From: Lars-Peter Clausen <lars@metafoo.de>
+Date: Fri, 31 Jul 2009 20:33:31 +0200
+Subject: [PATCH] ASoC: jack: Fix race in snd_soc_jack_add_gpios
+
+The irq can fire as soon as it has been requested, thus all fields accessed
+from within the irq handler must be initialized prior to requesting the irq.
+
+Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
+---
+The patch is against alsa-kernel/master but I guess something similar should also
+go into 2.6.31.
+
+ sound/soc/soc-jack.c | 6 +++---
+ 1 files changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/sound/soc/soc-jack.c b/sound/soc/soc-jack.c
+index 4aa7d8f..1d455ab 100644
+--- a/sound/soc/soc-jack.c
++++ b/sound/soc/soc-jack.c
+@@ -221,6 +221,9 @@ int snd_soc_jack_add_gpios(struct snd_soc_jack *jack, int count,
+ if (ret)
+ goto err;
+
++ INIT_WORK(&gpios[i].work, gpio_work);
++ gpios[i].jack = jack;
++
+ ret = request_irq(gpio_to_irq(gpios[i].gpio),
+ gpio_handler,
+ IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING,
+@@ -234,9 +237,6 @@ int snd_soc_jack_add_gpios(struct snd_soc_jack *jack, int count,
+ gpio_export(gpios[i].gpio, false);
+ #endif
+
+- INIT_WORK(&gpios[i].work, gpio_work);
+- gpios[i].jack = jack;
+-
+ /* Update initial jack status */
+ snd_soc_jack_gpio_detect(&gpios[i]);
+ }
+--
+1.5.6.5
+