regulator: ab8500: Use regulator_list_voltage_linear for fixed voltage
authorAxel Lin <axel.lin@ingics.com>
Mon, 25 Mar 2013 06:59:00 +0000 (14:59 +0800)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Mon, 25 Mar 2013 10:19:52 +0000 (10:19 +0000)
Both ab8500_regulator_mode_ops and ab8500_regulator_ops do not have volt_table
setting, thus we can not use regulator_list_voltage_table for them.
However, they have min_uV setting with n_voltages = 1, so use
regulator_list_voltage_linear instead.

Also remove setting set_voltage_time_sel for ab8500_regulator_mode_ops.
It is used only for fixed voltage, so it does not need to implement
set_voltage_time_sel.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
drivers/regulator/ab8500.c

index a2d19c650f2470dc7dfaedb67b35c881d187e43c..12e274039ae1d8c2d18ffc2725ca3ad10cc0d506 100644 (file)
@@ -363,8 +363,7 @@ static struct regulator_ops ab8500_regulator_mode_ops = {
        .set_mode               = ab8500_regulator_set_mode,
        .get_mode               = ab8500_regulator_get_mode,
        .get_voltage_sel        = ab8500_regulator_get_voltage_sel,
-       .list_voltage           = regulator_list_voltage_table,
-       .set_voltage_time_sel   = ab8500_regulator_set_voltage_time_sel,
+       .list_voltage           = regulator_list_voltage_linear,
 };
 
 static struct regulator_ops ab8500_regulator_ops = {
@@ -372,7 +371,7 @@ static struct regulator_ops ab8500_regulator_ops = {
        .disable                = ab8500_regulator_disable,
        .is_enabled             = ab8500_regulator_is_enabled,
        .get_voltage_sel        = ab8500_regulator_get_voltage_sel,
-       .list_voltage           = regulator_list_voltage_table,
+       .list_voltage           = regulator_list_voltage_linear,
 };
 
 static struct ab8500_regulator_info