ASoC: jack: Fix race in snd_soc_jack_add_gpios
authorlars <lars@3c298f89-4303-0410-b956-a3cf2f4a3e73>
Fri, 31 Jul 2009 23:58:29 +0000 (23:58 +0000)
committerlars <lars@3c298f89-4303-0410-b956-a3cf2f4a3e73>
Fri, 31 Jul 2009 23:58:29 +0000 (23:58 +0000)
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@17071 3c298f89-4303-0410-b956-a3cf2f4a3e73

target/linux/generic-2.6/patches-2.6.30/780-fix-race-in-snd_soc_jack_add_gpios.patch [new file with mode: 0644]

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 (file)
index 0000000..4185a8c
--- /dev/null
@@ -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
+