Skip to content

Commit

Permalink
mfd: arizona: Signedness bug in arizona_runtime_suspend()
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Dan Carpenter authored and Lee Jones committed Oct 30, 2015
1 parent e56e5ad commit a05950a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/mfd/arizona-core.c
Original file line number Diff line number Diff line change
Expand Up @@ -628,7 +628,7 @@ static int arizona_runtime_resume(struct device *dev)
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");
Expand Down

0 comments on commit a05950a

Please sign in to comment.