[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 04/11] hw/m68k/next-cube: Move scr_ops into NeXTPC device
From: |
Thomas Huth |
Subject: |
Re: [PATCH 04/11] hw/m68k/next-cube: Move scr_ops into NeXTPC device |
Date: |
Sat, 16 Jan 2021 09:18:53 +0100 |
Am Fri, 15 Jan 2021 20:11:59 +0000
schrieb Peter Maydell <peter.maydell@linaro.org>:
> Move the registers handled by the scr_ops struct into the NeXTPC
> device.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> hw/m68k/next-cube.c | 50
> ++++++++++++++++++++++----------------------- 1 file changed, 25
> insertions(+), 25 deletions(-)
>
> diff --git a/hw/m68k/next-cube.c b/hw/m68k/next-cube.c
> index ff121143e92..f5575cb43b8 100644
> --- a/hw/m68k/next-cube.c
> +++ b/hw/m68k/next-cube.c
> @@ -76,8 +76,6 @@ struct NeXTState {
> uint32_t int_mask;
> uint32_t int_status;
>
> - uint8_t scsi_csr_1;
> - uint8_t scsi_csr_2;
> next_dma dma[10];
> qemu_irq *scsi_irq;
> qemu_irq scsi_dma;
> @@ -97,9 +95,12 @@ struct NeXTPC {
> NeXTState *ns;
>
> MemoryRegion mmiomem;
> + MemoryRegion scrmem;
>
> uint32_t scr1;
> uint32_t scr2;
> + uint8_t scsi_csr_1;
> + uint8_t scsi_csr_2;
> };
>
> /* Thanks to NeXT forums for this */
> @@ -402,7 +403,7 @@ static const MemoryRegionOps mmio_ops = {
> .endianness = DEVICE_NATIVE_ENDIAN,
> };
>
> -static uint32_t scr_readb(NeXTState *s, hwaddr addr)
> +static uint32_t scr_readb(NeXTPC *s, hwaddr addr)
> {
> switch (addr) {
> case 0x14108:
> @@ -436,13 +437,13 @@ static uint32_t scr_readb(NeXTState *s, hwaddr
> addr) }
> }
>
> -static uint32_t scr_readw(NeXTState *s, hwaddr addr)
> +static uint32_t scr_readw(NeXTPC *s, hwaddr addr)
> {
> DPRINTF("BMAP Read W @ %x\n", (unsigned int)addr);
> return 0;
> }
>
> -static uint32_t scr_readl(NeXTState *s, hwaddr addr)
> +static uint32_t scr_readl(NeXTPC *s, hwaddr addr)
> {
> DPRINTF("BMAP Read L @ %x\n", (unsigned int)addr);
> return 0;
> @@ -455,7 +456,7 @@ static uint32_t scr_readl(NeXTState *s, hwaddr
> addr) #define SCSICSR_CPUDMA 0x10 /* if set, dma enabled */
> #define SCSICSR_INTMASK 0x20 /* if set, interrupt enabled */
>
> -static void scr_writeb(NeXTState *s, hwaddr addr, uint32_t value)
> +static void scr_writeb(NeXTPC *s, hwaddr addr, uint32_t value)
> {
> switch (addr) {
> case 0x14108:
> @@ -501,9 +502,9 @@ static void scr_writeb(NeXTState *s, hwaddr addr,
> uint32_t value) DPRINTF("SCSICSR CPUDMA\n");
> /* qemu_irq_raise(s->scsi_dma); */
>
> - s->int_status |= 0x4000000;
> + s->ns->int_status |= 0x4000000;
> } else {
> - s->int_status &= ~(0x4000000);
> + s->ns->int_status &= ~(0x4000000);
> }
> if (value & SCSICSR_INTMASK) {
> DPRINTF("SCSICSR INTMASK\n");
> @@ -533,27 +534,27 @@ static void scr_writeb(NeXTState *s, hwaddr
> addr, uint32_t value) }
> }
>
> -static void scr_writew(NeXTState *s, hwaddr addr, uint32_t value)
> +static void scr_writew(NeXTPC *s, hwaddr addr, uint32_t value)
> {
> DPRINTF("BMAP Write W @ %x with %x\n", (unsigned int)addr,
> value); }
>
> -static void scr_writel(NeXTState *s, hwaddr addr, uint32_t value)
> +static void scr_writel(NeXTPC *s, hwaddr addr, uint32_t value)
> {
> DPRINTF("BMAP Write L @ %x with %x\n", (unsigned int)addr,
> value); }
>
> static uint64_t scr_readfn(void *opaque, hwaddr addr, unsigned size)
> {
> - NeXTState *ns = NEXT_MACHINE(opaque);
> + NeXTPC *s = NEXT_PC(opaque);
>
> switch (size) {
> case 1:
> - return scr_readb(ns, addr);
> + return scr_readb(s, addr);
> case 2:
> - return scr_readw(ns, addr);
> + return scr_readw(s, addr);
> case 4:
> - return scr_readl(ns, addr);
> + return scr_readl(s, addr);
> default:
> g_assert_not_reached();
> }
> @@ -562,17 +563,17 @@ static uint64_t scr_readfn(void *opaque, hwaddr
> addr, unsigned size) static void scr_writefn(void *opaque, hwaddr
> addr, uint64_t value, unsigned size)
> {
> - NeXTState *ns = NEXT_MACHINE(opaque);
> + NeXTPC *s = NEXT_PC(opaque);
>
> switch (size) {
> case 1:
> - scr_writeb(ns, addr, value);
> + scr_writeb(s, addr, value);
> break;
> case 2:
> - scr_writew(ns, addr, value);
> + scr_writew(s, addr, value);
> break;
> case 4:
> - scr_writel(ns, addr, value);
> + scr_writel(s, addr, value);
> break;
> default:
> g_assert_not_reached();
> @@ -886,8 +887,10 @@ static void next_pc_realize(DeviceState *dev,
> Error **errp)
> memory_region_init_io(&s->mmiomem, OBJECT(s), &mmio_ops, s,
> "next.mmio", 0xD0000);
> -
> + memory_region_init_io(&s->scrmem, OBJECT(s), &scr_ops, s,
> + "next.scr", 0x20000);
> sysbus_init_mmio(sbd, &s->mmiomem);
> + sysbus_init_mmio(sbd, &s->scrmem);
> }
>
> static void next_pc_class_init(ObjectClass *klass, void *data)
> @@ -912,7 +915,6 @@ static void next_cube_init(MachineState *machine)
> M68kCPU *cpu;
> CPUM68KState *env;
> MemoryRegion *rom = g_new(MemoryRegion, 1);
> - MemoryRegion *scrmem = g_new(MemoryRegion, 1);
> MemoryRegion *dmamem = g_new(MemoryRegion, 1);
> MemoryRegion *bmapm1 = g_new(MemoryRegion, 1);
> MemoryRegion *bmapm2 = g_new(MemoryRegion, 1);
> @@ -956,6 +958,9 @@ static void next_cube_init(MachineState *machine)
> /* MMIO */
> sysbus_mmio_map(SYS_BUS_DEVICE(pcdev), 0, 0x02000000);
>
> + /* BMAP IO - acts as a catch-all for now */
> + sysbus_mmio_map(SYS_BUS_DEVICE(pcdev), 1, 0x02100000);
> +
> /* BMAP memory */
> memory_region_init_ram_shared_nomigrate(bmapm1, NULL,
> "next.bmapmem", 64, true, &error_fatal);
> @@ -964,11 +969,6 @@ static void next_cube_init(MachineState *machine)
> memory_region_init_alias(bmapm2, NULL, "next.bmapmem2", bmapm1,
> 0x0, 64); memory_region_add_subregion(sysmem, 0x820c0000, bmapm2);
>
> - /* BMAP IO - acts as a catch-all for now */
> - memory_region_init_io(scrmem, NULL, &scr_ops, machine,
> "next.scr",
> - 0x20000);
> - memory_region_add_subregion(sysmem, 0x02100000, scrmem);
> -
> /* KBD */
> dev = qdev_new(TYPE_NEXTKBD);
> sysbus_realize_and_unref(SYS_BUS_DEVICE(dev), &error_fatal);
Reviewed-by: Thomas Huth <huth@tuxfamily.org>
- [PATCH 00/11] hw/m68k/next-cube: refactor to fix Coverity issue, Peter Maydell, 2021/01/15
- [PATCH 01/11] hw/m68k/next-cube: Make next_irq() function static, Peter Maydell, 2021/01/15
- [PATCH 02/11] hw/m68k/next-cube: Move register/interrupt functionality into a device, Peter Maydell, 2021/01/15
- [PATCH 03/11] hw/m68k/next-cube: Move mmio_ops into NeXTPC device, Peter Maydell, 2021/01/15
- [PATCH 04/11] hw/m68k/next-cube: Move scr_ops into NeXTPC device, Peter Maydell, 2021/01/15
- Re: [PATCH 04/11] hw/m68k/next-cube: Move scr_ops into NeXTPC device,
Thomas Huth <=
- [PATCH 05/11] hw/m68k/next-cube: Make next_irq take NeXTPC* as its opaque, Peter Maydell, 2021/01/15
- [PATCH 06/11] hw/m68k/next-cube: Move int_status and int_mask to NeXTPC struct, Peter Maydell, 2021/01/15
- [PATCH 07/11] hw/m68k/next-cube: Make next_irq GPIO inputs to NEXT_PC device, Peter Maydell, 2021/01/15
- [PATCH 09/11] hw/m68k/next-cube: Remove unused fields from NeXTState, Peter Maydell, 2021/01/15
- [PATCH 08/11] hw/m68k/next-cube: Move rtc into NeXTPC struct, Peter Maydell, 2021/01/15