regulator: tps65910: Convert to regulator_set_voltage_time_sel()
Subsystems (2)
PATCH 1/1 regulator: tps65910: Convert to regulator_set_voltage_time_sel()
Signed-off-by Axel Lin <[email protected]>
--- drivers/regulator/tps65910-regulator.c | 34 ++++++-------------------------- 1 file changed, 6 insertions(+), 28 deletions(-)
drivers/regulator/tps65910-regulator.c +6 -28
@@ -824,31 +824,6 @@ static int tps65911_list_voltage(struct regulator_dev *dev, unsigned selector)return (LDO_MIN_VOLT + selector * step_mv) *1000 ; } -static int tps65910_set_voltage_dcdc_time_sel (struct regulator_dev *dev, -unsigned int old_selector,unsigned int new_selector) -{ -int id =rdev_get_id (dev); -int old_volt, new_volt; - - old_volt =tps65910_list_voltage_dcdc (dev, old_selector); -if (old_volt <0 ) -return old_volt; - - new_volt =tps65910_list_voltage_dcdc (dev, new_selector); -if (new_volt <0 ) -return new_volt; - -/* VDD1 and VDD2 are 12.5mV/us, VDDCTRL is 100mV/20us */ -switch (id) { -case TPS65910_REG_VDD1 : -case TPS65910_REG_VDD2 : -return DIV_ROUND_UP (abs (old_volt - new_volt),12500 ); -case TPS65911_REG_VDDCTRL : -return DIV_ROUND_UP (abs (old_volt - new_volt),5000 ); - } -return -EINVAL ; -} -/* Regulator ops (except VRTC) */ static struct regulator_ops tps65910_ops_dcdc = { .is_enabled = regulator_is_enabled_regmap,
@@ -859,7 +834,7 @@ static struct regulator_ops tps65910_ops_dcdc = { .get_mode = tps65910_get_mode, .get_voltage_sel = tps65910_get_voltage_dcdc_sel, .set_voltage_sel = tps65910_set_voltage_dcdc_sel, - .set_voltage_time_sel = tps65910_set_voltage_dcdc_time_sel, + .set_voltage_time_sel = regulator_set_voltage_time_sel, .list_voltage = tps65910_list_voltage_dcdc, };
@@ -1236,11 +1211,14 @@ static __devinit int tps65910_probe(struct platform_device *pdev) pmic->desc[i].ops = &tps65910_ops_dcdc; pmic->desc[i].n_voltages =VDD1_2_NUM_VOLT_FINE *VDD1_2_NUM_VOLT_COARSE ; + pmic->desc[i].ramp_delay =12500 ; }else if (i ==TPS65910_REG_VDD3 ) { -if (tps65910_chip_id (tps65910) ==TPS65910 ) +if (tps65910_chip_id (tps65910) ==TPS65910 ) { pmic->desc[i].ops = &tps65910_ops_vdd3; -else + }else { pmic->desc[i].ops = &tps65910_ops_dcdc; + pmic->desc[i].ramp_delay =5000 ; + } }else {if (tps65910_chip_id (tps65910) ==TPS65910 ) pmic->desc[i].ops = &tps65910_ops;
Signature · 1 line
1.7.9.5
Quoted 2 lines · Axel Lin
Signed-off-by Axel Lin<[email protected]>---
Acked-by Laxman Dewangan <[email protected]>
Looks good to me.
Mark Brown
AppliedQuoted 1 line · Axel Lin
Signed-off-by Axel Lin <[email protected]>
Applied, thanks.