diff options
author | Stephen Hemminger <shemminger@vyatta.com> | 2008-09-06 00:45:36 -0400 |
---|---|---|
committer | Jesse Barnes <jbarnes@virtuousgeek.org> | 2008-10-20 13:54:25 -0400 |
commit | 11d587429e9cbb40ac20d7ed8126c66da0d7aba5 (patch) | |
tree | bad00f92efe41f10d32c080fe82c950736853103 | |
parent | 50cbfa511a21cac1909b6b4c955fa39d1da81457 (diff) |
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) | |||
115 | { | 115 | { |
116 | struct list_head *l, *n; | 116 | struct list_head *l, *n; |
117 | 117 | ||
118 | if (dev->subordinate) { | 118 | if (dev->subordinate) |
119 | list_for_each_safe(l, n, &dev->subordinate->devices) { | 119 | list_for_each_safe(l, n, &dev->subordinate->devices) |
120 | struct pci_dev *dev = pci_dev_b(l); | 120 | pci_remove_bus_device(pci_dev_b(l)); |
121 | |||
122 | pci_remove_bus_device(dev); | ||
123 | } | ||
124 | } | ||
125 | } | 121 | } |
126 | 122 | ||
127 | static void pci_stop_bus_devices(struct pci_bus *bus) | 123 | static void pci_stop_bus_devices(struct pci_bus *bus) |