diff options
| author | Jingoo Han | 2014-01-08 02:02:16 +0100 |
|---|---|---|
| committer | Mark Brown | 2014-01-08 18:13:57 +0100 |
| commit | 0d25d09de114bffc984a03c417d4dddb53acd8d8 (patch) | |
| tree | 1cbb2d3932c43d14d043713f30064f7278e3293a /drivers/regulator/core.c | |
| parent | regulator: core: don't print an error when no regulator is found (diff) | |
| download | kernel-qcow2-linux-0d25d09de114bffc984a03c417d4dddb53acd8d8.tar.gz kernel-qcow2-linux-0d25d09de114bffc984a03c417d4dddb53acd8d8.tar.xz kernel-qcow2-linux-0d25d09de114bffc984a03c417d4dddb53acd8d8.zip | |
regulator: core: Fix checkpatch issue
Fix the following checkpatch errors and warnings.
ERROR: trailing whitespace
ERROR: return is not a function, parentheses are not required
WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
Diffstat (limited to 'drivers/regulator/core.c')
| -rw-r--r-- | drivers/regulator/core.c | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 9888f26aa131..b38a6b669e8c 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -1334,9 +1334,8 @@ static struct regulator *_regulator_get(struct device *dev, const char *id, * If we have return value from dev_lookup fail, we do not expect to * succeed, so, quit with appropriate error value */ - if (ret && ret != -ENODEV) { + if (ret && ret != -ENODEV) goto out; - } if (!devname) devname = "deviceless"; @@ -2245,7 +2244,7 @@ int regulator_is_supported_voltage(struct regulator *regulator, if (!(rdev->constraints->valid_ops_mask & REGULATOR_CHANGE_VOLTAGE)) { ret = regulator_get_voltage(regulator); if (ret >= 0) - return (min_uV <= ret && ret <= max_uV); + return min_uV <= ret && ret <= max_uV; else return ret; } @@ -2417,7 +2416,7 @@ int regulator_set_voltage(struct regulator *regulator, int min_uV, int max_uV) ret = regulator_check_voltage(rdev, &min_uV, &max_uV); if (ret < 0) goto out; - + /* restore original values in case of error */ old_min_uV = regulator->min_uV; old_max_uV = regulator->max_uV; @@ -2431,7 +2430,7 @@ int regulator_set_voltage(struct regulator *regulator, int min_uV, int max_uV) ret = _regulator_do_set_voltage(rdev, min_uV, max_uV); if (ret < 0) goto out2; - + out: mutex_unlock(&rdev->mutex); return ret; @@ -3836,9 +3835,8 @@ static int __init regulator_init_complete(void) * goes wrong. */ rdev_info(rdev, "disabling\n"); ret = ops->disable(rdev); - if (ret != 0) { + if (ret != 0) rdev_err(rdev, "couldn't disable: %d\n", ret); - } } else { /* The intention is that in future we will * assume that full constraints are provided |