regulator: db8500-prcmu: Separate regulator registration from probe
authorLee Jones <lee.jones@linaro.org>
Fri, 18 May 2012 08:39:04 +0000 (09:39 +0100)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Fri, 18 May 2012 15:36:36 +0000 (16:36 +0100)
This will provide us with a convenient way to register regulators when
booting with Device Tree both enabled & disabled and will save us a
great deal of code duplication in time.

Signed-off-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
drivers/regulator/db8500-prcmu.c

index 87b2e83be11c60f7db9f17b41e09b57e4221846a..d6b4d4ce1051ba51103f10947c39cd3419e3cf1e 100644 (file)
@@ -410,40 +410,58 @@ dbx500_regulator_info[DB8500_NUM_REGULATORS] = {
        },
 };
 
+static __devinit int db8500_regulator_register(struct platform_device *pdev,
+                                       struct regulator_init_data *init_data,
+                                       int id,
+                                       struct device_node *np)
+{
+       struct dbx500_regulator_info *info;
+       struct regulator_config config = { };
+       int err;
+
+       /* assign per-regulator data */
+       info = &dbx500_regulator_info[id];
+       info->dev = &pdev->dev;
+
+       config.dev = &pdev->dev;
+       config.init_data = init_data;
+       config.driver_data = info;
+       config.of_node = np;
+
+       /* register with the regulator framework */
+       info->rdev = regulator_register(&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;
+       }
+
+       dev_dbg(rdev_get_dev(info->rdev),
+               "regulator-%s-probed\n", info->desc.name);
+
+       return 0;
+}
+
 static int __devinit db8500_regulator_probe(struct platform_device *pdev)
 {
        struct regulator_init_data *db8500_init_data =
                                        dev_get_platdata(&pdev->dev);
-       struct regulator_config config = { };
        int i, err;
 
        /* register all regulators */
        for (i = 0; i < ARRAY_SIZE(dbx500_regulator_info); i++) {
-               struct dbx500_regulator_info *info;
-               struct regulator_init_data *init_data = &db8500_init_data[i];
-
-               /* assign per-regulator data */
-               info = &dbx500_regulator_info[i];
-               info->dev = &pdev->dev;
-
-               config.dev = &pdev->dev;
-               config.init_data = init_data;
-               config.driver_data = info;
-
-               /* register with the regulator framework */
-               info->rdev = regulator_register(&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 (--i >= 0) {
-                               info = &dbx500_regulator_info[i];
-                               regulator_unregister(info->rdev);
-                       }
+               err = db8500_regulator_register(pdev,
+                                               &db8500_init_data[i],
+                                               i, NULL);
+               if (err)
                        return err;
-               }
 
                dev_dbg(rdev_get_dev(info->rdev),
                        "regulator-%s-probed\n", info->desc.name);
@@ -451,8 +469,7 @@ static int __devinit db8500_regulator_probe(struct platform_device *pdev)
        err = ux500_regulator_debug_init(pdev,
                                         dbx500_regulator_info,
                                         ARRAY_SIZE(dbx500_regulator_info));
-
-       return err;
+       return 0;
 }
 
 static int __exit db8500_regulator_remove(struct platform_device *pdev)