diff options
author | Vinod Koul <vinod.koul@intel.com> | 2016-12-08 12:31:36 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-12-15 06:34:44 -0500 |
commit | e85a709974db40779f5942ed81e9262c62179863 (patch) | |
tree | f0837588aa785e78c3a4f7a98d75b35f88b60f66 /sound | |
parent | a5de5b74a50113564a1e0850e2da96c37c35e55d (diff) |
ASoC: samsung: smdk_wm8580: remove unused ‘bfs’
In smdk_hw_params(), 'bfs' is initialized and assigned bits based on
params_width, but never used.
We could have removed the whole switch case but then driver might be
relying on checking bits, so I have kept the case for now.
sound/soc/samsung/smdk_wm8580.c: In function ‘smdk_hw_params’:
sound/soc/samsung/smdk_wm8580.c:35:6: warning: variable ‘bfs’ set but not used [-Wunused-but-set-variable]
int bfs, rfs, ret;
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/soc/samsung/smdk_wm8580.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/sound/soc/samsung/smdk_wm8580.c b/sound/soc/samsung/smdk_wm8580.c index de724ce7b955..6e4dfa7e2c89 100644 --- a/sound/soc/samsung/smdk_wm8580.c +++ b/sound/soc/samsung/smdk_wm8580.c | |||
@@ -32,14 +32,11 @@ static int smdk_hw_params(struct snd_pcm_substream *substream, | |||
32 | struct snd_soc_pcm_runtime *rtd = substream->private_data; | 32 | struct snd_soc_pcm_runtime *rtd = substream->private_data; |
33 | struct snd_soc_dai *codec_dai = rtd->codec_dai; | 33 | struct snd_soc_dai *codec_dai = rtd->codec_dai; |
34 | unsigned int pll_out; | 34 | unsigned int pll_out; |
35 | int bfs, rfs, ret; | 35 | int rfs, ret; |
36 | 36 | ||
37 | switch (params_width(params)) { | 37 | switch (params_width(params)) { |
38 | case 8: | 38 | case 8: |
39 | bfs = 16; | ||
40 | break; | ||
41 | case 16: | 39 | case 16: |
42 | bfs = 32; | ||
43 | break; | 40 | break; |
44 | default: | 41 | default: |
45 | return -EINVAL; | 42 | return -EINVAL; |