Commit 5b5e977c authored by Axel Lin's avatar Axel Lin Committed by Mark Brown

regulator: Rename s5m8767_convert_voltage to s5m8767_convert_voltage_to_sel

This function finds the smallest voltage that falls within the specified range,
and then returns the selector. This rename makes the intention more clear.

Also remove unneeded local variables min_vol and max_vol in s5m8767_set_voltage
and s5m8767_set_voltage_buck.
Signed-off-by: default avatarAxel Lin <axel.lin@gmail.com>
Acked-by: default avatarSangbeom Kim <sbkim73@samsung.com>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 4d984d1c
...@@ -294,11 +294,11 @@ static int s5m8767_get_voltage_sel(struct regulator_dev *rdev) ...@@ -294,11 +294,11 @@ static int s5m8767_get_voltage_sel(struct regulator_dev *rdev)
return val; return val;
} }
static inline int s5m8767_convert_voltage( static int s5m8767_convert_voltage_to_sel(
const struct s5m_voltage_desc *desc, const struct s5m_voltage_desc *desc,
int min_vol, int max_vol) int min_vol, int max_vol)
{ {
int out_vol = 0; int selector = 0;
if (desc == NULL) if (desc == NULL)
return -EINVAL; return -EINVAL;
...@@ -306,19 +306,18 @@ static inline int s5m8767_convert_voltage( ...@@ -306,19 +306,18 @@ static inline int s5m8767_convert_voltage(
if (max_vol < desc->min || min_vol > desc->max) if (max_vol < desc->min || min_vol > desc->max)
return -EINVAL; return -EINVAL;
out_vol = (min_vol - desc->min) / desc->step; selector = (min_vol - desc->min) / desc->step;
if (desc->min + desc->step * out_vol > max_vol) if (desc->min + desc->step * selector > max_vol)
return -EINVAL; return -EINVAL;
return out_vol; return selector;
} }
static int s5m8767_set_voltage(struct regulator_dev *rdev, static int s5m8767_set_voltage(struct regulator_dev *rdev,
int min_uV, int max_uV, unsigned *selector) int min_uV, int max_uV, unsigned *selector)
{ {
struct s5m8767_info *s5m8767 = rdev_get_drvdata(rdev); struct s5m8767_info *s5m8767 = rdev_get_drvdata(rdev);
int min_vol = min_uV, max_vol = max_uV;
const struct s5m_voltage_desc *desc; const struct s5m_voltage_desc *desc;
int reg_id = rdev_get_id(rdev); int reg_id = rdev_get_id(rdev);
int reg, mask, ret; int reg, mask, ret;
...@@ -343,7 +342,7 @@ static int s5m8767_set_voltage(struct regulator_dev *rdev, ...@@ -343,7 +342,7 @@ static int s5m8767_set_voltage(struct regulator_dev *rdev,
desc = reg_voltage_map[reg_id]; desc = reg_voltage_map[reg_id];
i = s5m8767_convert_voltage(desc, min_vol, max_vol); i = s5m8767_convert_voltage_to_sel(desc, min_uV, max_uV);
if (i < 0) if (i < 0)
return i; return i;
...@@ -385,7 +384,6 @@ static int s5m8767_set_voltage_buck(struct regulator_dev *rdev, ...@@ -385,7 +384,6 @@ static int s5m8767_set_voltage_buck(struct regulator_dev *rdev,
int reg_id = rdev_get_id(rdev); int reg_id = rdev_get_id(rdev);
const struct s5m_voltage_desc *desc; const struct s5m_voltage_desc *desc;
int new_val, old_val, i = 0; int new_val, old_val, i = 0;
int min_vol = min_uV, max_vol = max_uV;
if (reg_id < S5M8767_BUCK1 || reg_id > S5M8767_BUCK6) if (reg_id < S5M8767_BUCK1 || reg_id > S5M8767_BUCK6)
return -EINVAL; return -EINVAL;
...@@ -402,7 +400,7 @@ static int s5m8767_set_voltage_buck(struct regulator_dev *rdev, ...@@ -402,7 +400,7 @@ static int s5m8767_set_voltage_buck(struct regulator_dev *rdev,
} }
desc = reg_voltage_map[reg_id]; desc = reg_voltage_map[reg_id];
new_val = s5m8767_convert_voltage(desc, min_vol, max_vol); new_val = s5m8767_convert_voltage_to_sel(desc, min_uV, max_uV);
if (new_val < 0) if (new_val < 0)
return new_val; return new_val;
...@@ -580,7 +578,7 @@ static __devinit int s5m8767_pmic_probe(struct platform_device *pdev) ...@@ -580,7 +578,7 @@ static __devinit int s5m8767_pmic_probe(struct platform_device *pdev)
for (i = 0; i < 8; i++) { for (i = 0; i < 8; i++) {
if (s5m8767->buck2_gpiodvs) { if (s5m8767->buck2_gpiodvs) {
s5m8767->buck2_vol[i] = s5m8767->buck2_vol[i] =
s5m8767_convert_voltage( s5m8767_convert_voltage_to_sel(
&buck_voltage_val2, &buck_voltage_val2,
pdata->buck2_voltage[i], pdata->buck2_voltage[i],
pdata->buck2_voltage[i] + pdata->buck2_voltage[i] +
...@@ -589,7 +587,7 @@ static __devinit int s5m8767_pmic_probe(struct platform_device *pdev) ...@@ -589,7 +587,7 @@ static __devinit int s5m8767_pmic_probe(struct platform_device *pdev)
if (s5m8767->buck3_gpiodvs) { if (s5m8767->buck3_gpiodvs) {
s5m8767->buck3_vol[i] = s5m8767->buck3_vol[i] =
s5m8767_convert_voltage( s5m8767_convert_voltage_to_sel(
&buck_voltage_val2, &buck_voltage_val2,
pdata->buck3_voltage[i], pdata->buck3_voltage[i],
pdata->buck3_voltage[i] + pdata->buck3_voltage[i] +
...@@ -598,7 +596,7 @@ static __devinit int s5m8767_pmic_probe(struct platform_device *pdev) ...@@ -598,7 +596,7 @@ static __devinit int s5m8767_pmic_probe(struct platform_device *pdev)
if (s5m8767->buck4_gpiodvs) { if (s5m8767->buck4_gpiodvs) {
s5m8767->buck4_vol[i] = s5m8767->buck4_vol[i] =
s5m8767_convert_voltage( s5m8767_convert_voltage_to_sel(
&buck_voltage_val2, &buck_voltage_val2,
pdata->buck4_voltage[i], pdata->buck4_voltage[i],
pdata->buck4_voltage[i] + pdata->buck4_voltage[i] +
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment