summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorlars <lars@3c298f89-4303-0410-b956-a3cf2f4a3e73>2009-07-31 23:58:29 +0000
committerlars <lars@3c298f89-4303-0410-b956-a3cf2f4a3e73>2009-07-31 23:58:29 +0000
commit4df39edd96856af9f1dd35ec2219f1b31a2e0bc3 (patch)
tree14c70baca6bb578f41a3127aeeae2d6df0654fea
parentcd38e695286156d85a17efd173e1808b042436e0 (diff)
ASoC: jack: Fix race in snd_soc_jack_add_gpios
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@17071 3c298f89-4303-0410-b956-a3cf2f4a3e73
-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 000000000..4185a8cb7
--- /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
+