regulator: tps6586x: Remove unnecessary rdev[] array
authorAxel Lin <axel.lin@ingics.com>
Sat, 8 Mar 2014 13:15:59 +0000 (21:15 +0800)
committerMark Brown <broonie@linaro.org>
Thu, 13 Mar 2014 11:42:23 +0000 (11:42 +0000)
Now we are using devm_regulator_register(), so we don't need the rdev[] array
to store return value of devm_regulator_register. Use a *rdev variable is
enough for checking return status.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/regulator/tps6586x-regulator.c

index e36f3569767fe653abd50a517840896761717036..32f38a63d944eaaf148a5a1f7c4034ab372e7f0d 100644 (file)
@@ -396,7 +396,7 @@ static int tps6586x_regulator_probe(struct platform_device *pdev)
 {
        struct tps6586x_regulator *ri = NULL;
        struct regulator_config config = { };
-       struct regulator_dev **rdev;
+       struct regulator_dev *rdev;
        struct regulator_init_data *reg_data;
        struct tps6586x_platform_data *pdata;
        struct of_regulator_match *tps6586x_reg_matches = NULL;
@@ -416,11 +416,6 @@ static int tps6586x_regulator_probe(struct platform_device *pdev)
                return -ENODEV;
        }
 
-       rdev = devm_kzalloc(&pdev->dev, TPS6586X_ID_MAX_REGULATOR *
-                               sizeof(*rdev), GFP_KERNEL);
-       if (!rdev)
-               return -ENOMEM;
-
        version = tps6586x_get_version(pdev->dev.parent);
 
        for (id = 0; id < TPS6586X_ID_MAX_REGULATOR; ++id) {
@@ -447,12 +442,11 @@ static int tps6586x_regulator_probe(struct platform_device *pdev)
                if (tps6586x_reg_matches)
                        config.of_node = tps6586x_reg_matches[id].of_node;
 
-               rdev[id] = devm_regulator_register(&pdev->dev, &ri->desc,
-                                                  &config);
-               if (IS_ERR(rdev[id])) {
+               rdev = devm_regulator_register(&pdev->dev, &ri->desc, &config);
+               if (IS_ERR(rdev)) {
                        dev_err(&pdev->dev, "failed to register regulator %s\n",
                                        ri->desc.name);
-                       return PTR_ERR(rdev[id]);
+                       return PTR_ERR(rdev);
                }
 
                if (reg_data) {