From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:57143) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rd80E-0008BJ-AS for qemu-devel@nongnu.org; Tue, 20 Dec 2011 17:17:43 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Rd80C-0000dq-Vw for qemu-devel@nongnu.org; Tue, 20 Dec 2011 17:17:42 -0500 Received: from e9.ny.us.ibm.com ([32.97.182.139]:33735) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rd80C-0000dk-QK for qemu-devel@nongnu.org; Tue, 20 Dec 2011 17:17:40 -0500 Received: from /spool/local by e9.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 20 Dec 2011 17:17:38 -0500 Received: from d01av03.pok.ibm.com (d01av03.pok.ibm.com [9.56.224.217]) by d01relay01.pok.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id pBKMHaY3259568 for ; Tue, 20 Dec 2011 17:17:36 -0500 Received: from d01av03.pok.ibm.com (loopback [127.0.0.1]) by d01av03.pok.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id pBKMHaD3015774 for ; Tue, 20 Dec 2011 20:17:36 -0200 Message-ID: <4EF1097F.4050705@us.ibm.com> Date: Tue, 20 Dec 2011 16:17:35 -0600 From: Anthony Liguori MIME-Version: 1.0 References: <1323983401-18345-1-git-send-email-hpoussin@reactos.org> In-Reply-To: <1323983401-18345-1-git-send-email-hpoussin@reactos.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH v3 00/11] isa: preliminary work for multiple buses List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?B?SGVydsOpIFBvdXNzaW5lYXU=?= Cc: qemu-devel@nongnu.org On 12/15/2011 03:09 PM, Hervé Poussineau wrote: > 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 need to be provided to support multiple ISA buses, > but they will mostly touch ISA bridges and hw/isa-bus.c file. Applied all. Thanks. Regards, Anthony Liguori > > Thanks > > [1] http://lists.gnu.org/archive/html/qemu-devel/2011-10/msg00094.html > > Changes v2->v3 > rebased > fixed compilation with some compilers (typedef redefinition) > > Changes v1->v2 > rebased > > Hervé 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 | 17 +++++++++++------ > 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 | 28 ++++++++++++++-------------- > 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, 204 insertions(+), 176 deletions(-) >