Commit 20a14b84 authored by Axel Lin's avatar Axel Lin Committed by Mark Brown

regulator: Kill s5m8767_get_reg_id function

Calling s5m8767_get_reg_id() is exactly the same as calling rdev_get_id().
It is pointless to add s5m8767_get_reg_id() function.
Use rdev_get_id() directly and remove s5m8767_get_reg_id() function.
Signed-off-by: default avatarAxel Lin <axel.lin@gmail.com>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 34ce8d07
...@@ -120,16 +120,11 @@ static const struct s5m_voltage_desc *reg_voltage_map[] = { ...@@ -120,16 +120,11 @@ static const struct s5m_voltage_desc *reg_voltage_map[] = {
[S5M8767_BUCK9] = &buck_voltage_val3, [S5M8767_BUCK9] = &buck_voltage_val3,
}; };
static inline int s5m8767_get_reg_id(struct regulator_dev *rdev)
{
return rdev_get_id(rdev);
}
static int s5m8767_list_voltage(struct regulator_dev *rdev, static int s5m8767_list_voltage(struct regulator_dev *rdev,
unsigned int selector) unsigned int selector)
{ {
const struct s5m_voltage_desc *desc; const struct s5m_voltage_desc *desc;
int reg_id = s5m8767_get_reg_id(rdev); int reg_id = rdev_get_id(rdev);
int val; int val;
if (reg_id >= ARRAY_SIZE(reg_voltage_map) || reg_id < 0) if (reg_id >= ARRAY_SIZE(reg_voltage_map) || reg_id < 0)
...@@ -148,7 +143,7 @@ static int s5m8767_list_voltage(struct regulator_dev *rdev, ...@@ -148,7 +143,7 @@ static int s5m8767_list_voltage(struct regulator_dev *rdev,
static int s5m8767_get_register(struct regulator_dev *rdev, int *reg) static int s5m8767_get_register(struct regulator_dev *rdev, int *reg)
{ {
int reg_id = s5m8767_get_reg_id(rdev); int reg_id = rdev_get_id(rdev);
switch (reg_id) { switch (reg_id) {
case S5M8767_LDO1 ... S5M8767_LDO2: case S5M8767_LDO1 ... S5M8767_LDO2:
...@@ -224,7 +219,7 @@ static int s5m8767_reg_disable(struct regulator_dev *rdev) ...@@ -224,7 +219,7 @@ static int s5m8767_reg_disable(struct regulator_dev *rdev)
static int s5m8767_get_voltage_register(struct regulator_dev *rdev, int *_reg) static int s5m8767_get_voltage_register(struct regulator_dev *rdev, int *_reg)
{ {
int reg_id = s5m8767_get_reg_id(rdev); int reg_id = rdev_get_id(rdev);
int reg; int reg;
switch (reg_id) { switch (reg_id) {
...@@ -265,7 +260,7 @@ static int s5m8767_get_voltage_sel(struct regulator_dev *rdev) ...@@ -265,7 +260,7 @@ static int s5m8767_get_voltage_sel(struct regulator_dev *rdev)
{ {
struct s5m8767_info *s5m8767 = rdev_get_drvdata(rdev); struct s5m8767_info *s5m8767 = rdev_get_drvdata(rdev);
int reg, mask = 0xff, ret; int reg, mask = 0xff, ret;
int reg_id = s5m8767_get_reg_id(rdev); int reg_id = rdev_get_id(rdev);
u8 val; u8 val;
ret = s5m8767_get_voltage_register(rdev, &reg); ret = s5m8767_get_voltage_register(rdev, &reg);
...@@ -325,7 +320,7 @@ static int s5m8767_set_voltage(struct regulator_dev *rdev, ...@@ -325,7 +320,7 @@ static int s5m8767_set_voltage(struct regulator_dev *rdev,
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; int min_vol = min_uV, max_vol = max_uV;
const struct s5m_voltage_desc *desc; const struct s5m_voltage_desc *desc;
int reg_id = s5m8767_get_reg_id(rdev); int reg_id = rdev_get_id(rdev);
int reg, mask, ret; int reg, mask, ret;
int i; int i;
u8 val; u8 val;
...@@ -387,7 +382,7 @@ static int s5m8767_set_voltage_buck(struct regulator_dev *rdev, ...@@ -387,7 +382,7 @@ static int s5m8767_set_voltage_buck(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 reg_id = s5m8767_get_reg_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; int min_vol = min_uV, max_vol = max_uV;
...@@ -456,7 +451,7 @@ static int s5m8767_set_voltage_time_sel(struct regulator_dev *rdev, ...@@ -456,7 +451,7 @@ static int s5m8767_set_voltage_time_sel(struct regulator_dev *rdev,
{ {
struct s5m8767_info *s5m8767 = rdev_get_drvdata(rdev); struct s5m8767_info *s5m8767 = rdev_get_drvdata(rdev);
const struct s5m_voltage_desc *desc; const struct s5m_voltage_desc *desc;
int reg_id = s5m8767_get_reg_id(rdev); int reg_id = rdev_get_id(rdev);
int mask; int mask;
int new_val, old_val; int new_val, old_val;
......
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