ASoC: tlv320dac33: clarify expression
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Fri, 12 Mar 2021 18:22:45 +0000 (12:22 -0600)
committerMark Brown <broonie@kernel.org>
Thu, 18 Mar 2021 13:50:54 +0000 (13:50 +0000)
cppcheck warning:

sound/soc/codecs/tlv320dac33.c:1074:43: style: Clarify calculation
precedence for '%' and '?'. [clarifyCalculation]
    (dac33->alarm_threshold % period_size ?
                                          ^

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20210312182246.5153-23-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/tlv320dac33.c

index d905e03..48572d6 100644 (file)
@@ -1071,7 +1071,7 @@ static void dac33_calculate_times(struct snd_pcm_substream *substream,
                         */
                        dac33->nsample = period_size *
                                ((dac33->alarm_threshold / period_size) +
-                               (dac33->alarm_threshold % period_size ?
+                                ((dac33->alarm_threshold % period_size) ?
                                1 : 0));
                else if (period_size > nsample_limit)
                        dac33->nsample = nsample_limit;