2012-05-31 09:39:33

by Axel Lin

[permalink] [raw]
Subject: [PATCH 1/2] regulator: wm831x-ldo: Use regulator_map_voltage_linear for wm831x_alive_ldo_ops

wm831x_alive_ldo_ops uses simple linear voltage maps.
Thus use regulator_map_voltage_linear is more efficient than using the default
regulator_map_voltage_iterate.

Signed-off-by: Axel Lin <[email protected]>
---
drivers/regulator/wm831x-ldo.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/regulator/wm831x-ldo.c b/drivers/regulator/wm831x-ldo.c
index a9a28d8..f23d020e9 100644
--- a/drivers/regulator/wm831x-ldo.c
+++ b/drivers/regulator/wm831x-ldo.c
@@ -690,6 +690,7 @@ static int wm831x_alive_ldo_get_status(struct regulator_dev *rdev)

static struct regulator_ops wm831x_alive_ldo_ops = {
.list_voltage = regulator_list_voltage_linear,
+ .map_voltage = regulator_map_voltage_linear,
.get_voltage_sel = regulator_get_voltage_sel_regmap,
.set_voltage = wm831x_alive_ldo_set_voltage,
.set_suspend_voltage = wm831x_alive_ldo_set_suspend_voltage,
--
1.7.9.5



2012-05-31 09:40:29

by Axel Lin

[permalink] [raw]
Subject: [PATCH 2/2] regulator: wm8400: Use regulator_map_voltage_linear for wm8400_dcdc_ops

wm8400_dcdc_ops uses simple linear voltage maps.
Thus use regulator_map_voltage_linear is more efficient than using the default
regulator_map_voltage_iterate.

Signed-off-by: Axel Lin <[email protected]>
---
drivers/regulator/wm8400-regulator.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/regulator/wm8400-regulator.c b/drivers/regulator/wm8400-regulator.c
index 69a2b7c..f365795 100644
--- a/drivers/regulator/wm8400-regulator.c
+++ b/drivers/regulator/wm8400-regulator.c
@@ -152,6 +152,7 @@ static struct regulator_ops wm8400_dcdc_ops = {
.enable = regulator_enable_regmap,
.disable = regulator_disable_regmap,
.list_voltage = regulator_list_voltage_linear,
+ .map_voltage = regulator_map_voltage_linear,
.get_voltage_sel = regulator_get_voltage_sel_regmap,
.set_voltage_sel = regulator_set_voltage_sel_regmap,
.get_mode = wm8400_dcdc_get_mode,
--
1.7.9.5


2012-06-01 14:27:56

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 1/2] regulator: wm831x-ldo: Use regulator_map_voltage_linear for wm831x_alive_ldo_ops

On Thu, May 31, 2012 at 05:39:00PM +0800, Axel Lin wrote:
> wm831x_alive_ldo_ops uses simple linear voltage maps.
> Thus use regulator_map_voltage_linear is more efficient than using the default
> regulator_map_voltage_iterate.

Applied both, thanks.


Attachments:
(No filename) (251.00 B)
signature.asc (836.00 B)
Digital signature
Download all attachments