From: John Hsu Date: Fri, 15 Jul 2016 02:06:17 +0000 (+0800) Subject: ASoC: nau8825: drop redundant idiom when converting integer to boolean X-Git-Tag: v4.8-rc1~29^2~7^2~5^6 X-Git-Url: http://git.cascardo.info/?a=commitdiff_plain;h=308e3e0bfa9ec9d1dc8c415e8a68ad4efd0fddfd;hp=-c;p=cascardo%2Flinux.git ASoC: nau8825: drop redundant idiom when converting integer to boolean Thanks Mark and Anatol for the discussion. According to the result, the standard C will translate any non-zero value into true, or false otherwise. QUOTE: "6.3.1.2 Boolean type When any scalar value is converted to _Bool, the result is 0 if the value compares equal to 0; otherwise, the result is 1 " Thus, the "!!" idiom is removed. Signed-off-by: John Hsu Signed-off-by: Mark Brown --- 308e3e0bfa9ec9d1dc8c415e8a68ad4efd0fddfd diff --git a/sound/soc/codecs/nau8825.c b/sound/soc/codecs/nau8825.c index a97418deb034..5c9707ac4bbf 100644 --- a/sound/soc/codecs/nau8825.c +++ b/sound/soc/codecs/nau8825.c @@ -1349,9 +1349,9 @@ static bool nau8825_is_jack_inserted(struct regmap *regmap) int status, jkdet; regmap_read(regmap, NAU8825_REG_JACK_DET_CTRL, &jkdet); - active_high = !!(jkdet & NAU8825_JACK_POLARITY); + active_high = jkdet & NAU8825_JACK_POLARITY; regmap_read(regmap, NAU8825_REG_I2C_DEVICE_ID, &status); - is_high = !!(status & NAU8825_GPIO2JD1); + is_high = status & NAU8825_GPIO2JD1; /* return jack connection status according to jack insertion logic * active high or active low. */