Skip to content

Commit 6f6006a

Browse files
committed
hw/intc/loongarch_extioi: Code cleanup about loongarch_extioi
Remove definition about LoongArchExtIOI and LOONGARCH_EXTIOI, and replace them with LoongArchExtIOICommonState and macro LOONGARCH_EXTIOI_COMMON separately. Also remove unnecessary header files. Signed-off-by: Bibo Mao <[email protected]> Reviewed-by: Song Gao <[email protected]>
1 parent ff09444 commit 6f6006a

File tree

2 files changed

+14
-19
lines changed

2 files changed

+14
-19
lines changed

hw/intc/loongarch_extioi.c

Lines changed: 14 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -10,16 +10,13 @@
1010
#include "qemu/log.h"
1111
#include "qapi/error.h"
1212
#include "hw/irq.h"
13-
#include "hw/sysbus.h"
1413
#include "hw/loongarch/virt.h"
15-
#include "hw/qdev-properties.h"
1614
#include "exec/address-spaces.h"
1715
#include "hw/intc/loongarch_extioi.h"
18-
#include "migration/vmstate.h"
1916
#include "trace.h"
2017

2118

22-
static void extioi_update_irq(LoongArchExtIOI *s, int irq, int level)
19+
static void extioi_update_irq(LoongArchExtIOICommonState *s, int irq, int level)
2320
{
2421
int ipnum, cpu, found, irq_index, irq_mask;
2522

@@ -54,7 +51,7 @@ static void extioi_update_irq(LoongArchExtIOI *s, int irq, int level)
5451

5552
static void extioi_setirq(void *opaque, int irq, int level)
5653
{
57-
LoongArchExtIOI *s = LOONGARCH_EXTIOI(opaque);
54+
LoongArchExtIOICommonState *s = LOONGARCH_EXTIOI_COMMON(opaque);
5855
trace_loongarch_extioi_setirq(irq, level);
5956
if (level) {
6057
set_bit32(irq, s->isr);
@@ -67,7 +64,7 @@ static void extioi_setirq(void *opaque, int irq, int level)
6764
static MemTxResult extioi_readw(void *opaque, hwaddr addr, uint64_t *data,
6865
unsigned size, MemTxAttrs attrs)
6966
{
70-
LoongArchExtIOI *s = LOONGARCH_EXTIOI(opaque);
67+
LoongArchExtIOICommonState *s = LOONGARCH_EXTIOI_COMMON(opaque);
7168
unsigned long offset = addr & 0xffff;
7269
uint32_t index, cpu;
7370

@@ -106,7 +103,7 @@ static MemTxResult extioi_readw(void *opaque, hwaddr addr, uint64_t *data,
106103
return MEMTX_OK;
107104
}
108105

109-
static inline void extioi_enable_irq(LoongArchExtIOI *s, int index,\
106+
static inline void extioi_enable_irq(LoongArchExtIOICommonState *s, int index,\
110107
uint32_t mask, int level)
111108
{
112109
uint32_t val;
@@ -125,8 +122,8 @@ static inline void extioi_enable_irq(LoongArchExtIOI *s, int index,\
125122
}
126123
}
127124

128-
static inline void extioi_update_sw_coremap(LoongArchExtIOI *s, int irq,
129-
uint64_t val, bool notify)
125+
static inline void extioi_update_sw_coremap(LoongArchExtIOICommonState *s,
126+
int irq, uint64_t val, bool notify)
130127
{
131128
int i, cpu;
132129

@@ -162,8 +159,8 @@ static inline void extioi_update_sw_coremap(LoongArchExtIOI *s, int irq,
162159
}
163160
}
164161

165-
static inline void extioi_update_sw_ipmap(LoongArchExtIOI *s, int index,
166-
uint64_t val)
162+
static inline void extioi_update_sw_ipmap(LoongArchExtIOICommonState *s,
163+
int index, uint64_t val)
167164
{
168165
int i;
169166
uint8_t ipnum;
@@ -186,7 +183,7 @@ static MemTxResult extioi_writew(void *opaque, hwaddr addr,
186183
uint64_t val, unsigned size,
187184
MemTxAttrs attrs)
188185
{
189-
LoongArchExtIOI *s = LOONGARCH_EXTIOI(opaque);
186+
LoongArchExtIOICommonState *s = LOONGARCH_EXTIOI_COMMON(opaque);
190187
int cpu, index, old_data, irq;
191188
uint32_t offset;
192189

@@ -266,7 +263,7 @@ static const MemoryRegionOps extioi_ops = {
266263
static MemTxResult extioi_virt_readw(void *opaque, hwaddr addr, uint64_t *data,
267264
unsigned size, MemTxAttrs attrs)
268265
{
269-
LoongArchExtIOI *s = LOONGARCH_EXTIOI(opaque);
266+
LoongArchExtIOICommonState *s = LOONGARCH_EXTIOI_COMMON(opaque);
270267

271268
switch (addr) {
272269
case EXTIOI_VIRT_FEATURES:
@@ -286,7 +283,7 @@ static MemTxResult extioi_virt_writew(void *opaque, hwaddr addr,
286283
uint64_t val, unsigned size,
287284
MemTxAttrs attrs)
288285
{
289-
LoongArchExtIOI *s = LOONGARCH_EXTIOI(opaque);
286+
LoongArchExtIOICommonState *s = LOONGARCH_EXTIOI_COMMON(opaque);
290287

291288
switch (addr) {
292289
case EXTIOI_VIRT_FEATURES:
@@ -365,21 +362,21 @@ static void loongarch_extioi_realize(DeviceState *dev, Error **errp)
365362

366363
static void loongarch_extioi_unrealize(DeviceState *dev)
367364
{
368-
LoongArchExtIOICommonState *s = LOONGARCH_EXTIOI(dev);
365+
LoongArchExtIOICommonState *s = LOONGARCH_EXTIOI_COMMON(dev);
369366

370367
g_free(s->cpu);
371368
}
372369

373370
static void loongarch_extioi_reset(DeviceState *d)
374371
{
375-
LoongArchExtIOI *s = LOONGARCH_EXTIOI(d);
372+
LoongArchExtIOICommonState *s = LOONGARCH_EXTIOI_COMMON(d);
376373

377374
s->status = 0;
378375
}
379376

380377
static int vmstate_extioi_post_load(void *opaque, int version_id)
381378
{
382-
LoongArchExtIOI *s = LOONGARCH_EXTIOI(opaque);
379+
LoongArchExtIOICommonState *s = LOONGARCH_EXTIOI_COMMON(opaque);
383380
int i, start_irq;
384381

385382
for (i = 0; i < (EXTIOI_IRQS / 4); i++) {

include/hw/intc/loongarch_extioi.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,4 @@ struct LoongArchExtIOIClass {
2424
DeviceUnrealize parent_unrealize;
2525
};
2626

27-
#define LoongArchExtIOI LoongArchExtIOICommonState
28-
#define LOONGARCH_EXTIOI(obj) ((LoongArchExtIOICommonState *)obj)
2927
#endif /* LOONGARCH_EXTIOI_H */

0 commit comments

Comments
 (0)