diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-03-11 03:11:39 -0500 |
---|---|---|
committer | Lee Jones <lee.jones@linaro.org> | 2016-04-11 08:31:40 -0400 |
commit | 22aab38e7b59fd79ce1045006be69a9abab58e5a (patch) | |
tree | 8102ecb8201621f05eff3b043de7e6fa095506c1 /drivers/mfd | |
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')
-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) | |||
112 | struct lp8788_irq_data *irqd = ptr; | 112 | struct lp8788_irq_data *irqd = ptr; |
113 | struct lp8788 *lp = irqd->lp; | 113 | struct lp8788 *lp = irqd->lp; |
114 | u8 status[NUM_REGS], addr, mask; | 114 | u8 status[NUM_REGS], addr, mask; |
115 | bool handled; | 115 | bool handled = false; |
116 | int i; | 116 | int i; |
117 | 117 | ||
118 | if (lp8788_read_multi_bytes(lp, LP8788_INT_1, status, NUM_REGS)) | 118 | if (lp8788_read_multi_bytes(lp, LP8788_INT_1, status, NUM_REGS)) |