diff options
author | Andrzej Hajda <a.hajda@samsung.com> | 2016-02-23 07:45:14 +0100 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-02-24 13:07:02 +0900 |
commit | 127a9cbb5860a7cf15f4d4703ab0454e4603e8ac (patch) | |
tree | e8670d453327f73f82221d119f017acc97f199c5 /sound/soc/codecs | |
parent | 250a99e70bf8628397901e10ad99a281dfc489d5 (diff) | |
download | op-kernel-dev-127a9cbb5860a7cf15f4d4703ab0454e4603e8ac.zip op-kernel-dev-127a9cbb5860a7cf15f4d4703ab0454e4603e8ac.tar.gz |
ASoC: max9867: fix type of variable containing error codes
value variable can contain error values and is compared with zero.
Its type must be signed.
The problem has been detected using coccinelle script
scripts/coccinelle/tests/unsigned_lesser_than_zero.cocci
Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Reviewed-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/codecs')
-rwxr-xr-x | sound/soc/codecs/max9867.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sound/soc/codecs/max9867.c b/sound/soc/codecs/max9867.c index ab7a94d..4fb6fc7 100755 --- a/sound/soc/codecs/max9867.c +++ b/sound/soc/codecs/max9867.c @@ -180,7 +180,8 @@ static int max9867_dai_hw_params(struct snd_pcm_substream *substream, { struct snd_soc_codec *codec = dai->codec; struct max9867_priv *max9867 = snd_soc_codec_get_drvdata(codec); - unsigned int ni_h, ni_l, value; + unsigned int ni_h, ni_l; + int value; value = get_ni_value(max9867->sysclk, params_rate(params)); if (value < 0) |