diff options
author | Lars-Peter Clausen <lars@metafoo.de> | 2014-11-23 07:37:38 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2014-11-24 13:37:15 -0500 |
commit | a5dde8c42ef6b3cb47c69905ee51520e18ac6515 (patch) | |
tree | bc5ec81967a222b3aba8643ccdf857d1f1d13ee5 /sound | |
parent | 5d01410fe4d92081f349b013a2e7a95429e4f2c9 (diff) |
ASoC: wm8985: Cleanup manual bias level transitions
Set the CODEC driver's suspend_bias_off flag rather than manually going to
SND_SOC_BIAS_OFF in suspend and SND_SOC_BIAS_STANDBY in resume. This makes
the code a bit shorter and cleaner.
Since the ASoC core now takes care of setting the bias level to
SND_SOC_BIAS_OFF when removing the CODEC there is no need to do it manually
anymore either.
The manual transition to SND_SOC_BIAS_STANDBY at the end of CODEC probe()
can also be removed as the core will automatically do this after the CODEC
has been probed.
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Acked-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/soc/codecs/wm8985.c | 28 |
1 files changed, 1 insertions, 27 deletions
diff --git a/sound/soc/codecs/wm8985.c b/sound/soc/codecs/wm8985.c index ee380190399f..0b3b54c9971d 100644 --- a/sound/soc/codecs/wm8985.c +++ b/sound/soc/codecs/wm8985.c | |||
@@ -961,29 +961,6 @@ static int wm8985_set_bias_level(struct snd_soc_codec *codec, | |||
961 | return 0; | 961 | return 0; |
962 | } | 962 | } |
963 | 963 | ||
964 | #ifdef CONFIG_PM | ||
965 | static int wm8985_suspend(struct snd_soc_codec *codec) | ||
966 | { | ||
967 | wm8985_set_bias_level(codec, SND_SOC_BIAS_OFF); | ||
968 | return 0; | ||
969 | } | ||
970 | |||
971 | static int wm8985_resume(struct snd_soc_codec *codec) | ||
972 | { | ||
973 | wm8985_set_bias_level(codec, SND_SOC_BIAS_STANDBY); | ||
974 | return 0; | ||
975 | } | ||
976 | #else | ||
977 | #define wm8985_suspend NULL | ||
978 | #define wm8985_resume NULL | ||
979 | #endif | ||
980 | |||
981 | static int wm8985_remove(struct snd_soc_codec *codec) | ||
982 | { | ||
983 | wm8985_set_bias_level(codec, SND_SOC_BIAS_OFF); | ||
984 | return 0; | ||
985 | } | ||
986 | |||
987 | static int wm8985_probe(struct snd_soc_codec *codec) | 964 | static int wm8985_probe(struct snd_soc_codec *codec) |
988 | { | 965 | { |
989 | size_t i; | 966 | size_t i; |
@@ -1023,7 +1000,6 @@ static int wm8985_probe(struct snd_soc_codec *codec) | |||
1023 | snd_soc_update_bits(codec, WM8985_BIAS_CTRL, WM8985_BIASCUT, | 1000 | snd_soc_update_bits(codec, WM8985_BIAS_CTRL, WM8985_BIASCUT, |
1024 | WM8985_BIASCUT); | 1001 | WM8985_BIASCUT); |
1025 | 1002 | ||
1026 | wm8985_set_bias_level(codec, SND_SOC_BIAS_STANDBY); | ||
1027 | return 0; | 1003 | return 0; |
1028 | 1004 | ||
1029 | err_reg_enable: | 1005 | err_reg_enable: |
@@ -1064,10 +1040,8 @@ static struct snd_soc_dai_driver wm8985_dai = { | |||
1064 | 1040 | ||
1065 | static struct snd_soc_codec_driver soc_codec_dev_wm8985 = { | 1041 | static struct snd_soc_codec_driver soc_codec_dev_wm8985 = { |
1066 | .probe = wm8985_probe, | 1042 | .probe = wm8985_probe, |
1067 | .remove = wm8985_remove, | ||
1068 | .suspend = wm8985_suspend, | ||
1069 | .resume = wm8985_resume, | ||
1070 | .set_bias_level = wm8985_set_bias_level, | 1043 | .set_bias_level = wm8985_set_bias_level, |
1044 | .suspend_bias_off = true, | ||
1071 | 1045 | ||
1072 | .controls = wm8985_snd_controls, | 1046 | .controls = wm8985_snd_controls, |
1073 | .num_controls = ARRAY_SIZE(wm8985_snd_controls), | 1047 | .num_controls = ARRAY_SIZE(wm8985_snd_controls), |