ASoC: nau8825: jack connection decision with different insertion logic

The original design only covers the jack insertion logic is active low.
Add more condition to cover no matter the logic is active low and high.

Signed-off-by: John Hsu <KCHSU0@nuvoton.com>
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/codecs/nau8825.c b/sound/soc/codecs/nau8825.c
index 3f30e6ed210cb3ec2a4f48d326f3fd5de77db319..a97418deb034c1a98260b34060170eb3eb85d59d 100644 (file)
--- a/sound/soc/codecs/nau8825.c
+++ b/sound/soc/codecs/nau8825.c
@@ -1345,10 +1345,17 @@ EXPORT_SYMBOL_GPL(nau8825_enable_jack_detect);
 
 static bool nau8825_is_jack_inserted(struct regmap *regmap)
 {
-       int status;
+       bool active_high, is_high;
+       int status, jkdet;
 
+       regmap_read(regmap, NAU8825_REG_JACK_DET_CTRL, &jkdet);
+       active_high = !!(jkdet & NAU8825_JACK_POLARITY);
        regmap_read(regmap, NAU8825_REG_I2C_DEVICE_ID, &status);
-       return !(status & NAU8825_GPIO2JD1);
+       is_high = !!(status & NAU8825_GPIO2JD1);
+       /* return jack connection status according to jack insertion logic
+        * active high or active low.
+        */
+       return active_high == is_high;
 }
 
 static void nau8825_restart_jack_detection(struct regmap *regmap)