From patchwork Thu Aug 27 23:15:13 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Juan Quintela X-Patchwork-Id: 32331 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [199.232.76.165]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by bilbo.ozlabs.org (Postfix) with ESMTPS id 5F5D0B7BD2 for ; Fri, 28 Aug 2009 10:06:52 +1000 (EST) Received: from localhost ([127.0.0.1]:34115 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MgozQ-0004BM-4V for incoming@patchwork.ozlabs.org; Thu, 27 Aug 2009 20:06:48 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MgowB-000377-NK for qemu-devel@nongnu.org; Thu, 27 Aug 2009 20:03:27 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Mgow7-00034N-8e for qemu-devel@nongnu.org; Thu, 27 Aug 2009 20:03:27 -0400 Received: from [199.232.76.173] (port=37722 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Mgow6-000348-Pm for qemu-devel@nongnu.org; Thu, 27 Aug 2009 20:03:22 -0400 Received: from mx20.gnu.org ([199.232.41.8]:6252) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1MgoFE-0005Ms-Ve for qemu-devel@nongnu.org; Thu, 27 Aug 2009 19:19:05 -0400 Received: from mx1.redhat.com ([209.132.183.28]) by mx20.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MgoFD-0005f7-7P for qemu-devel@nongnu.org; Thu, 27 Aug 2009 19:19:03 -0400 Received: from int-mx03.intmail.prod.int.phx2.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.16]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id n7RNHrBL006273 for ; Thu, 27 Aug 2009 19:17:53 -0400 Received: from localhost.localdomain (vpn1-4-202.ams2.redhat.com [10.36.4.202]) by int-mx03.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id n7RNHeu0031343; Thu, 27 Aug 2009 19:17:52 -0400 From: Juan Quintela To: qemu-devel@nongnu.org Date: Fri, 28 Aug 2009 01:15:13 +0200 Message-Id: <48318e8e512ca38f0af1078bd8f5394432e1bd3e.1251414727.git.quintela@redhat.com> In-Reply-To: References: In-Reply-To: References: X-Scanned-By: MIMEDefang 2.67 on 10.5.11.16 X-Detected-Operating-System: by mx20.gnu.org: Genre and OS details not recognized. X-detected-operating-system: by monty-python.gnu.org: Genre and OS details not recognized. Subject: [Qemu-devel] [PATCH 09/13] Fold piix3_init() intto i440fx_init X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org i440fx_init will now work properly if we don't setup piix3 Signed-off-by: Juan Quintela --- hw/pc.c | 3 +-- hw/pc.h | 3 +-- hw/piix_pci.c | 18 ++++++------------ 3 files changed, 8 insertions(+), 16 deletions(-) diff --git a/hw/pc.c b/hw/pc.c index b240ae5..c05b759 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -1279,8 +1279,7 @@ static void pc_init1(ram_addr_t ram_size, ferr_irq = isa_irq[13]; if (pci_enabled) { - pci_bus = i440fx_init(&i440fx_state, isa_irq); - piix3_devfn = piix3_init(pci_bus, -1); + pci_bus = i440fx_init(&i440fx_state, &piix3_devfn, isa_irq); } else { pci_bus = NULL; } diff --git a/hw/pc.h b/hw/pc.h index 3604c70..2234df5 100644 --- a/hw/pc.h +++ b/hw/pc.h @@ -120,9 +120,8 @@ int pcspk_audio_init(qemu_irq *pic); struct PCII440FXState; typedef struct PCII440FXState PCII440FXState; -PCIBus *i440fx_init(PCII440FXState **pi440fx_state, qemu_irq *pic); +PCIBus *i440fx_init(PCII440FXState **pi440fx_state, int *piix_devfn, qemu_irq *pic); void i440fx_set_smm(PCII440FXState *d, int val); -int piix3_init(PCIBus *bus, int devfn); void i440fx_init_memory_mappings(PCII440FXState *d); /* piix4.c */ diff --git a/hw/piix_pci.c b/hw/piix_pci.c index 0443a1d..66aeb72 100644 --- a/hw/piix_pci.c +++ b/hw/piix_pci.c @@ -215,7 +215,9 @@ static void i440fx_initfn(PCIDevice *dev) register_savevm("I440FX", 0, 2, i440fx_save, i440fx_load, d); } -PCIBus *i440fx_init(PCII440FXState **pi440fx_state, qemu_irq *pic) +static PIIX3State *piix3_dev; + +PCIBus *i440fx_init(PCII440FXState **pi440fx_state, int *piix3_devfn, qemu_irq *pic) { DeviceState *dev; PCIBus *b; @@ -234,13 +236,14 @@ PCIBus *i440fx_init(PCII440FXState **pi440fx_state, qemu_irq *pic) d = pci_create_simple(b, 0, "i440FX"); *pi440fx_state = DO_UPCAST(PCII440FXState, dev, d); + piix3_dev = DO_UPCAST(PIIX3State, dev, pci_create_simple(b, -1, "PIIX3")); + *piix3_devfn = piix3_dev->dev.devfn; + return b; } /* PIIX3 PCI to ISA bridge */ -static PIIX3State *piix3_dev; - static void piix3_set_irq(void *opaque, int irq_num, int level) { int i, pic_irq, pic_level; @@ -332,19 +335,10 @@ static void piix3_initfn(PCIDevice *dev) pci_conf[PCI_HEADER_TYPE] = PCI_HEADER_TYPE_NORMAL | PCI_HEADER_TYPE_MULTI_FUNCTION; // header_type = PCI_multifunction, generic - piix3_dev = d; piix3_reset(d); qemu_register_reset(piix3_reset, d); } -int piix3_init(PCIBus *bus, int devfn) -{ - PCIDevice *d; - - d = pci_create_simple(bus, devfn, "PIIX3"); - return d->devfn; -} - static PCIDeviceInfo i440fx_info[] = { { .qdev.name = "i440FX",