regulator: db8500-prcmu: use devm_regulator_register()
authorJingoo Han <jg1.han@samsung.com>
Fri, 6 Dec 2013 07:08:38 +0000 (16:08 +0900)
committerMark Brown <broonie@linaro.org>
Mon, 9 Dec 2013 17:40:57 +0000 (17:40 +0000)
Use devm_regulator_register() to make cleanup paths simpler.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Bengt Jonsson <bengt.g.jonsson@stericsson.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/regulator/db8500-prcmu.c

index a53c11a529d5f65dc55e6790167c4d69f5c68de0..846acf240e486d138319cbf26ce9434b71e6bc6e 100644 (file)
@@ -431,17 +431,11 @@ static int db8500_regulator_register(struct platform_device *pdev,
        config.of_node = np;
 
        /* register with the regulator framework */
-       info->rdev = regulator_register(&info->desc, &config);
+       info->rdev = devm_regulator_register(&pdev->dev, &info->desc, &config);
        if (IS_ERR(info->rdev)) {
                err = PTR_ERR(info->rdev);
                dev_err(&pdev->dev, "failed to register %s: err %i\n",
                        info->desc.name, err);
-
-               /* if failing, unregister all earlier regulators */
-               while (--id >= 0) {
-                       info = &dbx500_regulator_info[id];
-                       regulator_unregister(info->rdev);
-               }
                return err;
        }
 
@@ -530,20 +524,8 @@ static int db8500_regulator_probe(struct platform_device *pdev)
 
 static int db8500_regulator_remove(struct platform_device *pdev)
 {
-       int i;
-
        ux500_regulator_debug_exit();
 
-       for (i = 0; i < ARRAY_SIZE(dbx500_regulator_info); i++) {
-               struct dbx500_regulator_info *info;
-               info = &dbx500_regulator_info[i];
-
-               dev_vdbg(rdev_get_dev(info->rdev),
-                       "regulator-%s-remove\n", info->desc.name);
-
-               regulator_unregister(info->rdev);
-       }
-
        return 0;
 }