[PATCH] ASoC: Mark cache as dirty when suspending

[PATCH] ASoC: Mark cache as dirty when suspending

[Mark Brown]

Since quite a few drivers are not managing to flag the cache as needing
to be resynced after suspend and it's a reasonable thing to do flag the
cache as needing sync automatically when suspending.

The expectation is that systems will mainly only keep the CODEC powered
when doing audio through the CODEC so we won't actually suspend the
device anyway; drivers which want to can override this behaviour when
they resume.

Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
 sound/soc/soc-core.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
index e44267f..93109a4 100644
--- a/sound/soc/soc-core.c
+++ b/sound/soc/soc-core.c
@@ -577,6 +577,7 @@ int snd_soc_suspend(struct device *dev)
 			case SND_SOC_BIAS_OFF:
 				codec->driver->suspend(codec, PMSG_SUSPEND);
 				codec->suspended = 1;
+				codec->cache_sync = 1;
 				break;
 			default:
 				dev_dbg(codec->dev, "CODEC is on over suspend\n");
-- 
1.7.5.4

Re: [PATCH] ASoC: Mark cache as dirty when suspending

[Liam Girdwood]

On 18/07/11 05:25, Mark Brown wrote:
> Since quite a few drivers are not managing to flag the cache as needing
> to be resynced after suspend and it's a reasonable thing to do flag the
> cache as needing sync automatically when suspending.
> 
> The expectation is that systems will mainly only keep the CODEC powered
> when doing audio through the CODEC so we won't actually suspend the
> device anyway; drivers which want to can override this behaviour when
> they resume.
> 
> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
> ---
>  sound/soc/soc-core.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
> index e44267f..93109a4 100644
> --- a/sound/soc/soc-core.c
> +++ b/sound/soc/soc-core.c
> @@ -577,6 +577,7 @@ int snd_soc_suspend(struct device *dev)
>  			case SND_SOC_BIAS_OFF:
>  				codec->driver->suspend(codec, PMSG_SUSPEND);
>  				codec->suspended = 1;
> +				codec->cache_sync = 1;
>  				break;
>  			default:
>  				dev_dbg(codec->dev, "CODEC is on over suspend\n");

Acked-by: Liam Girdwood <lrg@ti.com>