regulator: tps65217: Use simplified DT parsing
Remove a bunch of code in favour of data. Signed-off-by: Mark Brown <broonie@linaro.org>
This commit is contained in:
parent
dc1e0b1db4
commit
7d42a7f293
|
@ -27,10 +27,13 @@
|
|||
#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, ®ulators[i],
|
||||
&config);
|
||||
|
@ -255,6 +206,7 @@ static int tps65217_regulator_probe(struct platform_device *pdev)
|
|||
return PTR_ERR(rdev);
|
||||
}
|
||||
}
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue