diff options
| author | Alex Williamson <alex.williamson@redhat.com> | 2012-07-03 22:39:27 -0600 |
|---|---|---|
| committer | Michael S. Tsirkin <mst@redhat.com> | 2012-07-04 15:52:55 +0300 |
| commit | f90c2bcdbc69e41e575f868b984c3e2de8f51bac (patch) | |
| tree | 1955dd08ce0a5e42ef1079ee292a4e5d95073776 /hw/xio3130_downstream.c | |
| parent | 572992eefa74bfb92c24a28bd268de91a9311b0f (diff) | |
| download | focaccia-qemu-f90c2bcdbc69e41e575f868b984c3e2de8f51bac.tar.gz focaccia-qemu-f90c2bcdbc69e41e575f868b984c3e2de8f51bac.zip | |
pci: convert PCIUnregisterFunc to void
Not a single driver has any possibility of failure on their exit function, let's keep it that way. Signed-off-by: Alex Williamson <alex.williamson@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'hw/xio3130_downstream.c')
| -rw-r--r-- | hw/xio3130_downstream.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/hw/xio3130_downstream.c b/hw/xio3130_downstream.c index 56d1b353d0..0d8a5e7020 100644 --- a/hw/xio3130_downstream.c +++ b/hw/xio3130_downstream.c @@ -60,7 +60,6 @@ static int xio3130_downstream_initfn(PCIDevice *d) PCIEPort *p = DO_UPCAST(PCIEPort, br, br); PCIESlot *s = DO_UPCAST(PCIESlot, port, p); int rc; - int tmp; rc = pci_bridge_initfn(d); if (rc < 0) { @@ -108,12 +107,11 @@ err_pcie_cap: err_msi: msi_uninit(d); err_bridge: - tmp = pci_bridge_exitfn(d); - assert(!tmp); + pci_bridge_exitfn(d); return rc; } -static int xio3130_downstream_exitfn(PCIDevice *d) +static void xio3130_downstream_exitfn(PCIDevice *d) { PCIBridge* br = DO_UPCAST(PCIBridge, dev, d); PCIEPort *p = DO_UPCAST(PCIEPort, br, br); @@ -123,7 +121,7 @@ static int xio3130_downstream_exitfn(PCIDevice *d) pcie_chassis_del_slot(s); pcie_cap_exit(d); msi_uninit(d); - return pci_bridge_exitfn(d); + pci_bridge_exitfn(d); } PCIESlot *xio3130_downstream_init(PCIBus *bus, int devfn, bool multifunction, |