[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 11/18] hw/intc/sh_intc: Drop another useless macro
From: |
BALATON Zoltan |
Subject: |
[PATCH v3 11/18] hw/intc/sh_intc: Drop another useless macro |
Date: |
Wed, 27 Oct 2021 23:54:31 +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>
---
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
- Re: [PATCH v3 14/18] hw/intc/sh_intc: Use array index instead of pointer arithmetics, (continued)
- [PATCH v3 07/18] hw/char/sh_serial: Add device id to trace output, BALATON Zoltan, 2021/10/27
- [PATCH v3 13/18] hw/intc/sh_intc: Remove excessive parenthesis, BALATON Zoltan, 2021/10/27
- [PATCH v3 01/18] hw/sh4: Fix typos in a comment, BALATON Zoltan, 2021/10/27
- [PATCH v3 09/18] hw/intc/sh_intc: Turn some defines into an enum, BALATON Zoltan, 2021/10/27
- [PATCH v3 15/18] hw/sh4/sh_intc: Inline and drop sh_intc_source() function, BALATON Zoltan, 2021/10/27
- [PATCH v3 10/18] hw/intc/sh_intc: Rename iomem region, BALATON Zoltan, 2021/10/27
- [PATCH v3 11/18] hw/intc/sh_intc: Drop another useless macro,
BALATON Zoltan <=
- [PATCH v3 03/18] hw/sh4: Change debug printfs to traces, BALATON Zoltan, 2021/10/27
- [PATCH v3 08/18] hw/intc/sh_intc: Use existing macro instead of local one, BALATON Zoltan, 2021/10/27
- [PATCH v3 04/18] hw/sh4/r2d: Use error_report instead of fprintf to stderr, BALATON Zoltan, 2021/10/27
- [PATCH v3 12/18] hw/intc/sh_intc: Move sh_intc_register() closer to its only user, BALATON Zoltan, 2021/10/27
- [PATCH v3 16/18] hw/intc/sh_intc: Replace abort() with g_assert_not_reached(), BALATON Zoltan, 2021/10/27