From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42057) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dvRzP-0008AA-5V for qemu-devel@nongnu.org; Fri, 22 Sep 2017 13:43:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dvRzL-00063n-1z for qemu-devel@nongnu.org; Fri, 22 Sep 2017 13:43:47 -0400 Received: from iserv.reactos.org ([2a01:4f8:221:4c5::3]:40816) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dvRzK-000621-LO for qemu-devel@nongnu.org; Fri, 22 Sep 2017 13:43:42 -0400 References: <20170922154014.29350-1-f4bug@amsat.org> <20170922160111.31885-1-f4bug@amsat.org> From: =?UTF-8?Q?Herv=c3=a9_Poussineau?= Message-ID: Date: Fri, 22 Sep 2017 19:43:29 +0200 MIME-Version: 1.0 In-Reply-To: <20170922160111.31885-1-f4bug@amsat.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH 23/34] hw/dma/i8257: rename DMA_init() to i8257_dma_init() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , Peter Maydell , Thomas Huth , Paolo Bonzini , "Michael S. Tsirkin" , Aurelien Jarno , Yongbok Kim , Richard Henderson , David Gibson , Alexander Graf , Mark Cave-Ayland , Artyom Tarasenko Cc: qemu-devel@nongnu.org, qemu-ppc@nongnu.org, qemu-trivial@nongnu.org Hi, Le 22/09/2017 à 18:01, Philippe Mathieu-Daudé a écrit : > - move the header from hw/isa/ to hw/dma/ > - add entry for i82374 in MAINTAINERS > - remove the old i386/pc dependency > > Signed-off-by: Philippe Mathieu-Daudé > --- > include/hw/{isa/i8257.h => dma/i8257_dma.h} | 6 ++++++ > include/hw/isa/isa.h | 2 -- > hw/dma/i82374.c | 3 ++- > hw/dma/i8257.c | 4 ++-- > hw/i386/pc.c | 3 ++- > hw/mips/mips_fulong2e.c | 3 ++- > hw/mips/mips_jazz.c | 3 ++- > hw/mips/mips_malta.c | 3 ++- > hw/sparc/sun4m.c | 4 ---- > hw/sparc64/sun4u.c | 4 ---- > MAINTAINERS | 2 ++ > 11 files changed, 20 insertions(+), 17 deletions(-) > rename include/hw/{isa/i8257.h => dma/i8257_dma.h} (86%) > > diff --git a/include/hw/isa/i8257.h b/include/hw/dma/i8257_dma.h > similarity index 86% > rename from include/hw/isa/i8257.h > rename to include/hw/dma/i8257_dma.h > index 88a2766a3f..0041565177 100644 > --- a/include/hw/isa/i8257.h > +++ b/include/hw/dma/i8257_dma.h > @@ -1,6 +1,10 @@ > #ifndef HW_I8257_H > #define HW_I8257_H > > +#include "hw/hw.h" > +#include "hw/isa/isa.h" > +#include "exec/ioport.h" > + > #define TYPE_I8257 "i8257" > > typedef struct I8257Regs { > @@ -40,4 +44,6 @@ typedef struct I8257State { > PortioList portio_pageh; > } I8257State; > > +void i8257_dma_init(ISABus *bus, int high_page_enable); > + As you're changing the DMA_init prototype, can you add an Error **errp parameter, and pass &error_abort in each caller? See http://lists.gnu.org/archive/html/qemu-devel/2017-09/msg00262.html for more details Hervé > #endif > diff --git a/include/hw/isa/isa.h b/include/hw/isa/isa.h > index 95593408ef..b9dbab24b4 100644 > --- a/include/hw/isa/isa.h > +++ b/include/hw/isa/isa.h > @@ -151,6 +151,4 @@ static inline ISABus *isa_bus_from_device(ISADevice *d) > return ISA_BUS(qdev_get_parent_bus(DEVICE(d))); > } > > -/* i8257.c */ > -void DMA_init(ISABus *bus, int high_page_enable); > #endif > diff --git a/hw/dma/i82374.c b/hw/dma/i82374.c > index 6c0f975df0..4048bedcf2 100644 > --- a/hw/dma/i82374.c > +++ b/hw/dma/i82374.c > @@ -24,6 +24,7 @@ > > #include "qemu/osdep.h" > #include "hw/isa/isa.h" > +#include "hw/dma/i8257_dma.h" > > #define TYPE_I82374 "i82374" > #define I82374(obj) OBJECT_CHECK(I82374State, (obj), TYPE_I82374) > @@ -123,7 +124,7 @@ static void i82374_realize(DeviceState *dev, Error **errp) > portio_list_add(&s->port_list, isa_address_space_io(&s->parent_obj), > s->iobase); > > - DMA_init(isa_bus_from_device(ISA_DEVICE(dev)), 1); > + i8257_dma_init(isa_bus_from_device(ISA_DEVICE(dev)), 1); > memset(s->commands, 0, sizeof(s->commands)); > } > > diff --git a/hw/dma/i8257.c b/hw/dma/i8257.c > index bd23e893bf..a8f26b003b 100644 > --- a/hw/dma/i8257.c > +++ b/hw/dma/i8257.c > @@ -24,7 +24,7 @@ > #include "qemu/osdep.h" > #include "hw/hw.h" > #include "hw/isa/isa.h" > -#include "hw/isa/i8257.h" > +#include "hw/dma/i8257_dma.h" > #include "qemu/main-loop.h" > #include "trace.h" > > @@ -622,7 +622,7 @@ static void i8257_register_types(void) > > type_init(i8257_register_types) > > -void DMA_init(ISABus *bus, int high_page_enable) > +void i8257_dma_init(ISABus *bus, int high_page_enable) > { > ISADevice *isa1, *isa2; > DeviceState *d; > diff --git a/hw/i386/pc.c b/hw/i386/pc.c > index 3aa10c780b..76d4be5991 100644 > --- a/hw/i386/pc.c > +++ b/hw/i386/pc.c > @@ -39,6 +39,7 @@ > #include "elf.h" > #include "multiboot.h" > #include "hw/timer/mc146818rtc.h" > +#include "hw/dma/i8257_dma.h" > #include "hw/timer/i8254.h" > #include "hw/input/i8042.h" > #include "hw/audio/pcspk.h" > @@ -1582,7 +1583,7 @@ void pc_basic_device_init(ISABus *isa_bus, qemu_irq *gsi, > port92_init(port92, a20_line[1]); > g_free(a20_line); > > - DMA_init(isa_bus, 0); > + i8257_dma_init(isa_bus, 0); > > for(i = 0; i < MAX_FD; i++) { > fd[i] = drive_get(IF_FLOPPY, 0, i); > diff --git a/hw/mips/mips_fulong2e.c b/hw/mips/mips_fulong2e.c > index a35745a407..6dc6a0ac18 100644 > --- a/hw/mips/mips_fulong2e.c > +++ b/hw/mips/mips_fulong2e.c > @@ -22,6 +22,7 @@ > #include "qapi/error.h" > #include "hw/hw.h" > #include "hw/i386/pc.h" > +#include "hw/dma/i8257_dma.h" > #include "hw/char/serial.h" > #include "hw/block/fdc.h" > #include "net/net.h" > @@ -365,7 +366,7 @@ static void mips_fulong2e_init(MachineState *machine) > > /* init other devices */ > pit = i8254_pit_init(isa_bus, 0x40, 0, NULL); > - DMA_init(isa_bus, 0); > + i8257_dma_init(isa_bus, 0); > > /* Super I/O */ > isa_create_simple(isa_bus, TYPE_I8042); > diff --git a/hw/mips/mips_jazz.c b/hw/mips/mips_jazz.c > index 1bbef7eeb5..1b54602c1d 100644 > --- a/hw/mips/mips_jazz.c > +++ b/hw/mips/mips_jazz.c > @@ -27,6 +27,7 @@ > #include "hw/mips/mips.h" > #include "hw/mips/cpudevs.h" > #include "hw/i386/pc.h" > +#include "hw/dma/i8257_dma.h" > #include "hw/char/serial.h" > #include "hw/isa/isa.h" > #include "hw/block/fdc.h" > @@ -223,7 +224,7 @@ static void mips_jazz_init(MachineState *machine, > /* ISA devices */ > i8259 = i8259_init(isa_bus, env->irq[4]); > isa_bus_irqs(isa_bus, i8259); > - DMA_init(isa_bus, 0); > + i8257_dma_init(isa_bus, 0); > pit = i8254_pit_init(isa_bus, 0x40, 0, NULL); > pcspk_init(isa_bus, pit); > > diff --git a/hw/mips/mips_malta.c b/hw/mips/mips_malta.c > index 45c7f589aa..050da36f1d 100644 > --- a/hw/mips/mips_malta.c > +++ b/hw/mips/mips_malta.c > @@ -27,6 +27,7 @@ > #include "cpu.h" > #include "hw/hw.h" > #include "hw/i386/pc.h" > +#include "hw/dma/i8257_dma.h" > #include "hw/char/serial.h" > #include "hw/block/fdc.h" > #include "net/net.h" > @@ -1219,7 +1220,7 @@ void mips_malta_init(MachineState *machine) > smbus_eeprom_init(smbus, 8, smbus_eeprom_buf, smbus_eeprom_size); > g_free(smbus_eeprom_buf); > pit = i8254_pit_init(isa_bus, 0x40, 0, NULL); > - DMA_init(isa_bus, 0); > + i8257_dma_init(isa_bus, 0); > > /* Super I/O */ > isa_create_simple(isa_bus, TYPE_I8042); > diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c > index e1bdd4828d..a4ba3d2917 100644 > --- a/hw/sparc/sun4m.c > +++ b/hw/sparc/sun4m.c > @@ -101,10 +101,6 @@ struct sun4m_hwdef { > uint8_t nvram_machine_id; > }; > > -void DMA_init(ISABus *bus, int high_page_enable) > -{ > -} > - > static void fw_cfg_boot_set(void *opaque, const char *boot_device, > Error **errp) > { > diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c > index 973daed977..fd10741607 100644 > --- a/hw/sparc64/sun4u.c > +++ b/hw/sparc64/sun4u.c > @@ -86,10 +86,6 @@ typedef struct EbusState { > MemoryRegion bar1; > } EbusState; > > -void DMA_init(ISABus *bus, int high_page_enable) > -{ > -} > - > static void fw_cfg_boot_set(void *opaque, const char *boot_device, > Error **errp) > { > diff --git a/MAINTAINERS b/MAINTAINERS > index cc1f8bb66b..a79723601c 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -707,6 +707,7 @@ F: hw/ppc/rs6000_mc.c > F: hw/pci-host/prep.[hc] > F: hw/isa/i82378.c > F: hw/isa/pc87312.[hc] > +F: hw/dma/i82374.c > F: pc-bios/ppc_rom.bin > > sPAPR > @@ -849,6 +850,7 @@ F: hw/misc/pc-testdev.c > F: hw/timer/hpet* > F: hw/timer/i8254* > F: hw/timer/mc146818rtc* > +F: include/hw/dma/i8257_dma.h > F: include/hw/i2c/pm_smbus.h > F: include/hw/input/i8042.h > F: include/hw/timer/hpet.h >