* [PATCH 2/4] ASoC: wm8753: Use power efficient workqueue
2013-07-18 21:57 [PATCH 1/4] ASoC: wm8350: Use power efficient workqueue Mark Brown
@ 2013-07-18 21:57 ` Mark Brown
2013-07-18 21:57 ` [PATCH 3/4] ASoC: wm8962: " Mark Brown
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Mark Brown @ 2013-07-18 21:57 UTC (permalink / raw)
To: Liam Girdwood; +Cc: alsa-devel, patches, linaro-kernel, Mark Brown
From: Mark Brown <broonie@linaro.org>
The work used to allow the capcitors to ramp is not performance sensitive
so let the scheduler run it wherever is most efficient rather than in a
per CPU workqueue by using the system power efficient workqueue.
Signed-off-by: Mark Brown <broonie@linaro.org>
---
sound/soc/codecs/wm8753.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/sound/soc/codecs/wm8753.c b/sound/soc/codecs/wm8753.c
index 0a4ab4c..d96ebf5 100644
--- a/sound/soc/codecs/wm8753.c
+++ b/sound/soc/codecs/wm8753.c
@@ -1456,8 +1456,9 @@ static int wm8753_resume(struct snd_soc_codec *codec)
if (codec->dapm.suspend_bias_level == SND_SOC_BIAS_ON) {
wm8753_set_bias_level(codec, SND_SOC_BIAS_PREPARE);
codec->dapm.bias_level = SND_SOC_BIAS_ON;
- schedule_delayed_work(&codec->dapm.delayed_work,
- msecs_to_jiffies(caps_charge));
+ queue_delayed_work(system_power_efficient_wq,
+ &codec->dapm.delayed_work,
+ msecs_to_jiffies(caps_charge));
}
return 0;
--
1.8.3.2
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH 3/4] ASoC: wm8962: Use power efficient workqueue
2013-07-18 21:57 [PATCH 1/4] ASoC: wm8350: Use power efficient workqueue Mark Brown
2013-07-18 21:57 ` [PATCH 2/4] ASoC: wm8753: " Mark Brown
@ 2013-07-18 21:57 ` Mark Brown
2013-07-18 21:57 ` [PATCH 4/4] ASoC: wm8994: " Mark Brown
2013-07-19 4:16 ` [PATCH 1/4] ASoC: wm8350: " Viresh Kumar
3 siblings, 0 replies; 5+ messages in thread
From: Mark Brown @ 2013-07-18 21:57 UTC (permalink / raw)
To: Liam Girdwood; +Cc: alsa-devel, patches, linaro-kernel, Mark Brown
From: Mark Brown <broonie@linaro.org>
The accessory detect debounce work is not performance sensitive so let
the scheduler run it wherever is most efficient rather than in a per CPU
workqueue by using the system power efficient workqueue.
Signed-off-by: Mark Brown <broonie@linaro.org>
---
sound/soc/codecs/wm8962.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/sound/soc/codecs/wm8962.c b/sound/soc/codecs/wm8962.c
index e2de9ec..f79df8b 100644
--- a/sound/soc/codecs/wm8962.c
+++ b/sound/soc/codecs/wm8962.c
@@ -3046,8 +3046,9 @@ static irqreturn_t wm8962_irq(int irq, void *data)
pm_wakeup_event(dev, 300);
- schedule_delayed_work(&wm8962->mic_work,
- msecs_to_jiffies(250));
+ queue_delayed_work(system_power_efficient_wq,
+ &wm8962->mic_work,
+ msecs_to_jiffies(250));
}
return IRQ_HANDLED;
--
1.8.3.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 4/4] ASoC: wm8994: Use power efficient workqueue
2013-07-18 21:57 [PATCH 1/4] ASoC: wm8350: Use power efficient workqueue Mark Brown
2013-07-18 21:57 ` [PATCH 2/4] ASoC: wm8753: " Mark Brown
2013-07-18 21:57 ` [PATCH 3/4] ASoC: wm8962: " Mark Brown
@ 2013-07-18 21:57 ` Mark Brown
2013-07-19 4:16 ` [PATCH 1/4] ASoC: wm8350: " Viresh Kumar
3 siblings, 0 replies; 5+ messages in thread
From: Mark Brown @ 2013-07-18 21:57 UTC (permalink / raw)
To: Liam Girdwood; +Cc: alsa-devel, patches, linaro-kernel, Mark Brown
From: Mark Brown <broonie@linaro.org>
The accessory detect debounce work is not performance sensitive so let
the scheduler run it wherever is most efficient rather than in a per CPU
workqueue by using the system power efficient workqueue.
Signed-off-by: Mark Brown <broonie@linaro.org>
---
sound/soc/codecs/wm8994.c | 23 ++++++++++++++---------
1 file changed, 14 insertions(+), 9 deletions(-)
diff --git a/sound/soc/codecs/wm8994.c b/sound/soc/codecs/wm8994.c
index 357f439..a8d7b0c 100644
--- a/sound/soc/codecs/wm8994.c
+++ b/sound/soc/codecs/wm8994.c
@@ -819,8 +819,9 @@ static int clk_sys_event(struct snd_soc_dapm_widget *w,
* don't want false reports.
*/
if (wm8994->jackdet && !wm8994->clk_has_run) {
- schedule_delayed_work(&wm8994->jackdet_bootstrap,
- msecs_to_jiffies(1000));
+ queue_delayed_work(system_power_efficient_wq,
+ &wm8994->jackdet_bootstrap,
+ msecs_to_jiffies(1000));
wm8994->clk_has_run = true;
}
break;
@@ -3487,7 +3488,8 @@ static irqreturn_t wm8994_mic_irq(int irq, void *data)
pm_wakeup_event(codec->dev, 300);
- schedule_delayed_work(&priv->mic_work, msecs_to_jiffies(250));
+ queue_delayed_work(system_power_efficient_wq,
+ &priv->mic_work, msecs_to_jiffies(250));
return IRQ_HANDLED;
}
@@ -3575,8 +3577,9 @@ static void wm8958_mic_id(void *data, u16 status)
/* If nothing present then clear our statuses */
dev_dbg(codec->dev, "Detected open circuit\n");
- schedule_delayed_work(&wm8994->open_circuit_work,
- msecs_to_jiffies(2500));
+ queue_delayed_work(system_power_efficient_wq,
+ &wm8994->open_circuit_work,
+ msecs_to_jiffies(2500));
return;
}
@@ -3690,8 +3693,9 @@ static irqreturn_t wm1811_jackdet_irq(int irq, void *data)
WM1811_JACKDET_DB, 0);
delay = control->pdata.micdet_delay;
- schedule_delayed_work(&wm8994->mic_work,
- msecs_to_jiffies(delay));
+ queue_delayed_work(system_power_efficient_wq,
+ &wm8994->mic_work,
+ msecs_to_jiffies(delay));
} else {
dev_dbg(codec->dev, "Jack not detected\n");
@@ -3936,8 +3940,9 @@ static irqreturn_t wm8958_mic_irq(int irq, void *data)
id_delay = wm8994->wm8994->pdata.mic_id_delay;
if (wm8994->mic_detecting)
- schedule_delayed_work(&wm8994->mic_complete_work,
- msecs_to_jiffies(id_delay));
+ queue_delayed_work(system_power_efficient_wq,
+ &wm8994->mic_complete_work,
+ msecs_to_jiffies(id_delay));
else
wm8958_button_det(codec, reg);
--
1.8.3.2
^ permalink raw reply related [flat|nested] 5+ messages in thread