diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-10-13 04:47:13 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-10-13 09:01:32 -0400 |
commit | c975e39ccadf6dc047356c60773a043a2293b8cd (patch) | |
tree | 92f39190b50557bba806288797ba6ae43ba8bfa6 /sound/soc/codecs | |
parent | 513e43efafe329dad7b5794583b67ac898dcbdca (diff) |
ASoC: rt5663: fix a debug statement
We increment "i" before printing the debug statement. That makes it
the wrong sleep_time[] information and Smatch complains that the last
increment could be beyond the end of the array.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Bard Liao <bardliao@realtek.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/codecs')
-rw-r--r-- | sound/soc/codecs/rt5663.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sound/soc/codecs/rt5663.c b/sound/soc/codecs/rt5663.c index 01a18d88f1eb..00ff2788879e 100644 --- a/sound/soc/codecs/rt5663.c +++ b/sound/soc/codecs/rt5663.c | |||
@@ -1547,11 +1547,11 @@ static int rt5663_jack_detect(struct snd_soc_codec *codec, int jack_insert) | |||
1547 | msleep(sleep_time[i]); | 1547 | msleep(sleep_time[i]); |
1548 | val = snd_soc_read(codec, RT5663_EM_JACK_TYPE_2) & | 1548 | val = snd_soc_read(codec, RT5663_EM_JACK_TYPE_2) & |
1549 | 0x0003; | 1549 | 0x0003; |
1550 | dev_dbg(codec->dev, "%s: MX-00e7 val=%x sleep %d\n", | ||
1551 | __func__, val, sleep_time[i]); | ||
1550 | i++; | 1552 | i++; |
1551 | if (val == 0x1 || val == 0x2 || val == 0x3) | 1553 | if (val == 0x1 || val == 0x2 || val == 0x3) |
1552 | break; | 1554 | break; |
1553 | dev_dbg(codec->dev, "%s: MX-00e7 val=%x sleep %d\n", | ||
1554 | __func__, val, sleep_time[i]); | ||
1555 | } | 1555 | } |
1556 | dev_dbg(codec->dev, "%s val = %d\n", __func__, val); | 1556 | dev_dbg(codec->dev, "%s val = %d\n", __func__, val); |
1557 | switch (val) { | 1557 | switch (val) { |