diff options
| author | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2009-11-23 08:11:53 -0500 |
|---|---|---|
| committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2009-11-23 08:11:53 -0500 |
| commit | 50b6bce59d154b5db137907a5c0ed45a4e7a3829 (patch) | |
| tree | 8aec79bc5a5c72bac2eb480638803a9ca62e50ce | |
| parent | bab0212467e58929470ae3ae32515f17e30c3926 (diff) | |
ASoC: Fix suspend with active audio streams
When we get a stream suspend event force the power down since otherwise
the stream would remain marked as active. In future we'll probably want
to make this stream-specific and add an interface to make the power down
of other widgets optional in order to support leaving bypass paths
active while suspending the processor.
Cc: stable@kernel.org
Reported-by: Joonyoung Shim <jy0922.shim@samsung.com>
Tested-by: Joonyoung Shim <jy0922.shim@samsung.com>
Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
| -rw-r--r-- | sound/soc/soc-dapm.c | 20 |
1 files changed, 17 insertions, 3 deletions
diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c index d89f6dc00908..66d4c165f99b 100644 --- a/sound/soc/soc-dapm.c +++ b/sound/soc/soc-dapm.c | |||
| @@ -973,9 +973,19 @@ static int dapm_power_widgets(struct snd_soc_codec *codec, int event) | |||
| 973 | if (!w->power_check) | 973 | if (!w->power_check) |
| 974 | continue; | 974 | continue; |
| 975 | 975 | ||
| 976 | power = w->power_check(w); | 976 | /* If we're suspending then pull down all the |
| 977 | if (power) | 977 | * power. */ |
| 978 | sys_power = 1; | 978 | switch (event) { |
| 979 | case SND_SOC_DAPM_STREAM_SUSPEND: | ||
| 980 | power = 0; | ||
| 981 | break; | ||
| 982 | |||
| 983 | default: | ||
| 984 | power = w->power_check(w); | ||
| 985 | if (power) | ||
| 986 | sys_power = 1; | ||
| 987 | break; | ||
| 988 | } | ||
| 979 | 989 | ||
| 980 | if (w->power == power) | 990 | if (w->power == power) |
| 981 | continue; | 991 | continue; |
| @@ -999,8 +1009,12 @@ static int dapm_power_widgets(struct snd_soc_codec *codec, int event) | |||
| 999 | case SND_SOC_DAPM_STREAM_RESUME: | 1009 | case SND_SOC_DAPM_STREAM_RESUME: |
| 1000 | sys_power = 1; | 1010 | sys_power = 1; |
| 1001 | break; | 1011 | break; |
| 1012 | case SND_SOC_DAPM_STREAM_SUSPEND: | ||
| 1013 | sys_power = 0; | ||
| 1014 | break; | ||
| 1002 | case SND_SOC_DAPM_STREAM_NOP: | 1015 | case SND_SOC_DAPM_STREAM_NOP: |
| 1003 | sys_power = codec->bias_level != SND_SOC_BIAS_STANDBY; | 1016 | sys_power = codec->bias_level != SND_SOC_BIAS_STANDBY; |
| 1017 | break; | ||
| 1004 | default: | 1018 | default: |
| 1005 | break; | 1019 | break; |
| 1006 | } | 1020 | } |
