ASoC: codecs: rt56*: Remove IRQF_ONESHOT

Passing IRQF_ONESHOT ensures that the interrupt source is masked until
the secondary (threaded) handler is done. If only a primary handler is
used then the flag makes no sense because the interrupt can not fire
(again) while its handler is running.
The flag also disallows force-threading of the primary handler and the
irq-core will warn about this.

Remove IRQF_ONESHOT from irqflags.

Cc: Oder Chiou <oder_chiou@realtek.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Mark Brown <broonie@kernel.org>
Cc: Jaroslav Kysela <perex@perex.cz>
Cc: Takashi Iwai <tiwai@suse.com>
Cc: linux-sound@vger.kernel.org
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Link: https://patch.msgid.link/20260123113708.416727-13-bigeasy@linutronix.de
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/codecs/rt5640.c b/sound/soc/codecs/rt5640.c
index 4c08c274f50e2e114de89c59cd4857138981d89c..db2222e6f2e758151ccf41e523904fae39efde96 100644 (file)
--- a/sound/soc/codecs/rt5640.c
+++ b/sound/soc/codecs/rt5640.c
@@ -2564,7 +2564,7 @@ static void rt5640_enable_jack_detect(struct snd_soc_component *component,
                rt5640->use_platform_clock = jack_data->use_platform_clock;
 
        ret = request_irq(rt5640->irq, rt5640_irq,
-                         IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
+                         IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING,
                          "rt5640", rt5640);
        if (ret) {
                dev_warn(component->dev, "Failed to request IRQ %d: %d\n", rt5640->irq, ret);
@@ -2618,7 +2618,7 @@ static void rt5640_enable_hda_jack_detect(
        rt5640->jack = jack;
 
        ret = request_irq(rt5640->irq, rt5640_irq,
-                         IRQF_TRIGGER_RISING | IRQF_ONESHOT, "rt5640", rt5640);
+                         IRQF_TRIGGER_RISING, "rt5640", rt5640);
        if (ret) {
                dev_warn(component->dev, "Failed to request IRQ %d: %d\n", rt5640->irq, ret);
                rt5640->jack = NULL;

diff --git a/sound/soc/codecs/rt5651.c b/sound/soc/codecs/rt5651.c
index 9af65a38f0ee6161048c70a2c51b5a2b2dc95bc4..23c4bf3da2982f19fb5ee8d856355350427226ea 100644 (file)
--- a/sound/soc/codecs/rt5651.c
+++ b/sound/soc/codecs/rt5651.c
@@ -2262,7 +2262,7 @@ static int rt5651_i2c_probe(struct i2c_client *i2c)
 
        ret = devm_request_irq(&i2c->dev, rt5651->irq, rt5651_irq,
                               IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING
-                              | IRQF_ONESHOT | IRQF_NO_AUTOEN, "rt5651", rt5651);
+                              | IRQF_NO_AUTOEN, "rt5651", rt5651);
        if (ret) {
                dev_warn(&i2c->dev, "Failed to request IRQ %d: %d\n",
                         rt5651->irq, ret);

diff --git a/sound/soc/codecs/rt5663.c b/sound/soc/codecs/rt5663.c
index e4d8785e64c108b37bf0e49946262279aaa06722..eee1c98cc4aa6d2d52292e59dcbb602301f6dbae 100644 (file)
--- a/sound/soc/codecs/rt5663.c
+++ b/sound/soc/codecs/rt5663.c
@@ -3689,8 +3689,8 @@ static int rt5663_i2c_probe(struct i2c_client *i2c)
 
        if (i2c->irq) {
                ret = request_irq(i2c->irq, rt5663_irq,
-                       IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING
-                       | IRQF_ONESHOT, "rt5663", rt5663);
+                       IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING,
+                       "rt5663", rt5663);
                if (ret) {
                        dev_err(&i2c->dev, "%s Failed to request IRQ: %d\n",
                                __func__, ret);