diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2015-01-28 14:15:31 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-01-28 14:43:43 -0500 |
commit | a412ec27957c9dc216b19d638c808d7ddfa2debd (patch) | |
tree | 55550b9e75530de0608f10a959767c508d9ee0b6 /drivers/regulator | |
parent | 97bf6af1f928216fd6c5a66e8a57bfa95a659672 (diff) |
regulator: qcom-rpm: signedness bug in probe()
"force_mode" is a u32 so it is never "< 0", but because of type
promotion then comparing "== -1" will do what we want.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator')
-rw-r--r-- | drivers/regulator/qcom_rpm-regulator.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/regulator/qcom_rpm-regulator.c b/drivers/regulator/qcom_rpm-regulator.c index 8364ff331a81..3ab65c62fdd5 100644 --- a/drivers/regulator/qcom_rpm-regulator.c +++ b/drivers/regulator/qcom_rpm-regulator.c | |||
@@ -768,7 +768,7 @@ static int rpm_reg_probe(struct platform_device *pdev) | |||
768 | break; | 768 | break; |
769 | } | 769 | } |
770 | 770 | ||
771 | if (force_mode < 0) { | 771 | if (force_mode == -1) { |
772 | dev_err(&pdev->dev, "invalid force mode\n"); | 772 | dev_err(&pdev->dev, "invalid force mode\n"); |
773 | return -EINVAL; | 773 | return -EINVAL; |
774 | } | 774 | } |