diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-03-11 11:11:39 +0300 |
|---|---|---|
| committer | Lee Jones <lee.jones@linaro.org> | 2016-04-11 13:31:40 +0100 |
| commit | 22aab38e7b59fd79ce1045006be69a9abab58e5a (patch) | |
| tree | 8102ecb8201621f05eff3b043de7e6fa095506c1 /drivers/mfd/lp8788-irq.c | |
| parent | adae28c59a6a71a971ffed713550911546df0e20 (diff) | |
mfd: lp8788-irq: Uninitialized variable in irq handler
Instead to being true/false, the "handled" is true/uninitialized.
Presumably this doesn't cause that many problems in real life because
normally we handle the IRQ.
Fixes: eea6b7cc53aa ('mfd: Add lp8788 mfd driver')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Milo Kim <milo.kim@ti.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/mfd/lp8788-irq.c')
| -rw-r--r-- | drivers/mfd/lp8788-irq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mfd/lp8788-irq.c b/drivers/mfd/lp8788-irq.c index c7a9825aa4ce..792d51bae20f 100644 --- a/drivers/mfd/lp8788-irq.c +++ b/drivers/mfd/lp8788-irq.c @@ -112,7 +112,7 @@ static irqreturn_t lp8788_irq_handler(int irq, void *ptr) struct lp8788_irq_data *irqd = ptr; struct lp8788 *lp = irqd->lp; u8 status[NUM_REGS], addr, mask; - bool handled; + bool handled = false; int i; if (lp8788_read_multi_bytes(lp, LP8788_INT_1, status, NUM_REGS)) |