Re: [Xen-devel] [PATCH 14/32] piix4: add a i8257 dma controller as specified in datasheet
On 15/10/2019 18.26, Philippe Mathieu-Daudé wrote: > > Remove i8257 instanciated in malta board, to not have it twice. s/instanciated/instantiated/ Thomas ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH 14/32] piix4: add a i8257 dma controller as specified in datasheet
On Tuesday, October 15, 2019, Philippe Mathieu-Daudé wrote: > From: Hervé Poussineau > > Remove i8257 instanciated in malta board, to not have it twice. > > Acked-by: Michael S. Tsirkin > Acked-by: Paolo Bonzini > Signed-off-by: Hervé Poussineau > Message-Id: <20171216090228.28505-9-hpous...@reactos.org> > [PMD: rebased] > Signed-off-by: Philippe Mathieu-Daudé > --- > hw/isa/piix4.c | 4 > hw/mips/mips_malta.c | 2 -- > 2 files changed, 4 insertions(+), 2 deletions(-) > > Reviewed-by: Aleksandar Markovic > diff --git a/hw/isa/piix4.c b/hw/isa/piix4.c > index 8998b0ca47..1bc91b590c 100644 > --- a/hw/isa/piix4.c > +++ b/hw/isa/piix4.c > @@ -29,6 +29,7 @@ > #include "hw/pci/pci.h" > #include "hw/isa/isa.h" > #include "hw/sysbus.h" > +#include "hw/dma/i8257.h" > #include "migration/vmstate.h" > #include "sysemu/reset.h" > #include "sysemu/runstate.h" > @@ -164,6 +165,9 @@ static void piix4_realize(PCIDevice *pci_dev, Error > **errp) > /* initialize ISA irqs */ > isa_bus_irqs(isa_bus, s->isa); > > +/* DMA */ > +i8257_dma_init(isa_bus, 0); > + > piix4_dev = pci_dev; > } > > diff --git a/hw/mips/mips_malta.c b/hw/mips/mips_malta.c > index e499b7a6bb..df247177ca 100644 > --- a/hw/mips/mips_malta.c > +++ b/hw/mips/mips_malta.c > @@ -28,7 +28,6 @@ > #include "cpu.h" > #include "hw/i386/pc.h" > #include "hw/isa/superio.h" > -#include "hw/dma/i8257.h" > #include "hw/char/serial.h" > #include "net/net.h" > #include "hw/boards.h" > @@ -1430,7 +1429,6 @@ void mips_malta_init(MachineState *machine) > smbus = piix4_pm_init(pci_bus, piix4_devfn + 3, 0x1100, >isa_get_irq(NULL, 9), NULL, 0, NULL); > pit = i8254_pit_init(isa_bus, 0x40, 0, NULL); > -i8257_dma_init(isa_bus, 0); > mc146818_rtc_init(isa_bus, 2000, NULL); > > /* generate SPD EEPROM data */ > -- > 2.21.0 > > > ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
[Xen-devel] [PATCH 14/32] piix4: add a i8257 dma controller as specified in datasheet
From: Hervé Poussineau Remove i8257 instanciated in malta board, to not have it twice. Acked-by: Michael S. Tsirkin Acked-by: Paolo Bonzini Signed-off-by: Hervé Poussineau Message-Id: <20171216090228.28505-9-hpous...@reactos.org> [PMD: rebased] Signed-off-by: Philippe Mathieu-Daudé --- hw/isa/piix4.c | 4 hw/mips/mips_malta.c | 2 -- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/hw/isa/piix4.c b/hw/isa/piix4.c index 8998b0ca47..1bc91b590c 100644 --- a/hw/isa/piix4.c +++ b/hw/isa/piix4.c @@ -29,6 +29,7 @@ #include "hw/pci/pci.h" #include "hw/isa/isa.h" #include "hw/sysbus.h" +#include "hw/dma/i8257.h" #include "migration/vmstate.h" #include "sysemu/reset.h" #include "sysemu/runstate.h" @@ -164,6 +165,9 @@ static void piix4_realize(PCIDevice *pci_dev, Error **errp) /* initialize ISA irqs */ isa_bus_irqs(isa_bus, s->isa); +/* DMA */ +i8257_dma_init(isa_bus, 0); + piix4_dev = pci_dev; } diff --git a/hw/mips/mips_malta.c b/hw/mips/mips_malta.c index e499b7a6bb..df247177ca 100644 --- a/hw/mips/mips_malta.c +++ b/hw/mips/mips_malta.c @@ -28,7 +28,6 @@ #include "cpu.h" #include "hw/i386/pc.h" #include "hw/isa/superio.h" -#include "hw/dma/i8257.h" #include "hw/char/serial.h" #include "net/net.h" #include "hw/boards.h" @@ -1430,7 +1429,6 @@ void mips_malta_init(MachineState *machine) smbus = piix4_pm_init(pci_bus, piix4_devfn + 3, 0x1100, isa_get_irq(NULL, 9), NULL, 0, NULL); pit = i8254_pit_init(isa_bus, 0x40, 0, NULL); -i8257_dma_init(isa_bus, 0); mc146818_rtc_init(isa_bus, 2000, NULL); /* generate SPD EEPROM data */ -- 2.21.0 ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
