regulator: tps65217: Use simplified DT parsing
authorMark Brown <broonie@kernel.org>
Tue, 9 Sep 2014 22:16:11 +0000 (23:16 +0100)
committerMark Brown <broonie@kernel.org>
Wed, 10 Sep 2014 11:02:48 +0000 (12:02 +0100)
Remove a bunch of code in favour of data.

Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/regulator/tps65217-regulator.c

index 42315468ec680b82ac54d56183bb09aa327dfcd7..adbe4fc5cf07b1951a501d6d320fc6c56a88a0c3 100644 (file)
 #include <linux/regulator/machine.h>
 #include <linux/mfd/tps65217.h>
 
-#define TPS65217_REGULATOR(_name, _id, _ops, _n, _vr, _vm, _em, _t, _lr, _nlr) \
+#define TPS65217_REGULATOR(_name, _id, _of_match, _ops, _n, _vr, _vm, _em, \
+                           _t, _lr, _nlr) \
        {                                               \
                .name           = _name,                \
                .id             = _id,                  \
+               .of_match       = of_match_ptr(_of_match),    \
+               .regulators_node= of_match_ptr("regulators"), \
                .ops            = &_ops,                \
                .n_voltages     = _n,                   \
                .type           = REGULATOR_VOLTAGE,    \
@@ -138,87 +141,40 @@ static struct regulator_ops tps65217_pmic_ldo1_ops = {
 };
 
 static const struct regulator_desc regulators[] = {
-       TPS65217_REGULATOR("DCDC1", TPS65217_DCDC_1, tps65217_pmic_ops, 64,
-                          TPS65217_REG_DEFDCDC1, TPS65217_DEFDCDCX_DCDC_MASK,
-                          TPS65217_ENABLE_DC1_EN, NULL, tps65217_uv1_ranges,
-                          2),  /* DCDC1 voltage range: 900000 ~ 1800000 */
-       TPS65217_REGULATOR("DCDC2", TPS65217_DCDC_2, tps65217_pmic_ops, 64,
-                          TPS65217_REG_DEFDCDC2, TPS65217_DEFDCDCX_DCDC_MASK,
-                          TPS65217_ENABLE_DC2_EN, NULL, tps65217_uv1_ranges,
+       TPS65217_REGULATOR("DCDC1", TPS65217_DCDC_1, "dcdc1",
+                          tps65217_pmic_ops, 64, TPS65217_REG_DEFDCDC1,
+                          TPS65217_DEFDCDCX_DCDC_MASK, TPS65217_ENABLE_DC1_EN,
+                          NULL, tps65217_uv1_ranges, 2),
+       TPS65217_REGULATOR("DCDC2", TPS65217_DCDC_2, "dcdc2",
+                          tps65217_pmic_ops, 64, TPS65217_REG_DEFDCDC2,
+                          TPS65217_DEFDCDCX_DCDC_MASK, TPS65217_ENABLE_DC2_EN,
+                          NULL, tps65217_uv1_ranges,
                           ARRAY_SIZE(tps65217_uv1_ranges)),
-       TPS65217_REGULATOR("DCDC3", TPS65217_DCDC_3, tps65217_pmic_ops, 64,
-                          TPS65217_REG_DEFDCDC3, TPS65217_DEFDCDCX_DCDC_MASK,
-                          TPS65217_ENABLE_DC3_EN, NULL, tps65217_uv1_ranges,
-                          1),  /* DCDC3 voltage range: 900000 ~ 1500000 */
-       TPS65217_REGULATOR("LDO1", TPS65217_LDO_1, tps65217_pmic_ldo1_ops, 16,
-                          TPS65217_REG_DEFLDO1, TPS65217_DEFLDO1_LDO1_MASK,
-                          TPS65217_ENABLE_LDO1_EN, LDO1_VSEL_table, NULL, 0),
-       TPS65217_REGULATOR("LDO2", TPS65217_LDO_2, tps65217_pmic_ops, 64,
-                          TPS65217_REG_DEFLDO2, TPS65217_DEFLDO2_LDO2_MASK,
-                          TPS65217_ENABLE_LDO2_EN, NULL, tps65217_uv1_ranges,
+       TPS65217_REGULATOR("DCDC3", TPS65217_DCDC_3, "dcdc3",
+                          tps65217_pmic_ops, 64, TPS65217_REG_DEFDCDC3,
+                          TPS65217_DEFDCDCX_DCDC_MASK, TPS65217_ENABLE_DC3_EN,
+                          NULL, tps65217_uv1_ranges, 1),
+       TPS65217_REGULATOR("LDO1", TPS65217_LDO_1, "ldo1",
+                          tps65217_pmic_ldo1_ops, 16, TPS65217_REG_DEFLDO1,
+                          TPS65217_DEFLDO1_LDO1_MASK, TPS65217_ENABLE_LDO1_EN,
+                          LDO1_VSEL_table, NULL, 0),
+       TPS65217_REGULATOR("LDO2", TPS65217_LDO_2, "ldo2", tps65217_pmic_ops,
+                          64, TPS65217_REG_DEFLDO2,
+                          TPS65217_DEFLDO2_LDO2_MASK, TPS65217_ENABLE_LDO2_EN,
+                          NULL, tps65217_uv1_ranges,
                           ARRAY_SIZE(tps65217_uv1_ranges)),
-       TPS65217_REGULATOR("LDO3", TPS65217_LDO_3, tps65217_pmic_ops, 32,
-                          TPS65217_REG_DEFLS1, TPS65217_DEFLDO3_LDO3_MASK,
+       TPS65217_REGULATOR("LDO3", TPS65217_LDO_3, "ldo3", tps65217_pmic_ops,
+                          32, TPS65217_REG_DEFLS1, TPS65217_DEFLDO3_LDO3_MASK,
                           TPS65217_ENABLE_LS1_EN | TPS65217_DEFLDO3_LDO3_EN,
                           NULL, tps65217_uv2_ranges,
                           ARRAY_SIZE(tps65217_uv2_ranges)),
-       TPS65217_REGULATOR("LDO4", TPS65217_LDO_4, tps65217_pmic_ops, 32,
-                          TPS65217_REG_DEFLS2, TPS65217_DEFLDO4_LDO4_MASK,
+       TPS65217_REGULATOR("LDO4", TPS65217_LDO_4, "ldo4", tps65217_pmic_ops,
+                          32, TPS65217_REG_DEFLS2, TPS65217_DEFLDO4_LDO4_MASK,
                           TPS65217_ENABLE_LS2_EN | TPS65217_DEFLDO4_LDO4_EN,
                           NULL, tps65217_uv2_ranges,
                           ARRAY_SIZE(tps65217_uv2_ranges)),
 };
 
-#ifdef CONFIG_OF
-static struct of_regulator_match reg_matches[] = {
-       { .name = "dcdc1" },
-       { .name = "dcdc2" },
-       { .name = "dcdc3" },
-       { .name = "ldo1"  },
-       { .name = "ldo2"  },
-       { .name = "ldo3"  },
-       { .name = "ldo4"  },
-};
-
-static struct tps65217_board *tps65217_parse_dt(struct platform_device *pdev)
-{
-       struct tps65217 *tps = dev_get_drvdata(pdev->dev.parent);
-       struct device_node *node = tps->dev->of_node;
-       struct tps65217_board *pdata;
-       struct device_node *regs;
-       int i, count;
-
-       regs = of_get_child_by_name(node, "regulators");
-       if (!regs)
-               return NULL;
-
-       count = of_regulator_match(&pdev->dev, regs, reg_matches,
-                                  TPS65217_NUM_REGULATOR);
-       of_node_put(regs);
-       if ((count < 0) || (count > TPS65217_NUM_REGULATOR))
-               return NULL;
-
-       pdata = devm_kzalloc(&pdev->dev, sizeof(*pdata), GFP_KERNEL);
-       if (!pdata)
-               return NULL;
-
-       for (i = 0; i < count; i++) {
-               if (!reg_matches[i].of_node)
-                       continue;
-
-               pdata->tps65217_init_data[i] = reg_matches[i].init_data;
-               pdata->of_node[i] = reg_matches[i].of_node;
-       }
-
-       return pdata;
-}
-#else
-static struct tps65217_board *tps65217_parse_dt(struct platform_device *pdev)
-{
-       return NULL;
-}
-#endif
-
 static int tps65217_regulator_probe(struct platform_device *pdev)
 {
        struct tps65217 *tps = dev_get_drvdata(pdev->dev.parent);
@@ -227,9 +183,6 @@ static int tps65217_regulator_probe(struct platform_device *pdev)
        struct regulator_config config = { };
        int i;
 
-       if (tps->dev->of_node)
-               pdata = tps65217_parse_dt(pdev);
-
        if (tps65217_chip_id(tps) != TPS65217) {
                dev_err(&pdev->dev, "Invalid tps chip version\n");
                return -ENODEV;
@@ -244,8 +197,6 @@ static int tps65217_regulator_probe(struct platform_device *pdev)
                        config.init_data = pdata->tps65217_init_data[i];
                config.driver_data = tps;
                config.regmap = tps->regmap;
-               if (tps->dev->of_node)
-                       config.of_node = pdata->of_node[i];
 
                rdev = devm_regulator_register(&pdev->dev, &regulators[i],
                                               &config);
@@ -255,6 +206,7 @@ static int tps65217_regulator_probe(struct platform_device *pdev)
                        return PTR_ERR(rdev);
                }
        }
+
        return 0;
 }