diff options
author | Dan Carpenter <[email protected]> | 2018-10-18 17:15:22 +0300 |
---|---|---|
committer | Borislav Petkov <[email protected]> | 2018-11-06 12:03:16 +0100 |
commit | 8fd8cbfeada59d381c49ad700607376416e2fdd8 (patch) | |
tree | 65f57c0d7bd692842defeec52c58d44c0f40b836 | |
parent | 1a81361f75d8a5cbe9d77c30fe0f42b7cda621c8 (diff) |
EDAC, qcom_edac: Remove irq_handled local variable
irq_handled isn't initialized to false on function entry. However, it is
not really needed and the IRQ handler return value can be set directly
instead.
[ bp: rewrite commit message. ]
Fixes: 27450653f1db ("drivers: edac: Add EDAC driver support for QCOM SoCs")
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Borislav Petkov <[email protected]>
CC: Channagoud Kadabi <[email protected]>
CC: Mauro Carvalho Chehab <[email protected]>
CC: Venkata Narendra Kumar Gutta <[email protected]>
CC: [email protected]
CC: [email protected]
CC: linux-edac <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
-rw-r--r-- | drivers/edac/qcom_edac.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/edac/qcom_edac.c b/drivers/edac/qcom_edac.c index 82bd775124f2..97a27e42dd61 100644 --- a/drivers/edac/qcom_edac.c +++ b/drivers/edac/qcom_edac.c @@ -292,7 +292,6 @@ llcc_ecc_irq_handler(int irq, void *edev_ctl) struct llcc_drv_data *drv = edac_dev_ctl->pvt_info; irqreturn_t irq_rc = IRQ_NONE; u32 drp_error, trp_error, i; - bool irq_handled; int ret; /* Iterate over the banks and look for Tag RAM or Data RAM errors */ @@ -311,7 +310,7 @@ llcc_ecc_irq_handler(int irq, void *edev_ctl) ret = dump_syn_reg(edev_ctl, LLCC_DRAM_UE, i); } if (!ret) - irq_handled = true; + irq_rc = IRQ_HANDLED; ret = regmap_read(drv->regmap, drv->offsets[i] + TRP_INTERRUPT_0_STATUS, @@ -327,12 +326,9 @@ llcc_ecc_irq_handler(int irq, void *edev_ctl) ret = dump_syn_reg(edev_ctl, LLCC_TRAM_UE, i); } if (!ret) - irq_handled = true; + irq_rc = IRQ_HANDLED; } - if (irq_handled) - irq_rc = IRQ_HANDLED; - return irq_rc; } |