ab8500_charger: Use devm_regulator_get API
authorSachin Kamat <sachin.kamat@linaro.org>
Fri, 7 Dec 2012 11:53:28 +0000 (17:23 +0530)
committerAnton Vorontsov <anton@enomsg.org>
Sun, 6 Jan 2013 03:07:22 +0000 (19:07 -0800)
devm_regulator_get() is device managed and makes error handling and code
cleanup simpler.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Acked-by: Arun Murthy <arun.murthy@stericsson.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Anton Vorontsov <anton@enomsg.org>
drivers/power/ab8500_charger.c

index e5755f0ba831a998f8d126c5683e5b41756af335..c342f79cb5888dab1a43fd4e452b9fc6ab927203 100644 (file)
@@ -2604,9 +2604,6 @@ static int ab8500_charger_remove(struct platform_device *pdev)
                free_irq(irq, di);
        }
 
-       /* disable the regulator */
-       regulator_put(di->regu);
-
        /* Backup battery voltage and current disable */
        ret = abx500_mask_and_set_register_interruptible(di->dev,
                AB8500_RTC, AB8500_RTC_CTRL_REG, RTC_BUP_CH_ENA, 0);
@@ -2759,7 +2756,7 @@ static int ab8500_charger_probe(struct platform_device *pdev)
         * is a charger connected to avoid erroneous BTEMP_HIGH/LOW
         * interrupts during charging
         */
-       di->regu = regulator_get(di->dev, "vddadc");
+       di->regu = devm_regulator_get(di->dev, "vddadc");
        if (IS_ERR(di->regu)) {
                ret = PTR_ERR(di->regu);
                dev_err(di->dev, "failed to get vddadc regulator\n");
@@ -2771,14 +2768,14 @@ static int ab8500_charger_probe(struct platform_device *pdev)
        ret = ab8500_charger_init_hw_registers(di);
        if (ret) {
                dev_err(di->dev, "failed to initialize ABB registers\n");
-               goto free_regulator;
+               goto free_charger_wq;
        }
 
        /* Register AC charger class */
        ret = power_supply_register(di->dev, &di->ac_chg.psy);
        if (ret) {
                dev_err(di->dev, "failed to register AC charger\n");
-               goto free_regulator;
+               goto free_charger_wq;
        }
 
        /* Register USB charger class */
@@ -2852,8 +2849,6 @@ free_usb:
        power_supply_unregister(&di->usb_chg.psy);
 free_ac:
        power_supply_unregister(&di->ac_chg.psy);
-free_regulator:
-       regulator_put(di->regu);
 free_charger_wq:
        destroy_workqueue(di->charger_wq);
        return ret;