On Wed, Aug 03, 2022 at 01:25:14AM -0700, Atish Patra wrote: > Historically, The mtime/mtimecmp has been part of the CPU because > they are per hart entities. However, they actually belong to aclint > which is a MMIO device. > > Move them to the ACLINT device. This also emulates the real hardware > more closely. > > Reviewed-by: Anup Patel <a...@brainfault.org> > Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> > Signed-off-by: Atish Patra <ati...@rivosinc.com> > --- > hw/intc/riscv_aclint.c | 41 ++++++++++++++++++++++++---------- > hw/timer/ibex_timer.c | 18 ++++++--------- > include/hw/intc/riscv_aclint.h | 2 ++ > include/hw/timer/ibex_timer.h | 2 ++ > target/riscv/cpu.h | 2 -- > target/riscv/machine.c | 5 ++--- > 6 files changed, 42 insertions(+), 28 deletions(-) >
Reviewed-by: Andrew Jones <ajo...@ventanamicro.com>