diff options
| author |   Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> | 2019-04-22 19:28:24 +0200 |
|---|---|---|
| committer |   Mark Brown <broonie@kernel.org> | 2019-05-03 15:34:26 +0900 |
| commit | 498209445124920b365ef43aac93d6f1acbaa1b7 (patch) | |
| tree | d7a5715e59fd8170dc4a3122f3938e165b3755b9 /drivers/regulator/core.c | |
| parent | regulator: da9xxx: Switch to SPDX identifier (diff) | |
| download | linux-dev-498209445124920b365ef43aac93d6f1acbaa1b7.tar.xz linux-dev-498209445124920b365ef43aac93d6f1acbaa1b7.zip | |
regulator: core: simplify return value on suported_voltage
All the current clients of this API assume that 0 corresponds
to a failure and non-zero to a pass therefore ignoring the need to
handle a negative error code.
This commit modifies the API to follow that standard since returning a
negative (EINVAL) doesn't seem to provide enough value to justify
the need to handle it.
Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator/core.c')
| -rw-r--r-- | drivers/regulator/core.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 08ccabe07fe3..af8b4dadb09b 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -3002,7 +3002,7 @@ EXPORT_SYMBOL_GPL(regulator_get_linear_step); * @min_uV: Minimum required voltage in uV. * @max_uV: Maximum required voltage in uV. * - * Returns a boolean or a negative error code. + * Returns a boolean. */ int regulator_is_supported_voltage(struct regulator *regulator, int min_uV, int max_uV) @@ -3026,7 +3026,7 @@ int regulator_is_supported_voltage(struct regulator *regulator, ret = regulator_count_voltages(regulator); if (ret < 0) - return ret; + return 0; voltages = ret; for (i = 0; i < voltages; i++) { |