[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 11/23] hw/intc/sh_intc: Drop another useless macro
From: |
BALATON Zoltan |
Subject: |
[PATCH v4 11/23] hw/intc/sh_intc: Drop another useless macro |
Date: |
Thu, 28 Oct 2021 21:27:05 +0200 |
The INT_REG_PARAMS macro was only used a few times within one function
on adjacent lines and is actually more complex than writing out the
parameters so simplify it by expanding the macro at call sites and
dropping the #define.
Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
---
hw/intc/sh_intc.c | 15 ++++-----------
1 file changed, 4 insertions(+), 11 deletions(-)
diff --git a/hw/intc/sh_intc.c b/hw/intc/sh_intc.c
index e386372b6f..763ebbfec2 100644
--- a/hw/intc/sh_intc.c
+++ b/hw/intc/sh_intc.c
@@ -433,16 +433,12 @@ int sh_intc_init(MemoryRegion *sysmem,
memory_region_init_io(&desc->iomem, NULL, &sh_intc_ops, desc, "intc",
0x100000000ULL);
-#define INT_REG_PARAMS(reg_struct, type, action, j) \
- reg_struct->action##_reg, #type, #action, j
if (desc->mask_regs) {
for (i = 0; i < desc->nr_mask_regs; i++) {
struct intc_mask_reg *mr = desc->mask_regs + i;
- j += sh_intc_register(sysmem, desc,
- INT_REG_PARAMS(mr, mask, set, j));
- j += sh_intc_register(sysmem, desc,
- INT_REG_PARAMS(mr, mask, clr, j));
+ j += sh_intc_register(sysmem, desc, mr->set_reg, "mask", "set", j);
+ j += sh_intc_register(sysmem, desc, mr->clr_reg, "mask", "clr", j);
}
}
@@ -450,13 +446,10 @@ int sh_intc_init(MemoryRegion *sysmem,
for (i = 0; i < desc->nr_prio_regs; i++) {
struct intc_prio_reg *pr = desc->prio_regs + i;
- j += sh_intc_register(sysmem, desc,
- INT_REG_PARAMS(pr, prio, set, j));
- j += sh_intc_register(sysmem, desc,
- INT_REG_PARAMS(pr, prio, clr, j));
+ j += sh_intc_register(sysmem, desc, pr->set_reg, "prio", "set", j);
+ j += sh_intc_register(sysmem, desc, pr->clr_reg, "prio", "clr", j);
}
}
-#undef INT_REG_PARAMS
return 0;
}
--
2.21.4
- [PATCH v4 15/23] hw/intc/sh_intc: Inline and drop sh_intc_source() function, (continued)
- [PATCH v4 15/23] hw/intc/sh_intc: Inline and drop sh_intc_source() function, BALATON Zoltan, 2021/10/28
- [PATCH v4 12/23] hw/intc/sh_intc: Move sh_intc_register() closer to its only user, BALATON Zoltan, 2021/10/28
- [PATCH v4 14/23] hw/intc/sh_intc: Use array index instead of pointer arithmetics, BALATON Zoltan, 2021/10/28
- [PATCH v4 19/23] hw/intc/sh_intc: Remove unneeded local variable initialisers, BALATON Zoltan, 2021/10/28
- [PATCH v4 09/23] hw/intc/sh_intc: Turn some defines into an enum, BALATON Zoltan, 2021/10/28
- [PATCH v4 06/23] hw/char/sh_serial: QOM-ify, BALATON Zoltan, 2021/10/28
- [PATCH v4 13/23] hw/intc/sh_intc: Remove excessive parenthesis, BALATON Zoltan, 2021/10/28
- [PATCH v4 11/23] hw/intc/sh_intc: Drop another useless macro,
BALATON Zoltan <=
- [PATCH v4 23/23] hw/timer/sh_timer: Fix timer memory region size, BALATON Zoltan, 2021/10/28
- [PATCH v4 20/23] hw/timer/sh_timer: Rename sh_timer_state to SHTimerState, BALATON Zoltan, 2021/10/28
- [PATCH v4 16/23] hw/intc/sh_intc: Replace abort() with g_assert_not_reached(), BALATON Zoltan, 2021/10/28
- [PATCH v4 21/23] hw/timer/sh_timer: Fix format strings and remove casts, BALATON Zoltan, 2021/10/28
- [PATCH v4 22/23] hw/timer/sh_timer: Do not wrap lines that are not too long, BALATON Zoltan, 2021/10/28
- [PATCH v4 17/23] hw/intc/sh_intc: Avoid using continue in loops, BALATON Zoltan, 2021/10/28
- [PATCH v4 18/23] hw/intc/sh_intc: Simplify allocating sources array, BALATON Zoltan, 2021/10/28