The regulators need to be disabled in the aic32x4_register_clocks()
failure case as well.

Fixes: 9d4befff5a95 ("ASoC: codec: tlv3204: Moving GPIO reset and add ADC
reset")
Signed-off-by: Matthias Schiffer <[email protected]>
---
 sound/soc/codecs/tlv320aic32x4.c | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/sound/soc/codecs/tlv320aic32x4.c b/sound/soc/codecs/tlv320aic32x4.c
index 8dcea566b375..a45fb496082c 100644
--- a/sound/soc/codecs/tlv320aic32x4.c
+++ b/sound/soc/codecs/tlv320aic32x4.c
@@ -1230,8 +1230,7 @@ int aic32x4_probe(struct device *dev, struct regmap 
*regmap)
                        &soc_component_dev_aic32x4, &aic32x4_dai, 1);
        if (ret) {
                dev_err(dev, "Failed to register component\n");
-               aic32x4_disable_regulators(aic32x4);
-               return ret;
+               goto err_disable_regulators;
        }
 
        if (gpio_is_valid(aic32x4->rstn_gpio)) {
@@ -1242,9 +1241,14 @@ int aic32x4_probe(struct device *dev, struct regmap 
*regmap)
 
        ret = aic32x4_register_clocks(dev, aic32x4->mclk_name);
        if (ret)
-               return ret;
+               goto err_disable_regulators;
 
        return 0;
+
+err_disable_regulators:
+       aic32x4_disable_regulators(aic32x4);
+
+       return ret;
 }
 EXPORT_SYMBOL(aic32x4_probe);
 
-- 
2.17.1

Reply via email to