[PATCH] ASoC: compress: Use power efficient workqueue

[PATCH] ASoC: compress: Use power efficient workqueue

[Mark Brown]

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 */
-- 
1.8.4.rc1

Re: [PATCH] ASoC: compress: Use power efficient workqueue

[Vinod Koul]

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));
>  		}
>  	} else {
>  		/* capture streams can be powered down now */
> -- 
> 1.8.4.rc1
> 

--