From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:56231) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RMbb4-0003Uh-AH for qemu-devel@nongnu.org; Sat, 05 Nov 2011 04:27:27 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RMbb3-0002rc-2X for qemu-devel@nongnu.org; Sat, 05 Nov 2011 04:27:26 -0400 Received: from smtp5-g21.free.fr ([212.27.42.5]:33911) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RMbb2-0002rM-Fe for qemu-devel@nongnu.org; Sat, 05 Nov 2011 04:27:25 -0400 Message-ID: <4EB4F35E.1090001@reactos.org> Date: Sat, 05 Nov 2011 09:27:10 +0100 From: =?UTF-8?B?SGVydsOpIFBvdXNzaW5lYXU=?= MIME-Version: 1.0 References: <1319487505-5915-1-git-send-email-hpoussin@reactos.org> <4EAAE532.1030405@reactos.org> In-Reply-To: <4EAAE532.1030405@reactos.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 00/11] isa: preliminary work for multiple buses List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: =?UTF-8?B?SGVydsOpIFBvdXNzaW5lYXU=?= Ping #2. Herv=C3=A9 Poussineau a =C3=A9crit : > Ping. > > Herv=C3=A9 Poussineau a =C3=A9crit : >> Current patches are a rework of my patches already available at [1]. >> They don't provide full support for multiple ISA buses (yet), but >> add a ISABus or ISADevice argument to all ISA functions. >> They are mostly mechanically touching every instanciation of ISA >> devices, so number of lines is quite high even if impact is quite low. >> >> Some patches don't pass checkpass check due to spaces around >> parentheses, but malc asked to do so on files he maintains. >> >> Some more patches will be provided after Qemu 1.0 to support multiple >> ISA buses, but will mostly touch ISA bridges and hw/isa-bus.c file. >> >> I think that this first step can be applied now (before release), >> so ISA interface may be considered stable for devices and machine >> emulations. >> >> Please consider applying this before Qemu 1.0. >> >> Thanks >> >> [1] http://lists.gnu.org/archive/html/qemu-devel/2011-10/msg00094.html >> >> Herv=C3=A9 Poussineau (11): >> isa: give ISABus/ISADevice to isa_create(), isa_bus_irqs() and >> isa_get_irq() functions >> isa: move ISABus structure definition to header file >> i8259: give ISA device to isa_register_ioport() >> pc: give ISA bus to ISA methods >> alpha: give ISA bus to ISA methods >> sun4u: give ISA bus to ISA methods >> fulong2e: give ISA bus to ISA methods >> malta: give ISA bus to ISA methods >> isa: always use provided ISA bus when creating an isa device >> isa: always use provided ISA bus in isa_bus_irqs() >> audio: remove unused parameter isa_pic >> >> arch_init.c | 10 +++++----- >> arch_init.h | 2 +- >> hw/adlib.c | 2 +- >> hw/alpha_dp264.c | 12 +++++++----- >> hw/alpha_sys.h | 3 ++- >> hw/alpha_typhoon.c | 9 +++++---- >> hw/audiodev.h | 8 ++++---- >> hw/cs4231a.c | 4 ++-- >> hw/fdc.h | 4 ++-- >> hw/gus.c | 4 ++-- >> hw/i8254.c | 2 +- >> hw/i8259.c | 10 +++++----- >> hw/ide.h | 2 +- >> hw/ide/isa.c | 4 ++-- >> hw/ide/piix.c | 2 +- >> hw/ide/via.c | 2 +- >> hw/isa-bus.c | 33 ++++++++++++++++----------------- >> hw/isa.h | 16 +++++++++++----- >> hw/m48t59.c | 5 +++-- >> hw/mc146818rtc.c | 4 ++-- >> hw/mc146818rtc.h | 2 +- >> hw/mips_fulong2e.c | 20 ++++++++++---------- >> hw/mips_jazz.c | 13 +++++++------ >> hw/mips_malta.c | 27 ++++++++++++++------------- >> hw/mips_r4k.c | 21 +++++++++++---------- >> hw/nvram.h | 3 ++- >> hw/pc.c | 30 +++++++++++++++--------------- >> hw/pc.h | 39 ++++++++++++++++++++------------------- >> hw/pc_piix.c | 20 +++++++++++--------- >> hw/pcspk.c | 2 +- >> hw/piix4.c | 3 ++- >> hw/piix_pci.c | 8 +++++--- >> hw/ppc_prep.c | 20 +++++++++++--------- >> hw/sb16.c | 4 ++-- >> hw/sun4u.c | 24 +++++++++++++++--------- >> hw/vt82c686.c | 4 ++-- >> hw/vt82c686.h | 2 +- >> qemu-common.h | 1 + >> 38 files changed, 205 insertions(+), 176 deletions(-) >> > >