diff options
author | Fabio Estevam <fabio.estevam@freescale.com> | 2015-05-12 00:22:57 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-05-12 14:46:25 -0400 |
commit | 3d907cc30d072829b6682fda791005de5768f34e (patch) | |
tree | 03080492d9e60a0bc6d893b74be1a3e1e88ae47d | |
parent | b787f68c36d49bb1d9236f403813641efa74a031 (diff) |
ASoC: wm5100: Pass the IRQF_ONESHOT flag
Since commit 1c6c69525b40eb76de8adf039409722015927dc3 ("genirq: Reject
bogus threaded irq requests") threaded IRQs without a primary handler
need to be requested with IRQF_ONESHOT, otherwise the request will fail.
So pass the IRQF_ONESHOT flag in this case.
The semantic patch that makes this change is available
in scripts/coccinelle/misc/irqf_oneshot.cocci.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | sound/soc/codecs/wm5100.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/sound/soc/codecs/wm5100.c b/sound/soc/codecs/wm5100.c index 96740379b711..5de28bfd1079 100644 --- a/sound/soc/codecs/wm5100.c +++ b/sound/soc/codecs/wm5100.c | |||
@@ -2570,11 +2570,13 @@ static int wm5100_i2c_probe(struct i2c_client *i2c, | |||
2570 | 2570 | ||
2571 | if (irq_flags & (IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING)) | 2571 | if (irq_flags & (IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING)) |
2572 | ret = request_threaded_irq(i2c->irq, NULL, | 2572 | ret = request_threaded_irq(i2c->irq, NULL, |
2573 | wm5100_edge_irq, irq_flags, | 2573 | wm5100_edge_irq, |
2574 | irq_flags | IRQF_ONESHOT, | ||
2574 | "wm5100", wm5100); | 2575 | "wm5100", wm5100); |
2575 | else | 2576 | else |
2576 | ret = request_threaded_irq(i2c->irq, NULL, wm5100_irq, | 2577 | ret = request_threaded_irq(i2c->irq, NULL, wm5100_irq, |
2577 | irq_flags, "wm5100", | 2578 | irq_flags | IRQF_ONESHOT, |
2579 | "wm5100", | ||
2578 | wm5100); | 2580 | wm5100); |
2579 | 2581 | ||
2580 | if (ret != 0) { | 2582 | if (ret != 0) { |