Commit 633a2c7d authored by Shengjiu Wang's avatar Shengjiu Wang Committed by Mark Brown

ASoC: fsl_easrc: Fix -Wunused-but-set-variable

Obtained with:
$ make W=1

  unsigned int int_bits;
               ^
  struct device *dev;
                 ^
  struct device *dev;
                 ^

Fixes: 955ac624 ("ASoC: fsl_easrc: Add EASRC ASoC CPU DAI drivers")
Reported-by: default avatarkbuild test robot <lkp@intel.com>
Signed-off-by: default avatarShengjiu Wang <shengjiu.wang@nxp.com>
Acked-by: default avatarNicolin Chen <nicoleotsuka@gmail.com>
sound/soc/fsl/fsl_easrc.c: In function 'fsl_easrc_set_rs_ratio':
sound/soc/fsl/fsl_easrc.c:182:15: warning: variable 'int_bits' set but not used [-Wunused-but-set-variable]
sound/soc/fsl/fsl_easrc.c: In function 'fsl_easrc_set_ctx_organziation':
sound/soc/fsl/fsl_easrc.c:1204:17: warning: variable 'dev' set but not used [-Wunused-but-set-variable]
sound/soc/fsl/fsl_easrc.c: In function 'fsl_easrc_release_context':
sound/soc/fsl/fsl_easrc.c:1294:17: warning: variable 'dev' set but not used [-Wunused-but-set-variable]
Link: https://lore.kernel.org/r/91ceb59e3bce31c9e93abba06f5156692ff5c71e.1591155860.git.shengjiu.wang@nxp.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent e4cc0aaa
...@@ -179,22 +179,21 @@ static int fsl_easrc_set_rs_ratio(struct fsl_asrc_pair *ctx) ...@@ -179,22 +179,21 @@ static int fsl_easrc_set_rs_ratio(struct fsl_asrc_pair *ctx)
struct fsl_easrc_ctx_priv *ctx_priv = ctx->private; struct fsl_easrc_ctx_priv *ctx_priv = ctx->private;
unsigned int in_rate = ctx_priv->in_params.norm_rate; unsigned int in_rate = ctx_priv->in_params.norm_rate;
unsigned int out_rate = ctx_priv->out_params.norm_rate; unsigned int out_rate = ctx_priv->out_params.norm_rate;
unsigned int int_bits;
unsigned int frac_bits; unsigned int frac_bits;
u64 val; u64 val;
u32 *r; u32 *r;
switch (easrc_priv->rs_num_taps) { switch (easrc_priv->rs_num_taps) {
case EASRC_RS_32_TAPS: case EASRC_RS_32_TAPS:
int_bits = 5; /* integer bits = 5; */
frac_bits = 39; frac_bits = 39;
break; break;
case EASRC_RS_64_TAPS: case EASRC_RS_64_TAPS:
int_bits = 6; /* integer bits = 6; */
frac_bits = 38; frac_bits = 38;
break; break;
case EASRC_RS_128_TAPS: case EASRC_RS_128_TAPS:
int_bits = 7; /* integer bits = 7; */
frac_bits = 37; frac_bits = 37;
break; break;
default: default:
...@@ -1201,7 +1200,6 @@ static int fsl_easrc_set_ctx_format(struct fsl_asrc_pair *ctx, ...@@ -1201,7 +1200,6 @@ static int fsl_easrc_set_ctx_format(struct fsl_asrc_pair *ctx,
static int fsl_easrc_set_ctx_organziation(struct fsl_asrc_pair *ctx) static int fsl_easrc_set_ctx_organziation(struct fsl_asrc_pair *ctx)
{ {
struct fsl_easrc_ctx_priv *ctx_priv; struct fsl_easrc_ctx_priv *ctx_priv;
struct device *dev;
struct fsl_asrc *easrc; struct fsl_asrc *easrc;
if (!ctx) if (!ctx)
...@@ -1209,7 +1207,6 @@ static int fsl_easrc_set_ctx_organziation(struct fsl_asrc_pair *ctx) ...@@ -1209,7 +1207,6 @@ static int fsl_easrc_set_ctx_organziation(struct fsl_asrc_pair *ctx)
easrc = ctx->asrc; easrc = ctx->asrc;
ctx_priv = ctx->private; ctx_priv = ctx->private;
dev = &easrc->pdev->dev;
/* input interleaving parameters */ /* input interleaving parameters */
regmap_update_bits(easrc->regmap, REG_EASRC_CIA(ctx->index), regmap_update_bits(easrc->regmap, REG_EASRC_CIA(ctx->index),
...@@ -1291,13 +1288,11 @@ static void fsl_easrc_release_context(struct fsl_asrc_pair *ctx) ...@@ -1291,13 +1288,11 @@ static void fsl_easrc_release_context(struct fsl_asrc_pair *ctx)
{ {
unsigned long lock_flags; unsigned long lock_flags;
struct fsl_asrc *easrc; struct fsl_asrc *easrc;
struct device *dev;
if (!ctx) if (!ctx)
return; return;
easrc = ctx->asrc; easrc = ctx->asrc;
dev = &easrc->pdev->dev;
spin_lock_irqsave(&easrc->lock, lock_flags); spin_lock_irqsave(&easrc->lock, lock_flags);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment