From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50301) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eQxZZ-0002VP-FK for qemu-devel@nongnu.org; Mon, 18 Dec 2017 10:43:23 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eQxZV-0002Ix-D5 for qemu-devel@nongnu.org; Mon, 18 Dec 2017 10:43:21 -0500 Received: from mx1.redhat.com ([209.132.183.28]:42302) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1eQxZV-0002HF-04 for qemu-devel@nongnu.org; Mon, 18 Dec 2017 10:43:17 -0500 References: <20171218151244.9975-1-f4bug@amsat.org> <20171218151244.9975-4-f4bug@amsat.org> From: Marcel Apfelbaum Message-ID: <00f65d06-fcb4-ac56-2247-727891546b9a@redhat.com> Date: Mon, 18 Dec 2017 17:43:08 +0200 MIME-Version: 1.0 In-Reply-To: <20171218151244.9975-4-f4bug@amsat.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v2 3/4] hw/pci-host/xilinx: QOM'ify the AXI-PCIe host bridge List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , "Michael S. Tsirkin" , Eduardo Habkost , Paul Burton , Yongbok Kim , "Edgar E . Iglesias" , Alistair Francis Cc: qemu-devel@nongnu.org, James Hogan On 18/12/2017 17:12, Philippe Mathieu-Daud=C3=A9 wrote: > Signed-off-by: Philippe Mathieu-Daud=C3=A9 > --- > v2: use 'pci_dev' variable, replaced hw_error() -> error_setg() >=20 > hw/pci-host/xilinx-pcie.c | 20 +++++++++----------- > 1 file changed, 9 insertions(+), 11 deletions(-) >=20 > diff --git a/hw/pci-host/xilinx-pcie.c b/hw/pci-host/xilinx-pcie.c > index 7659253090..681fdf486a 100644 > --- a/hw/pci-host/xilinx-pcie.c > +++ b/hw/pci-host/xilinx-pcie.c > @@ -267,24 +267,22 @@ static void xilinx_pcie_root_config_write(PCIDevi= ce *d, uint32_t address, > } > } > =20 > -static int xilinx_pcie_root_init(PCIDevice *dev) > +static void xilinx_pcie_root_realize(PCIDevice *pci_dev, Error **errp) > { > - BusState *bus =3D qdev_get_parent_bus(DEVICE(dev)); > + BusState *bus =3D qdev_get_parent_bus(DEVICE(pci_dev)); > XilinxPCIEHost *s =3D XILINX_PCIE_HOST(bus->parent); > =20 > - pci_set_word(dev->config + PCI_COMMAND, > + pci_set_word(pci_dev->config + PCI_COMMAND, > PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER); > - pci_set_word(dev->config + PCI_MEMORY_BASE, s->mmio_base >> 16); > - pci_set_word(dev->config + PCI_MEMORY_LIMIT, > + pci_set_word(pci_dev->config + PCI_MEMORY_BASE, s->mmio_base >> 16= ); > + pci_set_word(pci_dev->config + PCI_MEMORY_LIMIT, > ((s->mmio_base + s->mmio_size - 1) >> 16) & 0xfff0); > =20 > - pci_bridge_initfn(dev, TYPE_PCI_BUS); > + pci_bridge_initfn(pci_dev, TYPE_PCI_BUS); > =20 > - if (pcie_endpoint_cap_v1_init(dev, 0x80) < 0) { > - hw_error("Failed to initialize PCIe capability"); > + if (pcie_endpoint_cap_v1_init(pci_dev, 0x80) < 0) { > + error_setg(errp, "Failed to initialize PCIe capability"); > } > - > - return 0; > } > =20 > static void xilinx_pcie_root_class_init(ObjectClass *klass, void *dat= a) > @@ -300,7 +298,7 @@ static void xilinx_pcie_root_class_init(ObjectClass= *klass, void *data) > k->class_id =3D PCI_CLASS_BRIDGE_HOST; > k->is_express =3D true; > k->is_bridge =3D true; > - k->init =3D xilinx_pcie_root_init; > + k->realize =3D xilinx_pcie_root_realize; > k->exit =3D pci_bridge_exitfn; > dc->reset =3D pci_bridge_reset; > k->config_read =3D xilinx_pcie_root_config_read; >=20 Reviewed-by: Marcel Apfelbaum Thanks, Marcel