qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Eduardo Habkost <ehabkost@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>, qemu-devel@nongnu.org
Cc: "Paolo Bonzini" <pbonzini@redhat.com>,
	"Alexander Graf" <agraf@suse.de>, "Rob Herring" <robh@kernel.org>,
	libvir-list@redhat.com, "Richard Henderson" <rth@twiddle.net>,
	"David Gibson" <david@gibson.dropbear.id.au>,
	"Eric Blake" <eblake@redhat.com>,
	"Igor Mammedov" <imammedo@redhat.com>,
	qemu-arm@nongnu.org,
	"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
	"Peter Crosthwaite" <crosthwaite.peter@gmail.com>,
	"Markus Armbruster" <armbru@redhat.com>,
	"Artyom Tarasenko" <atar4qemu@gmail.com>,
	"Mark Cave-Ayland" <mark.cave-ayland@ilande.co.uk>,
	"Eduardo Habkost" <ehabkost@redhat.com>,
	"Michael Walle" <michael@walle.cc>,
	"Thomas Huth" <thuth@redhat.com>,
	"Marcel Apfelbaum" <marcel.apfelbaum@gmail.com>,
	"Aleksandar Markovic" <amarkovic@wavecomp.com>,
	"Aurelien Jarno" <aurelien@aurel32.net>,
	"Alistair Francis" <alistair@alistair23.me>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Jason Wang" <jasowang@redhat.com>,
	"Laurent Vivier" <lvivier@redhat.com>,
	qemu-ppc@nongnu.org,
	"Xiao Guangrong" <xiaoguangrong.eric@gmail.com>,
	"Max Filippov" <jcmvbkbc@gmail.com>,
	"Philippe Mathieu-Daudé" <f4bug@amsat.org>
Subject: [Qemu-devel] [PULL v2 08/43] hw/sh4/sh_pci: Use DeviceState::realize rather than SysBusDevice::init
Date: Thu, 25 Oct 2018 10:32:26 -0300	[thread overview]
Message-ID: <20181025133301.16578-9-ehabkost@redhat.com> (raw)
In-Reply-To: <20181025133301.16578-1-ehabkost@redhat.com>

From: Philippe Mathieu-Daudé <f4bug@amsat.org>

Move from the legacy SysBusDevice::init method to using DeviceState::realize.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20181002212522.23303-6-f4bug@amsat.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
---
 hw/sh4/sh_pci.c | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/hw/sh4/sh_pci.c b/hw/sh4/sh_pci.c
index 4ec2e35500..379d0685ed 100644
--- a/hw/sh4/sh_pci.c
+++ b/hw/sh4/sh_pci.c
@@ -120,16 +120,15 @@ static void sh_pci_set_irq(void *opaque, int irq_num, int level)
     qemu_set_irq(pic[irq_num], level);
 }
 
-static int sh_pci_device_init(SysBusDevice *dev)
+static void sh_pci_device_realize(DeviceState *dev, Error **errp)
 {
-    PCIHostState *phb;
-    SHPCIState *s;
+    SysBusDevice *sbd = SYS_BUS_DEVICE(dev);
+    SHPCIState *s = SH_PCI_HOST_BRIDGE(dev);
+    PCIHostState *phb = PCI_HOST_BRIDGE(s);
     int i;
 
-    s = SH_PCI_HOST_BRIDGE(dev);
-    phb = PCI_HOST_BRIDGE(s);
     for (i = 0; i < 4; i++) {
-        sysbus_init_irq(dev, &s->irq[i]);
+        sysbus_init_irq(sbd, &s->irq[i]);
     }
     phb->bus = pci_register_root_bus(DEVICE(dev), "pci",
                                      sh_pci_set_irq, sh_pci_map_irq,
@@ -143,13 +142,12 @@ static int sh_pci_device_init(SysBusDevice *dev)
                              &s->memconfig_p4, 0, 0x224);
     memory_region_init_alias(&s->isa, OBJECT(s), "sh_pci.isa",
                              get_system_io(), 0, 0x40000);
-    sysbus_init_mmio(dev, &s->memconfig_p4);
-    sysbus_init_mmio(dev, &s->memconfig_a7);
+    sysbus_init_mmio(sbd, &s->memconfig_p4);
+    sysbus_init_mmio(sbd, &s->memconfig_a7);
     s->iobr = 0xfe240000;
     memory_region_add_subregion(get_system_memory(), s->iobr, &s->isa);
 
     s->dev = pci_create_simple(phb->bus, PCI_DEVFN(0, 0), "sh_pci_host");
-    return 0;
 }
 
 static void sh_pci_host_realize(PCIDevice *d, Error **errp)
@@ -187,9 +185,9 @@ static const TypeInfo sh_pci_host_info = {
 
 static void sh_pci_device_class_init(ObjectClass *klass, void *data)
 {
-    SysBusDeviceClass *sdc = SYS_BUS_DEVICE_CLASS(klass);
+    DeviceClass *dc = DEVICE_CLASS(klass);
 
-    sdc->init = sh_pci_device_init;
+    dc->realize = sh_pci_device_realize;
 }
 
 static const TypeInfo sh_pci_device_info = {
-- 
2.18.0.rc1.1.g3f1ff2140

  parent reply	other threads:[~2018-10-25 13:34 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-25 13:32 [Qemu-devel] [PULL v2 00/43] Machine queue, 2018-10-25 Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 01/43] hostmem-file: fixed the memory leak while get pmem path Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 02/43] vl.c deprecate incorrect CPUs topology Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 03/43] vl:c: make sure that sockets are calculated correctly in '-smp X' case Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 04/43] trace-events: Fix copy/paste typo Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 05/43] hw/timer/sun4v-rtc: Convert from DPRINTF() macro to trace events Eduardo Habkost
2018-10-25 17:17   ` David Gibson
2018-10-26  8:27     ` Eduardo Habkost
2018-10-26 10:49       ` Philippe Mathieu-Daudé
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 06/43] hw/timer/sun4v-rtc: Use DeviceState::realize rather than SysBusDevice::init Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 07/43] hw/ssi/xilinx_spi: " Eduardo Habkost
2018-10-25 13:32 ` Eduardo Habkost [this message]
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 09/43] hw/pci-host/bonito: " Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 10/43] hw/mips/gt64xxx_pci: Convert gt64120_reset() function into Device reset method Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 11/43] hw/mips/gt64xxx_pci: Mark as bridge device Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 12/43] hw/sparc64/niagara: Model the I/O Bridge with the 'unimplemented_device' Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 13/43] hw/alpha/typhoon: Remove unuseful code Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 14/43] hw/hppa/dino: " Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 15/43] hw/mips/malta: " Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 16/43] machine: fix a typo Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 17/43] memory-device: fix alignment error message Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 18/43] memory-device: fix error message when hinted address is too small Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 19/43] memory-device: improve "range conflicts" error message Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 20/43] pc-dimm: pass PCDIMMDevice to pc_dimm_.*plug Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 21/43] memory-device: use memory device terminology in error messages Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 22/43] memory-device: introduce separate config option Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 23/43] memory-device: forward errors in get_region_size()/get_plugged_size() Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 24/43] memory-device: document MemoryDeviceClass Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 25/43] memory-device: add and use memory_device_get_region_size() Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 26/43] memory-device: factor out get_memory_region() from pc-dimm Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 27/43] memory-device: drop get_region_size() Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 28/43] memory-device: add device class function set_addr() Eduardo Habkost
2018-10-25 13:32 ` [Qemu-devel] [PULL v2 29/43] memory-device: complete factoring out pre_plug handling Eduardo Habkost
2018-10-26 19:16 ` [Qemu-devel] [PULL v2 00/43] Machine queue, 2018-10-25 Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181025133301.16578-9-ehabkost@redhat.com \
    --to=ehabkost@redhat.com \
    --cc=agraf@suse.de \
    --cc=alistair@alistair23.me \
    --cc=amarkovic@wavecomp.com \
    --cc=armbru@redhat.com \
    --cc=atar4qemu@gmail.com \
    --cc=aurelien@aurel32.net \
    --cc=crosthwaite.peter@gmail.com \
    --cc=david@gibson.dropbear.id.au \
    --cc=eblake@redhat.com \
    --cc=edgar.iglesias@gmail.com \
    --cc=f4bug@amsat.org \
    --cc=imammedo@redhat.com \
    --cc=jasowang@redhat.com \
    --cc=jcmvbkbc@gmail.com \
    --cc=libvir-list@redhat.com \
    --cc=lvivier@redhat.com \
    --cc=marcel.apfelbaum@gmail.com \
    --cc=mark.cave-ayland@ilande.co.uk \
    --cc=michael@walle.cc \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=robh@kernel.org \
    --cc=rth@twiddle.net \
    --cc=thuth@redhat.com \
    --cc=xiaoguangrong.eric@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).