diff options
| author | Axel Lin <axel.lin@ingics.com> | 2016-06-15 10:21:34 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-08-16 09:33:17 +0200 |
| commit | 141c47d95f6d09478fe53bcbb54c3155dc6de888 (patch) | |
| tree | 3c81b2283566da30fcc9b185712440f030ccbee5 | |
| parent | 4e495a5e3aac49954bdfea0013805a8c342bab85 (diff) | |
| download | lwn-141c47d95f6d09478fe53bcbb54c3155dc6de888.tar.gz lwn-141c47d95f6d09478fe53bcbb54c3155dc6de888.zip | |
regulator: qcom_smd: Remove list_voltage callback for rpm_smps_ldo_ops_fixed
commit 43160ffd12c8d1d331362362eea3c70e04b6f9c4 upstream.
Use regulator_list_voltage_linear_range in rpm_smps_ldo_ops_fixed is
wrong because it is used for fixed regulator without any linear range.
The rpm_smps_ldo_ops_fixed is used for pm8941_lnldo which has fixed_uV
set and n_voltages = 1. In this case, regulator_list_voltage() can return
rdev->desc->fixed_uV without .list_voltage implementation.
Fixes: 3bfbb4d1a480 ("regulator: qcom_smd: add list_voltage callback")
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/regulator/qcom_smd-regulator.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/regulator/qcom_smd-regulator.c b/drivers/regulator/qcom_smd-regulator.c index 6c7fe4778793..7e156a0b65dd 100644 --- a/drivers/regulator/qcom_smd-regulator.c +++ b/drivers/regulator/qcom_smd-regulator.c @@ -140,7 +140,6 @@ static const struct regulator_ops rpm_smps_ldo_ops = { .enable = rpm_reg_enable, .disable = rpm_reg_disable, .is_enabled = rpm_reg_is_enabled, - .list_voltage = regulator_list_voltage_linear_range, .get_voltage = rpm_reg_get_voltage, .set_voltage = rpm_reg_set_voltage, |