diff options
| author | Lars-Peter Clausen <lars@metafoo.de> | 2009-07-31 23:58:29 +0000 |
|---|---|---|
| committer | Lars-Peter Clausen <lars@metafoo.de> | 2009-07-31 23:58:29 +0000 |
| commit | 21a4ee51e910b0a0f4bd01e86b52b412c6ed80b3 (patch) | |
| tree | 14c70baca6bb578f41a3127aeeae2d6df0654fea | |
| parent | cbd8e85e0b4e4be3ab130eccd6c5df3f8ba396a2 (diff) | |
| download | upstream-21a4ee51e910b0a0f4bd01e86b52b412c6ed80b3.tar.gz upstream-21a4ee51e910b0a0f4bd01e86b52b412c6ed80b3.tar.bz2 upstream-21a4ee51e910b0a0f4bd01e86b52b412c6ed80b3.zip | |
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.patch | 43 |
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 + |