OSDN Git Service

ASoC: fsl_easrc: Fix -Wunused-but-set-variable
authorShengjiu Wang <shengjiu.wang@nxp.com>
Wed, 3 Jun 2020 03:39:40 +0000 (11:39 +0800)
committerMark Brown <broonie@kernel.org>
Mon, 15 Jun 2020 17:51:03 +0000 (18:51 +0100)
Obtained with:
$ make W=1

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

Fixes: 955ac624058f ("ASoC: fsl_easrc: Add EASRC ASoC CPU DAI drivers")
Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com>
Acked-by: Nicolin 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.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/fsl/fsl_easrc.c

index f227308..7d8bf9d 100644 (file)
@@ -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;
        unsigned int in_rate = ctx_priv->in_params.norm_rate;
        unsigned int out_rate = ctx_priv->out_params.norm_rate;
-       unsigned int int_bits;
        unsigned int frac_bits;
        u64 val;
        u32 *r;
 
        switch (easrc_priv->rs_num_taps) {
        case EASRC_RS_32_TAPS:
-               int_bits = 5;
+               /* integer bits = 5; */
                frac_bits = 39;
                break;
        case EASRC_RS_64_TAPS:
-               int_bits = 6;
+               /* integer bits = 6; */
                frac_bits = 38;
                break;
        case EASRC_RS_128_TAPS:
-               int_bits = 7;
+               /* integer bits = 7; */
                frac_bits = 37;
                break;
        default:
@@ -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)
 {
        struct fsl_easrc_ctx_priv *ctx_priv;
-       struct device *dev;
        struct fsl_asrc *easrc;
 
        if (!ctx)
@@ -1209,7 +1207,6 @@ static int fsl_easrc_set_ctx_organziation(struct fsl_asrc_pair *ctx)
 
        easrc = ctx->asrc;
        ctx_priv = ctx->private;
-       dev = &easrc->pdev->dev;
 
        /* input interleaving parameters */
        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)
 {
        unsigned long lock_flags;
        struct fsl_asrc *easrc;
-       struct device *dev;
 
        if (!ctx)
                return;
 
        easrc = ctx->asrc;
-       dev = &easrc->pdev->dev;
 
        spin_lock_irqsave(&easrc->lock, lock_flags);