diff options
author | Vinod Koul <vinod.koul@intel.com> | 2016-12-08 23:01:29 +0530 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-12-15 11:20:48 +0000 |
commit | 1c445a42c48754bb5f821478517ef1b9f861217a (patch) | |
tree | 9bfc5d0b3e32ebfd9bba0e81f71af77e8b7cf66a | |
parent | a5de5b74a50113564a1e0850e2da96c37c35e55d (diff) | |
download | linux-1c445a42c48754bb5f821478517ef1b9f861217a.tar.bz2 |
ASoC: max98090: remove superflous check for 'micbias'
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>
-rw-r--r-- | sound/soc/codecs/max98090.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/codecs/max98090.c b/sound/soc/codecs/max98090.c index 584aab83e478..66828480d484 100644 --- a/sound/soc/codecs/max98090.c +++ b/sound/soc/codecs/max98090.c @@ -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; } |