aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhilippe Mathieu-Daudé <philmd@linaro.org>2023-09-04 18:28:23 +0200
committerMarkus Armbruster <armbru@redhat.com>2023-09-29 10:07:17 +0200
commit1cc0c5dd38884a7c54bc806bb0ae182db275faf0 (patch)
tree060b9a3b855fa345c9a00786caaf5ea18df48837
parent5e0528a72570e95945eac841b0871f0cbba777b8 (diff)
downloadqemu-1cc0c5dd38884a7c54bc806bb0ae182db275faf0.zip
qemu-1cc0c5dd38884a7c54bc806bb0ae182db275faf0.tar.gz
qemu-1cc0c5dd38884a7c54bc806bb0ae182db275faf0.tar.bz2
hw/intc/openpic: Clean up local variable shadowing
Fix: hw/intc/openpic.c: In function ‘openpic_gbl_write’: hw/intc/openpic.c:614:17: warning: declaration of ‘idx’ shadows a previous local [-Wshadow=compatible-local] 614 | int idx; | ^~~ hw/intc/openpic.c:568:9: note: shadowed declaration is here 568 | int idx; | ^~~ Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-ID: <20230904162824.85385-3-philmd@linaro.org> Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Markus Armbruster <armbru@redhat.com>
-rw-r--r--hw/intc/openpic.c7
1 files changed, 2 insertions, 5 deletions
diff --git a/hw/intc/openpic.c b/hw/intc/openpic.c
index c757adb..a6f91d4 100644
--- a/hw/intc/openpic.c
+++ b/hw/intc/openpic.c
@@ -610,11 +610,8 @@ static void openpic_gbl_write(void *opaque, hwaddr addr, uint64_t val,
case 0x10B0:
case 0x10C0:
case 0x10D0:
- {
- int idx;
- idx = (addr - 0x10A0) >> 4;
- write_IRQreg_ivpr(opp, opp->irq_ipi0 + idx, val);
- }
+ idx = (addr - 0x10A0) >> 4;
+ write_IRQreg_ivpr(opp, opp->irq_ipi0 + idx, val);
break;
case 0x10E0: /* SPVE */
opp->spve = val & opp->vector_mask;