From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39666) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XLqNI-0003NJ-NU for qemu-devel@nongnu.org; Mon, 25 Aug 2014 05:15:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XLqN8-0002QS-CZ for qemu-devel@nongnu.org; Mon, 25 Aug 2014 05:15:40 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:48345) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XLqN7-0002QC-Mw for qemu-devel@nongnu.org; Mon, 25 Aug 2014 05:15:30 -0400 Message-ID: <1408958112.14053.192.camel@abi.no.oracle.com> From: Knut Omang Date: Mon, 25 Aug 2014 11:15:12 +0200 In-Reply-To: <33183CC9F5247A488A2544077AF1902086D6E203@SZXEMA503-MBS.china.huawei.com> References: <1408887140-3320-1-git-send-email-knut.omang@oracle.com> <1408887140-3320-4-git-send-email-knut.omang@oracle.com> <33183CC9F5247A488A2544077AF1902086D6E203@SZXEMA503-MBS.china.huawei.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 3/4] ioh3420: Remove obsoleted, unused ioh3420_init function List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Gonglei (Arei)" Cc: Marcel Apfelbaum , Alexey Kardashevskiy , Juan Quintela , Markus Armbruster , "qemu-devel@nongnu.org" , "Michael S. Tsirkin" , Igor Mammedov , Paolo Bonzini On Mon, 2014-08-25 at 07:47 +0000, Gonglei (Arei) wrote: > > -----Original Message----- > > From: Knut Omang [mailto:knut.omang@oracle.com] > > Sent: Sunday, August 24, 2014 9:32 PM > > To: qemu-devel@nongnu.org > > Cc: Michael S. Tsirkin; Alexey Kardashevskiy; Juan Quintela; Marcel Apfelbaum; > > Markus Armbruster; Paolo Bonzini; Gonglei (Arei); Igor Mammedov; Knut > > Omang > > Subject: [PATCH v2 3/4] ioh3420: Remove obsoleted, unused ioh3420_init > > function > > > > Signed-off-by: Knut Omang > > --- > > hw/pci-bridge/ioh3420.c | 24 ------------------------ > > 1 file changed, 24 deletions(-) > > > > diff --git a/hw/pci-bridge/ioh3420.c b/hw/pci-bridge/ioh3420.c > > index aed2bf1..e6674a1 100644 > > --- a/hw/pci-bridge/ioh3420.c > > +++ b/hw/pci-bridge/ioh3420.c > > @@ -157,30 +157,6 @@ static void ioh3420_exitfn(PCIDevice *d) > > pci_bridge_exitfn(d); > > } > > > > -PCIESlot *ioh3420_init(PCIBus *bus, int devfn, bool multifunction, > > - const char *bus_name, pci_map_irq_fn > > map_irq, > > - uint8_t port, uint8_t chassis, uint16_t slot) > > -{ > > - PCIDevice *d; > > - PCIBridge *br; > > - DeviceState *qdev; > > - > > - d = pci_create_multifunction(bus, devfn, multifunction, "ioh3420"); > > - if (!d) { > > - return NULL; > > - } > > - br = PCI_BRIDGE(d); > > - > > - qdev = DEVICE(d); > > - pci_bridge_map_irq(br, bus_name, map_irq); > > - qdev_prop_set_uint8(qdev, "port", port); > > - qdev_prop_set_uint8(qdev, "chassis", chassis); > > - qdev_prop_set_uint16(qdev, "slot", slot); > > - qdev_init_nofail(qdev); > > - > > - return PCIE_SLOT(d); > > -} > > - > > static Property ioh3420_props[] = { > > DEFINE_PROP_BIT(COMPAT_PROP_PCP, PCIDevice, cap_present, > > QEMU_PCIE_SLTCAP_PCP_BITNR, true), > > -- > > 1.9.0 > > That's OK. But the declaration of header file should also be removed. > I noticed MST have pulled into master tree. So, maybe I can post > another patch to fix this trivial problem. Yes, definitely, an obvious mistake from me, sorry. Don't understand how I could have overlooked that.. Knut > Best regards, > -Gonglei