From: Mark Brown broonie@linaro.org
There is no need for the power down work to be done on a per CPU workqueue especially considering the fairly long delay before powerdown.
Signed-off-by: Mark Brown broonie@linaro.org --- sound/soc/soc-compress.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/sound/soc/soc-compress.c b/sound/soc/soc-compress.c index d220150..53c9ecd 100644 --- a/sound/soc/soc-compress.c +++ b/sound/soc/soc-compress.c @@ -149,8 +149,9 @@ static int soc_compr_free(struct snd_compr_stream *cstream) SND_SOC_DAPM_STREAM_STOP); } else { rtd->pop_wait = 1; - schedule_delayed_work(&rtd->delayed_work, - msecs_to_jiffies(rtd->pmdown_time)); + queue_delayed_work(system_power_efficient_wq, + &rtd->delayed_work, + msecs_to_jiffies(rtd->pmdown_time)); } } else { /* capture streams can be powered down now */
On Fri, Aug 09, 2013 at 06:22:11PM +0100, Mark Brown wrote:
From: Mark Brown broonie@linaro.org
There is no need for the power down work to be done on a per CPU workqueue especially considering the fairly long delay before powerdown.
Signed-off-by: Mark Brown broonie@linaro.org
Acked-by: Vinod Koul vinod.koul@intel.com
Thanks ~Vinod
sound/soc/soc-compress.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/sound/soc/soc-compress.c b/sound/soc/soc-compress.c index d220150..53c9ecd 100644 --- a/sound/soc/soc-compress.c +++ b/sound/soc/soc-compress.c @@ -149,8 +149,9 @@ static int soc_compr_free(struct snd_compr_stream *cstream) SND_SOC_DAPM_STREAM_STOP); } else { rtd->pop_wait = 1;
schedule_delayed_work(&rtd->delayed_work,msecs_to_jiffies(rtd->pmdown_time));
queue_delayed_work(system_power_efficient_wq,&rtd->delayed_work,msecs_to_jiffies(rtd->pmdown_time));-- 1.8.4.rc1
linaro-kernel@lists.linaro.org
-
Mark Brown -
Vinod Koul