AND bitwise operation is used for retry counter and lock status, however in this case, it's invalid as condition statement.
This commit fixes this bug with simpler representation. This bug is detected by sparse with below warning:
sound/soc/codecs/da7213.c:775:57: warning: dubious: x & !y
Fixes: d575b0b0f01a ("ASoC: da7213: Add checking of SRM lock status before enabling DAI") Cc: stable@vger.kernel.org # 4.4+ Cc: Adam Thomson Adam.Thomson.Opensource@diasemi.com Signed-off-by: Takashi Sakamoto o-takashi@sakamocchi.jp --- sound/soc/codecs/da7213.c | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-)
diff --git a/sound/soc/codecs/da7213.c b/sound/soc/codecs/da7213.c index 6dd7578..11d257c 100644 --- a/sound/soc/codecs/da7213.c +++ b/sound/soc/codecs/da7213.c @@ -737,7 +737,6 @@ static int da7213_dai_event(struct snd_soc_dapm_widget *w, struct da7213_priv *da7213 = snd_soc_codec_get_drvdata(codec); u8 pll_ctrl, pll_status; int i = 0; - bool srm_lock = false;
switch (event) { case SND_SOC_DAPM_PRE_PMU: @@ -766,15 +765,12 @@ static int da7213_dai_event(struct snd_soc_dapm_widget *w, /* Check SRM has locked */ do { pll_status = snd_soc_read(codec, DA7213_PLL_STATUS); - if (pll_status & DA7219_PLL_SRM_LOCK) { - srm_lock = true; - } else { - ++i; - msleep(50); - } - } while ((i < DA7213_SRM_CHECK_RETRIES) & (!srm_lock)); + if (pll_status & DA7219_PLL_SRM_LOCK) + break; + msleep(50); + } while (++i < DA7213_SRM_CHECK_RETRIES);
- if (!srm_lock) + if (i >= DA7213_SRM_CHECK_RETRIES) dev_warn(codec->dev, "SRM failed to lock\n");
return 0;