regulator: max8997: Remove unnecessary **rdev from struct max8997_data
authorAxel Lin <axel.lin@ingics.com>
Sun, 9 Mar 2014 07:29:31 +0000 (15:29 +0800)
committerMark Brown <broonie@linaro.org>
Mon, 10 Mar 2014 10:11:15 +0000 (10:11 +0000)
Now we are using devm_regulator_register(), so we don't need to allocate *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>
Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/regulator/max8997.c

index f657d8d..90b4c53 100644 (file)
@@ -38,7 +38,6 @@ struct max8997_data {
        struct device *dev;
        struct max8997_dev *iodev;
        int num_regulators;
-       struct regulator_dev **rdev;
        int ramp_delay; /* in mV/us */
 
        bool buck1_gpiodvs;
@@ -1029,10 +1028,10 @@ static int max8997_pmic_probe(struct platform_device *pdev)
        struct max8997_dev *iodev = dev_get_drvdata(pdev->dev.parent);
        struct max8997_platform_data *pdata = iodev->pdata;
        struct regulator_config config = { };
-       struct regulator_dev **rdev;
+       struct regulator_dev *rdev;
        struct max8997_data *max8997;
        struct i2c_client *i2c;
-       int i, ret, size, nr_dvs;
+       int i, ret, nr_dvs;
        u8 max_buck1 = 0, max_buck2 = 0, max_buck5 = 0;
 
        if (!pdata) {
@@ -1051,12 +1050,6 @@ static int max8997_pmic_probe(struct platform_device *pdev)
        if (!max8997)
                return -ENOMEM;
 
-       size = sizeof(struct regulator_dev *) * pdata->num_regulators;
-       max8997->rdev = devm_kzalloc(&pdev->dev, size, GFP_KERNEL);
-       if (!max8997->rdev)
-               return -ENOMEM;
-
-       rdev = max8997->rdev;
        max8997->dev = &pdev->dev;
        max8997->iodev = iodev;
        max8997->num_regulators = pdata->num_regulators;
@@ -1204,12 +1197,12 @@ static int max8997_pmic_probe(struct platform_device *pdev)
                config.driver_data = max8997;
                config.of_node = pdata->regulators[i].reg_node;
 
-               rdev[i] = devm_regulator_register(&pdev->dev, &regulators[id],
-                                                 &config);
-               if (IS_ERR(rdev[i])) {
+               rdev = devm_regulator_register(&pdev->dev, &regulators[id],
+                                              &config);
+               if (IS_ERR(rdev)) {
                        dev_err(max8997->dev, "regulator init failed for %d\n",
                                        id);
-                       return PTR_ERR(rdev[i]);
+                       return PTR_ERR(rdev);
                }
        }