summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2015-10-19 12:18:05 +0200
committerLee Jones <lee.jones@linaro.org>2015-10-30 19:56:58 +0100
commita05950a4453fcfb136ee1eff0bd0a3cfb46a401e (patch)
treeb7b7f899b30570ea8b49ff831d0638c0334c75d1
parentmfd: axp20x: Add a cell for the power button part of the, axp288 PMICs (diff)
downloadlinux-a05950a4453fcfb136ee1eff0bd0a3cfb46a401e.tar.xz
linux-a05950a4453fcfb136ee1eff0bd0a3cfb46a401e.zip
mfd: arizona: Signedness bug in arizona_runtime_suspend()
The "jd_active" variable needs to be signed for the error handling to work. Fixes: 143e5887ae57 ('mfd: arizona: factor out checking of jack detection state') Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Acked-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com> Signed-off-by: Lee Jones <lee.jones@linaro.org>
-rw-r--r--drivers/mfd/arizona-core.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c
index 4e49210f7a5b..d474732cc65c 100644
--- a/drivers/mfd/arizona-core.c
+++ b/drivers/mfd/arizona-core.c
@@ -628,7 +628,7 @@ err:
static int arizona_runtime_suspend(struct device *dev)
{
struct arizona *arizona = dev_get_drvdata(dev);
- unsigned int jd_active = 0;
+ int jd_active = 0;
int ret;
dev_dbg(arizona->dev, "Entering AoD mode\n");