ASoC: wm5100: Remove spurious IRQF_ONESHOT flag
authorAxel Lin <axel.lin@ingics.com>
Thu, 2 Jul 2015 13:27:48 +0000 (21:27 +0800)
committerMark Brown <broonie@kernel.org>
Wed, 8 Jul 2015 18:30:29 +0000 (19:30 +0100)
This reverts 3d907cc30d07 ("ASoC: wm5100: Pass the IRQF_ONESHOT flag").

The coccinelle warnings is false positive because the original code does
set IRQF_ONESHOT by "trigger |= IRQF_ONESHOT;".

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Acked-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/wm5100.c

index 4c10cd88c1af66a1a54b168c34844c190e077b9d..98495dd6123998de3511656e3aa7ac7c1134c968 100644 (file)
@@ -2570,13 +2570,11 @@ static int wm5100_i2c_probe(struct i2c_client *i2c,
 
                if (irq_flags & (IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING))
                        ret = request_threaded_irq(i2c->irq, NULL,
-                                                  wm5100_edge_irq,
-                                                  irq_flags | IRQF_ONESHOT,
+                                                  wm5100_edge_irq, irq_flags,
                                                   "wm5100", wm5100);
                else
                        ret = request_threaded_irq(i2c->irq, NULL, wm5100_irq,
-                                                  irq_flags | IRQF_ONESHOT,
-                                                  "wm5100",
+                                                  irq_flags, "wm5100",
                                                   wm5100);
 
                if (ret != 0) {