mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-26 02:39:48 +00:00
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>
This commit is contained in:
parent
fd2f02f972
commit
4982094451
1 changed files with 2 additions and 2 deletions
|
@ -3002,7 +3002,7 @@ EXPORT_SYMBOL_GPL(regulator_get_linear_step);
|
||||||
* @min_uV: Minimum required voltage in uV.
|
* @min_uV: Minimum required voltage in uV.
|
||||||
* @max_uV: Maximum 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 regulator_is_supported_voltage(struct regulator *regulator,
|
||||||
int min_uV, int max_uV)
|
int min_uV, int max_uV)
|
||||||
|
@ -3026,7 +3026,7 @@ int regulator_is_supported_voltage(struct regulator *regulator,
|
||||||
|
|
||||||
ret = regulator_count_voltages(regulator);
|
ret = regulator_count_voltages(regulator);
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
return ret;
|
return 0;
|
||||||
voltages = ret;
|
voltages = ret;
|
||||||
|
|
||||||
for (i = 0; i < voltages; i++) {
|
for (i = 0; i < voltages; i++) {
|
||||||
|
|
Loading…
Reference in a new issue