From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755493Ab2HGQTA (ORCPT ); Tue, 7 Aug 2012 12:19:00 -0400 Received: from mail-gg0-f174.google.com ([209.85.161.174]:40380 "EHLO mail-gg0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753611Ab2HGQS6 (ORCPT ); Tue, 7 Aug 2012 12:18:58 -0400 From: Jiang Liu To: Bjorn Helgaas , Don Dutile , Yinghai Lu , Greg KH , Kenji Kaneshige Cc: Jiang Liu , Taku Izumi , "Rafael J . Wysocki" , Yijing Wang , Xinwei Hu , linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, Jiang Liu Subject: [RFC PATCH v1 02/22] PCI: trivial cleanups for drivers/pci/remove.c Date: Wed, 8 Aug 2012 00:10:42 +0800 Message-Id: <1344355862-2726-3-git-send-email-jiang.liu@huawei.com> X-Mailer: git-send-email 1.7.9.5 In-Reply-To: <1344355862-2726-1-git-send-email-jiang.liu@huawei.com> References: <1344355862-2726-1-git-send-email-jiang.liu@huawei.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Trivial cleanups for drivers/pci/remove.c: 1) move the comment for pci_stop_and_remove_bus_device() to the right place 2) rename __pci_remove_behind_bridge() to pci_remove_behind_bridge() Signed-off-by: Jiang Liu --- drivers/pci/remove.c | 33 +++++++++++++++++---------------- 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/drivers/pci/remove.c b/drivers/pci/remove.c index 04a4861..33b6318 100644 --- a/drivers/pci/remove.c +++ b/drivers/pci/remove.c @@ -78,25 +78,14 @@ void pci_remove_bus(struct pci_bus *pci_bus) } EXPORT_SYMBOL(pci_remove_bus); -static void __pci_remove_behind_bridge(struct pci_dev *dev); -/** - * pci_stop_and_remove_bus_device - remove a PCI device and any children - * @dev: the device to remove - * - * Remove a PCI device from the device lists, informing the drivers - * that the device has been removed. We also remove any subordinate - * buses and children in a depth-first manner. - * - * For each device we remove, delete the device structure from the - * device lists, remove the /proc entry, and notify userspace - * (/sbin/hotplug). - */ +static void pci_remove_behind_bridge(struct pci_dev *dev); + void __pci_remove_bus_device(struct pci_dev *dev) { if (dev->subordinate) { struct pci_bus *b = dev->subordinate; - __pci_remove_behind_bridge(dev); + pci_remove_behind_bridge(dev); pci_remove_bus(b); dev->subordinate = NULL; } @@ -105,13 +94,25 @@ void __pci_remove_bus_device(struct pci_dev *dev) } EXPORT_SYMBOL(__pci_remove_bus_device); +/** + * pci_stop_and_remove_bus_device - remove a PCI device and any children + * @dev: the device to remove + * + * Remove a PCI device from the device lists, informing the drivers + * that the device has been removed. We also remove any subordinate + * buses and children in a depth-first manner. + * + * For each device we remove, delete the device structure from the + * device lists, remove the /proc entry, and notify userspace + * (/sbin/hotplug). + */ void pci_stop_and_remove_bus_device(struct pci_dev *dev) { pci_stop_bus_device(dev); __pci_remove_bus_device(dev); } -static void __pci_remove_behind_bridge(struct pci_dev *dev) +static void pci_remove_behind_bridge(struct pci_dev *dev) { struct list_head *l, *n; @@ -141,7 +142,7 @@ static void pci_stop_behind_bridge(struct pci_dev *dev) void pci_stop_and_remove_behind_bridge(struct pci_dev *dev) { pci_stop_behind_bridge(dev); - __pci_remove_behind_bridge(dev); + pci_remove_behind_bridge(dev); } static void pci_stop_bus_devices(struct pci_bus *bus) -- 1.7.9.5