diff options
author | Fengguang Wu <fengguang.wu@intel.com> | 2012-08-30 11:16:52 -0400 |
---|---|---|
committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-09-05 18:20:02 -0400 |
commit | 58d468328646effa72ada4deaa33e80d678980d6 (patch) | |
tree | d8a934cdd44c664fe06c55292f998407ca5af5ad /sound/soc/codecs/wm0010.c | |
parent | 75d8f2931a803b803cb4a850448460475c20f30b (diff) |
ASoC: wm0010: Add missing IRQF_ONESHOT
FYI, there are new coccinelle warnings show up in
tree: git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-3.7
head: e3523e01869da20fdd12ffd19ae1df7bf492650e
commit: e3523e01869da20fdd12ffd19ae1df7bf492650e [95/95] ASoC: wm0010: Add initial wm0010 DSP driver
All coccinelle warnings:
+ sound/soc/codecs/wm0010.c:850:7-27: ERROR: Threaded IRQ with no primary handler requested without IRQF_ONESHOT
--
+ sound/soc/codecs/wm0010.c:660:1-7: preceding lock on line 359
vim +850 sound/soc/codecs/wm0010.c
847 trigger = IRQF_TRIGGER_FALLING;
848 trigger |= IRQF_ONESHOT;
849
> 850 ret = request_threaded_irq(irq, NULL, wm0010_irq, trigger,
851 "wm0010", wm0010);
852 if (ret)
853 dev_err(wm0010->dev, "Failed to request IRQ %d: %d\n",
Please consider folding the attached diff :-)
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound/soc/codecs/wm0010.c')
-rw-r--r-- | sound/soc/codecs/wm0010.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/codecs/wm0010.c b/sound/soc/codecs/wm0010.c index 5f99148447e1..a4c35119792e 100644 --- a/sound/soc/codecs/wm0010.c +++ b/sound/soc/codecs/wm0010.c | |||
@@ -881,7 +881,7 @@ static int __devinit wm0010_spi_probe(struct spi_device *spi) | |||
881 | trigger = IRQF_TRIGGER_FALLING; | 881 | trigger = IRQF_TRIGGER_FALLING; |
882 | trigger |= IRQF_ONESHOT; | 882 | trigger |= IRQF_ONESHOT; |
883 | 883 | ||
884 | ret = request_threaded_irq(irq, NULL, wm0010_irq, trigger, | 884 | ret = request_threaded_irq(irq, NULL, wm0010_irq, trigger | IRQF_ONESHOT, |
885 | "wm0010", wm0010); | 885 | "wm0010", wm0010); |
886 | if (ret) { | 886 | if (ret) { |
887 | dev_err(wm0010->dev, "Failed to request IRQ %d: %d\n", | 887 | dev_err(wm0010->dev, "Failed to request IRQ %d: %d\n", |