On 10/28/21 21:27, BALATON Zoltan wrote:
> Signed-off-by: BALATON Zoltan <bala...@eik.bme.hu>
> ---
> hw/char/sh_serial.c | 107 +++++++++++++++++++++++++++-----------------
> hw/sh4/sh7750.c | 62 ++++++++++++++++++-------
> include/hw/sh4/sh.h | 9 +---
> 3 files changed, 114 insertions(+), 64 deletions(-)
> +OBJECT_DECLARE_SIMPLE_TYPE(SHSerialState, SH_SERIAL)
> +
> +struct SHSerialState {
> + SysBusDevice parent;
[...]
> -} SHSerialState;
> +};
> +
> +typedef struct {} SHSerialStateClass;
OBJECT_DECLARE_TYPE()?
> -void sh_serial_init(MemoryRegion *sysmem,
> - hwaddr base, int feat,
> - uint32_t freq, Chardev *chr,
> - qemu_irq eri_source,
> - qemu_irq rxi_source,
> - qemu_irq txi_source,
> - qemu_irq tei_source,
> - qemu_irq bri_source)
> +static void sh_serial_reset(DeviceState *dev)
Can you extract sh_serial_reset() in a previous patch?
> {
> - SHSerialState *s = g_malloc0(sizeof(*s));
> + SHSerialState *s = SH_SERIAL(dev);
>
> - s->feat = feat;
> s->flags = SH_SERIAL_FLAG_TEND | SH_SERIAL_FLAG_TDE;
> s->rtrg = 1;
>
> @@ -397,38 +396,64 @@ void sh_serial_init(MemoryRegion *sysmem,
> s->scr = 1 << 5; /* pretend that TX is enabled so early printk works */
> s->sptr = 0;
>
> - if (feat & SH_SERIAL_FEAT_SCIF) {
> + if (s->feat & SH_SERIAL_FEAT_SCIF) {
> s->fcr = 0;
> } else {
> s->dr = 0xff;
> }
>
> sh_serial_clear_fifo(s);
> +}
>
> - memory_region_init_io(&s->iomem, NULL, &sh_serial_ops, s,
> - "serial", 0x100000000ULL);
Keep that, ...
> - memory_region_init_alias(&s->iomem_p4, NULL, "serial-p4", &s->iomem,
> - 0, 0x28);
> - memory_region_add_subregion(sysmem, P4ADDR(base), &s->iomem_p4);
> -
> - memory_region_init_alias(&s->iomem_a7, NULL, "serial-a7", &s->iomem,
> - 0, 0x28);
> - memory_region_add_subregion(sysmem, A7ADDR(base), &s->iomem_a7);
... and these lines become one single sysbus_init_mmio() ...
> -
> - if (chr) {
> - qemu_chr_fe_init(&s->chr, chr, &error_abort);
> +static void sh_serial_realize(DeviceState *d, Error **errp)
> +{
> + SHSerialState *s = SH_SERIAL(d);
> + MemoryRegion *iomem = g_malloc(sizeof(*iomem));
> +
> + assert(d->id);
> + memory_region_init_io(iomem, OBJECT(d), &sh_serial_ops, s, d->id, 0x28);
> + sysbus_init_mmio(SYS_BUS_DEVICE(d), iomem);
> + qdev_init_gpio_out_named(d, &s->eri, "eri", 1);
> + qdev_init_gpio_out_named(d, &s->rxi, "rxi", 1);
> + qdev_init_gpio_out_named(d, &s->txi, "txi", 1);
> + qdev_init_gpio_out_named(d, &s->tei, "tei", 1);
> + qdev_init_gpio_out_named(d, &s->bri, "bri", 1);
> +
> + if (qemu_chr_fe_backend_connected(&s->chr)) {
> qemu_chr_fe_set_handlers(&s->chr, sh_serial_can_receive1,
> sh_serial_receive1,
> sh_serial_event, NULL, s, NULL, true);
> }
>
> - s->fifo_timeout_timer = timer_new_ns(QEMU_CLOCK_VIRTUAL,
> - sh_serial_timeout_int, s);
> + timer_init_ns(&s->fifo_timeout_timer, QEMU_CLOCK_VIRTUAL,
> + sh_serial_timeout_int, s);
> s->etu = NANOSECONDS_PER_SECOND / 9600;
> - s->eri = eri_source;
> - s->rxi = rxi_source;
> - s->txi = txi_source;
> - s->tei = tei_source;
> - s->bri = bri_source;
> +}
> @@ -762,6 +766,9 @@ static const MemoryRegionOps sh7750_mmct_ops = {
> SH7750State *sh7750_init(SuperHCPU *cpu, MemoryRegion *sysmem)
> {
> SH7750State *s;
> + DeviceState *dev;
> + SysBusDevice *sb;
> + MemoryRegion *mr, *alias;
>
> s = g_malloc0(sizeof(SH7750State));
> s->cpu = cpu;
> @@ -807,21 +814,46 @@ SH7750State *sh7750_init(SuperHCPU *cpu, MemoryRegion
> *sysmem)
>
> cpu->env.intc_handle = &s->intc;
>
> - sh_serial_init(sysmem, 0x1fe00000,
> - 0, s->periph_freq, serial_hd(0),
> - s->intc.irqs[SCI1_ERI],
> - s->intc.irqs[SCI1_RXI],
> - s->intc.irqs[SCI1_TXI],
> - s->intc.irqs[SCI1_TEI],
> - NULL);
> - sh_serial_init(sysmem, 0x1fe80000,
> - SH_SERIAL_FEAT_SCIF,
> - s->periph_freq, serial_hd(1),
> - s->intc.irqs[SCIF_ERI],
> - s->intc.irqs[SCIF_RXI],
> - s->intc.irqs[SCIF_TXI],
> - NULL,
> - s->intc.irqs[SCIF_BRI]);
> + /* SCI */
> + dev = qdev_new(TYPE_SH_SERIAL);
> + dev->id = (char *)"sci";
> + qdev_prop_set_chr(dev, "chardev", serial_hd(0));
> + sb = SYS_BUS_DEVICE(dev);
> + sysbus_realize_and_unref(sb, &error_fatal);
> + mr = sysbus_mmio_get_region(sb, 0);
> + alias = g_malloc(sizeof(*alias));
> + memory_region_init_alias(alias, OBJECT(dev), "sci-p4", mr,
> + 0, memory_region_size(mr));
> + memory_region_add_subregion(sysmem, P4ADDR(0x1fe00000), alias);
> + alias = g_malloc(sizeof(*alias));
> + memory_region_init_alias(alias, OBJECT(dev), "sci-a7", mr,
> + 0, memory_region_size(mr));
> + memory_region_add_subregion(sysmem, A7ADDR(0x1fe00000), alias);
... then you can replace the aliases by 2 sysbus_mmio_map() calls.
> + qdev_connect_gpio_out_named(dev, "eri", 0, s->intc.irqs[SCI1_ERI]);
> + qdev_connect_gpio_out_named(dev, "rxi", 0, s->intc.irqs[SCI1_RXI]);
> + qdev_connect_gpio_out_named(dev, "txi", 0, s->intc.irqs[SCI1_TXI]);
> + qdev_connect_gpio_out_named(dev, "tei", 0, s->intc.irqs[SCI1_TEI]);
> +
> + /* SCIF */
> + dev = qdev_new(TYPE_SH_SERIAL);
> + dev->id = (char *)"scif";
> + qdev_prop_set_chr(dev, "chardev", serial_hd(1));
> + qdev_prop_set_uint8(dev, "features", SH_SERIAL_FEAT_SCIF);
> + sb = SYS_BUS_DEVICE(dev);
> + sysbus_realize_and_unref(sb, &error_fatal);
> + mr = sysbus_mmio_get_region(sb, 0);
> + alias = g_malloc(sizeof(*alias));
> + memory_region_init_alias(alias, OBJECT(dev), "scif-p4", mr,
> + 0, memory_region_size(mr));
> + memory_region_add_subregion(sysmem, P4ADDR(0x1fe80000), alias);
> + alias = g_malloc(sizeof(*alias));
> + memory_region_init_alias(alias, OBJECT(dev), "scif-a7", mr,
> + 0, memory_region_size(mr));
> + memory_region_add_subregion(sysmem, A7ADDR(0x1fe80000), alias);
> + qdev_connect_gpio_out_named(dev, "eri", 0, s->intc.irqs[SCIF_ERI]);
> + qdev_connect_gpio_out_named(dev, "rxi", 0, s->intc.irqs[SCIF_RXI]);
> + qdev_connect_gpio_out_named(dev, "txi", 0, s->intc.irqs[SCIF_TXI]);
> + qdev_connect_gpio_out_named(dev, "bri", 0, s->intc.irqs[SCIF_BRI]);
>
