]> Git Repo - linux.git/commitdiff
ASoC: rt5663: fix a debug statement
authorDan Carpenter <[email protected]>
Thu, 13 Oct 2016 08:47:13 +0000 (11:47 +0300)
committerMark Brown <[email protected]>
Thu, 13 Oct 2016 13:01:32 +0000 (15:01 +0200)
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 <[email protected]>
Acked-by: Bard Liao <[email protected]>
Signed-off-by: Mark Brown <[email protected]>
sound/soc/codecs/rt5663.c

index 01a18d88f1eb19f4311319ce0a47771da34df131..00ff2788879e2a8982b3d0e79fa18ddb86fc188a 100644 (file)
@@ -1547,11 +1547,11 @@ static int rt5663_jack_detect(struct snd_soc_codec *codec, int jack_insert)
                        msleep(sleep_time[i]);
                        val = snd_soc_read(codec, RT5663_EM_JACK_TYPE_2) &
                                0x0003;
+                       dev_dbg(codec->dev, "%s: MX-00e7 val=%x sleep %d\n",
+                               __func__, val, sleep_time[i]);
                        i++;
                        if (val == 0x1 || val == 0x2 || val == 0x3)
                                break;
-                       dev_dbg(codec->dev, "%s: MX-00e7 val=%x sleep %d\n",
-                               __func__, val, sleep_time[i]);
                }
                dev_dbg(codec->dev, "%s val = %d\n", __func__, val);
                switch (val) {
This page took 0.063429 seconds and 4 git commands to generate.