Re: [Xen-devel] [PATCH v2 09/20] hw/mips/mips_malta: Create IDE hard drive array dynamically

2019-10-21 Thread Li Qiang
; Signed-off-by: Philippe Mathieu-Daudé > Reviewed-by: Li Qiang > --- > hw/mips/mips_malta.c | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/hw/mips/mips_malta.c b/hw/mips/mips_malta.c > index 528c34a1c3..774bb810f6 100644 > --- a/hw/mi

Re: [Xen-devel] [PATCH v2 06/20] piix4: Add a i8257 DMA Controller as specified in datasheet

2019-10-21 Thread Li Qiang
> +/* DMA */ > +i8257_dma_init(isa_bus, 0); > + > piix4_dev = dev; > } > > Could you please explain why this is better calling 'i8257_dma_init' in piix4 realize function instead of calling it in mips_malta_init. I'm still a little of which things

Re: [Xen-devel] [PATCH v2 05/20] piix4: Rename PIIX4 object to piix4-isa

2019-10-21 Thread Li Qiang
Poussineau > Message-Id: <20171216090228.28505-15-hpous...@reactos.org> > Reviewed-by: Aleksandar Markovic > [PMD: rebased] > Signed-off-by: Philippe Mathieu-Daudé > Reviewed-by: Li Qiang > --- > hw/isa/piix4.c | 1 - > hw/mips/mips_malta.c | 2 +- > inc

Re: [Xen-devel] [PATCH v2 04/20] Revert "irq: introduce qemu_irq_proxy()"

2019-10-21 Thread Li Qiang
Philippe Mathieu-Daudé 于2019年10月18日周五 下午9:50写道: > From: Philippe Mathieu-Daudé > > This function isn't used anymore. > > This reverts commit 22ec3283efba9ba0792790da786d6776d83f2a92. > > Reviewed-by: Thomas Huth > Signed-off-by: Philippe Mathieu-Daudé > Revi

Re: [Xen-devel] [PATCH v2 03/20] piix4: Add a i8259 Interrupt Controller as specified in datasheet

2019-10-21 Thread Li Qiang
> -/* > - * Interrupt controller > - * The 8259 is attached to the MIPS CPU INT0 pin, ie interrupt 2 > - */ > -s->i8259 = i8259_init(isa_bus, i8259_irq); > +/* Interrupt controller */ > +qdev_connect_gpio_out_named(dev, "intr", 0, i8259

Re: [Xen-devel] [PATCH v2 01/20] MAINTAINERS: Keep PIIX4 South Bridge separate from PC Chipsets

2019-10-20 Thread Li Qiang
r Markovic > Signed-off-by: Philippe Mathieu-Daudé > Reviewed-by: Li Qiang > --- > MAINTAINERS | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index fe4dc51b08..c9f625fc2e 100644 > --- a/MAINTAINERS > ++

Re: [Xen-devel] [PATCH v2 02/20] piix4: Add the Reset Control Register

2019-10-20 Thread Li Qiang
- piix4_dev = &d->dev; > + > +memory_region_init_io(&s->rcr_mem, OBJECT(dev), &piix4_rcr_ops, s, > + "reset-control", 1); > +memory_region_add_subregion_overlap(pci_address_space_io(dev), 0xcf9, > Can we use