]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
ASoC: samsung: smdk_wm8580: remove unused ‘bfs’
authorVinod Koul <vinod.koul@intel.com>
Thu, 8 Dec 2016 17:31:36 +0000 (23:01 +0530)
committerMark Brown <broonie@kernel.org>
Thu, 15 Dec 2016 11:34:44 +0000 (11:34 +0000)
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>
sound/soc/samsung/smdk_wm8580.c

index de724ce7b955e2935f9df628bd3f8bc533b67fd3..6e4dfa7e2c89434c26fc6eff4f21e72c5cadebbf 100644 (file)
@@ -32,14 +32,11 @@ static int smdk_hw_params(struct snd_pcm_substream *substream,
        struct snd_soc_pcm_runtime *rtd = substream->private_data;
        struct snd_soc_dai *codec_dai = rtd->codec_dai;
        unsigned int pll_out;
-       int bfs, rfs, ret;
+       int rfs, ret;
 
        switch (params_width(params)) {
        case 8:
-               bfs = 16;
-               break;
        case 16:
-               bfs = 32;
                break;
        default:
                return -EINVAL;