ASoC: tlv320adcx140: Register a callback to disable the regulator_disable
authorZheyu Ma <zheyuma97@gmail.com>
Tue, 10 May 2022 05:30:31 +0000 (13:30 +0800)
committerMark Brown <broonie@kernel.org>
Tue, 10 May 2022 11:31:32 +0000 (12:31 +0100)
The driver should register a callback that will deal with the disabling
when it fails to probe.

Signed-off-by: Zheyu Ma <zheyuma97@gmail.com>
Link: https://lore.kernel.org/r/20220510053031.1685337-1-zheyuma97@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/tlv320adcx140.c

index 0b06fbb141713e681e282866684ecdb70a315246..b55f0b836932ba9655bcba780569f5efdd0f8405 100644 (file)
@@ -1083,6 +1083,13 @@ static const struct of_device_id tlv320adcx140_of_match[] = {
 MODULE_DEVICE_TABLE(of, tlv320adcx140_of_match);
 #endif
 
+static void adcx140_disable_regulator(void *arg)
+{
+       struct adcx140_priv *adcx140 = arg;
+
+       regulator_disable(adcx140->supply_areg);
+}
+
 static int adcx140_i2c_probe(struct i2c_client *i2c)
 {
        struct adcx140_priv *adcx140;
@@ -1112,6 +1119,10 @@ static int adcx140_i2c_probe(struct i2c_client *i2c)
                        dev_err(adcx140->dev, "Failed to enable areg\n");
                        return ret;
                }
+
+               ret = devm_add_action_or_reset(&i2c->dev, adcx140_disable_regulator, adcx140);
+               if (ret)
+                       return ret;
        }
 
        adcx140->regmap = devm_regmap_init_i2c(i2c, &adcx140_i2c_regmap);