OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

hw/intc/arm_gicv3: Report correct PIDR0 values for ID registers

Browse Source 
We use the common function gicv3_idreg() to supply the CoreSight ID
register values for the GICv3 for the copies of these ID registers in
the distributor, redistributor and ITS register frames.  This isn't
quite correct, because while most of the register values are the
same, the PIDR0 value should vary to indicate which of these three
frames it is.  (You can see this and also the correct values of these
PIDR0 registers by looking at the GIC-600 or GIC-700 TRMs, for
example.)

Make gicv3_idreg() take an extra argument for the PIDR0 value.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20220408141550.1271295-5-peter.maydell@linaro.org
This commit is contained in:
Peter Maydell 2022-04-08 15:15:13 +01:00
parent 671927a116
commit 50a3a309e1
4 changed files with 16 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hw/intc/arm_gicv3_dist.c
View File

@ -557,7 +557,7 @@ static bool gicd_readl(GICv3State *s, hwaddr offset,
} }
case GICD_IDREGS ... GICD_IDREGS + 0x2f: case GICD_IDREGS ... GICD_IDREGS + 0x2f:
/* ID registers */ /* ID registers */
*data = gicv3_idreg(offset - GICD_IDREGS); *data = gicv3_idreg(offset - GICD_IDREGS, GICV3_PIDR0_DIST);
return true; return true;
case GICD_SGIR: case GICD_SGIR:
/* WO registers, return unknown value */ /* WO registers, return unknown value */

2
hw/intc/arm_gicv3_its.c
View File

@ -1161,7 +1161,7 @@ static bool its_readl(GICv3ITSState *s, hwaddr offset,
break; break;
case GITS_IDREGS ... GITS_IDREGS + 0x2f: case GITS_IDREGS ... GITS_IDREGS + 0x2f:
/* ID registers */ /* ID registers */
*data = gicv3_idreg(offset - GITS_IDREGS); *data = gicv3_idreg(offset - GITS_IDREGS, GICV3_PIDR0_ITS);
break; break;
case GITS_TYPER: case GITS_TYPER:
*data = extract64(s->typer, 0, 32); *data = extract64(s->typer, 0, 32);

2
hw/intc/arm_gicv3_redist.c
View File

@ -234,7 +234,7 @@ static MemTxResult gicr_readl(GICv3CPUState *cs, hwaddr offset,
*data = cs->gicr_nsacr; *data = cs->gicr_nsacr;
return MEMTX_OK; return MEMTX_OK;
case GICR_IDREGS ... GICR_IDREGS + 0x2f: case GICR_IDREGS ... GICR_IDREGS + 0x2f:
*data = gicv3_idreg(offset - GICR_IDREGS); *data = gicv3_idreg(offset - GICR_IDREGS, GICV3_PIDR0_REDIST);
return MEMTX_OK; return MEMTX_OK;
default: default:
return MEMTX_ERROR; return MEMTX_ERROR;

15
hw/intc/gicv3_internal.h
View File

@ -555,7 +555,12 @@ static inline uint32_t gicv3_iidr(void)
return 0x43b; return 0x43b;
} }
static inline uint32_t gicv3_idreg(int regoffset) /* CoreSight PIDR0 values for ARM GICv3 implementations */
#define GICV3_PIDR0_DIST 0x92
#define GICV3_PIDR0_REDIST 0x93
#define GICV3_PIDR0_ITS 0x94
static inline uint32_t gicv3_idreg(int regoffset, uint8_t pidr0)
{ {
/* Return the value of the CoreSight ID register at the specified /* Return the value of the CoreSight ID register at the specified
* offset from the first ID register (as found in the distributor * offset from the first ID register (as found in the distributor
@ -565,7 +570,13 @@ static inline uint32_t gicv3_idreg(int regoffset)
static const uint8_t gicd_ids[] = { static const uint8_t gicd_ids[] = {
0x44, 0x00, 0x00, 0x00, 0x92, 0xB4, 0x3B, 0x00, 0x0D, 0xF0, 0x05, 0xB1 0x44, 0x00, 0x00, 0x00, 0x92, 0xB4, 0x3B, 0x00, 0x0D, 0xF0, 0x05, 0xB1
}; };
return gicd_ids[regoffset / 4];
regoffset /= 4;
if (regoffset == 4) {
return pidr0;
}
return gicd_ids[regoffset];
} }
/** /**