diff options
| author | Stephen Hemminger <shemminger@vyatta.com> | 2008-09-06 06:45:36 +0200 |
|---|---|---|
| committer | Jesse Barnes <jbarnes@virtuousgeek.org> | 2008-10-20 19:54:25 +0200 |
| commit | 11d587429e9cbb40ac20d7ed8126c66da0d7aba5 (patch) | |
| tree | bad00f92efe41f10d32c080fe82c950736853103 | |
| parent | PCI: fix MSI-HOWTO.txt info about MSI-X MMIO space (diff) | |
| download | linux-11d587429e9cbb40ac20d7ed8126c66da0d7aba5.tar.xz linux-11d587429e9cbb40ac20d7ed8126c66da0d7aba5.zip | |
PCI: fix sparse warning in pci_remove_behind_bridge
Get rid of the second definition of dev which hides the earlier one in
the argument list and causes a warning from sparse.
Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
| -rw-r--r-- | drivers/pci/remove.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/drivers/pci/remove.c b/drivers/pci/remove.c index f94f6d5ae297..042e08924421 100644 --- a/drivers/pci/remove.c +++ b/drivers/pci/remove.c @@ -115,13 +115,9 @@ void pci_remove_behind_bridge(struct pci_dev *dev) { struct list_head *l, *n; - if (dev->subordinate) { - list_for_each_safe(l, n, &dev->subordinate->devices) { - struct pci_dev *dev = pci_dev_b(l); - - pci_remove_bus_device(dev); - } - } + if (dev->subordinate) + list_for_each_safe(l, n, &dev->subordinate->devices) + pci_remove_bus_device(pci_dev_b(l)); } static void pci_stop_bus_devices(struct pci_bus *bus) |