leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

regulator: core: simplify return value on suported_voltage

Browse Source 
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:
Jorge Ramirez-Ortiz 2019-04-22 19:28:24 +02:00 committed by Mark Brown
parent fd2f02f972
commit 4982094451
No known key found for this signature in database
GPG Key ID: 24D68B725D5487D0
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/regulator/core.c
View File

@ -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++) {