ASoC: max98090: remove superflous check for 'micbias'
authorVinod Koul <vinod.koul@intel.com>
Thu, 8 Dec 2016 17:31:29 +0000 (23:01 +0530)
committerMark Brown <broonie@kernel.org>
Thu, 15 Dec 2016 11:20:48 +0000 (11:20 +0000)
In max98090_probe(), code checks for micbias being out of range. The
'micbias' variable in unsigned and checked against M98090_MBVSEL_2V2 which
is zero, so remove this check.

sound/soc/codecs/max98090.c: In function ‘max98090_probe’:
sound/soc/codecs/max98090.c:2459:2: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
  } else if (micbias < M98090_MBVSEL_2V2 || micbias > M98090_MBVSEL_2V8) {

Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/max98090.c

index 584aab83e4783b12232cc94000431efacbc39d95..66828480d484ca11912d8b163efe0c51445b155c 100644 (file)
@@ -2456,7 +2456,7 @@ static int max98090_probe(struct snd_soc_codec *codec)
        if (err) {
                micbias = M98090_MBVSEL_2V8;
                dev_info(codec->dev, "use default 2.8v micbias\n");
-       } else if (micbias < M98090_MBVSEL_2V2 || micbias > M98090_MBVSEL_2V8) {
+       } else if (micbias > M98090_MBVSEL_2V8) {
                dev_err(codec->dev, "micbias out of range 0x%x\n", micbias);
                micbias = M98090_MBVSEL_2V8;
        }