diff options
author | Yijing Wang <wangyijing@huawei.com> | 2014-05-29 23:01:03 -0400 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2014-05-30 11:34:27 -0400 |
commit | c893d133eaccdda2516a3e71cd05a7dac2e14b00 (patch) | |
tree | 2f296b5a88e4e794fb48d4c22c8108ce6f671bbc | |
parent | d97ffe236894856d08146390ef3fbe6448a8ac2b (diff) |
PCI: Make pci_bus_add_device() void
pci_bus_add_device() always returns 0, so there's no point in returning
anything at all. Make it a void function and remove the tests of the
return value from the callers.
[bhelgaas: changelog, remove unused "err" from i82875p_setup_overfl_dev()]
Signed-off-by: Yijing Wang <wangyijing@huawei.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
-rw-r--r-- | drivers/edac/i82875p_edac.c | 8 | ||||
-rw-r--r-- | drivers/pci/bus.c | 10 | ||||
-rw-r--r-- | drivers/pci/iov.c | 2 | ||||
-rw-r--r-- | drivers/platform/x86/asus-wmi.c | 3 | ||||
-rw-r--r-- | drivers/platform/x86/eeepc-laptop.c | 3 | ||||
-rw-r--r-- | include/linux/pci.h | 2 |
6 files changed, 7 insertions, 21 deletions
diff --git a/drivers/edac/i82875p_edac.c b/drivers/edac/i82875p_edac.c index 8d0450b9b9af..4009077c8839 100644 --- a/drivers/edac/i82875p_edac.c +++ b/drivers/edac/i82875p_edac.c | |||
@@ -275,7 +275,6 @@ static int i82875p_setup_overfl_dev(struct pci_dev *pdev, | |||
275 | { | 275 | { |
276 | struct pci_dev *dev; | 276 | struct pci_dev *dev; |
277 | void __iomem *window; | 277 | void __iomem *window; |
278 | int err; | ||
279 | 278 | ||
280 | *ovrfl_pdev = NULL; | 279 | *ovrfl_pdev = NULL; |
281 | *ovrfl_window = NULL; | 280 | *ovrfl_window = NULL; |
@@ -293,12 +292,7 @@ static int i82875p_setup_overfl_dev(struct pci_dev *pdev, | |||
293 | if (dev == NULL) | 292 | if (dev == NULL) |
294 | return 1; | 293 | return 1; |
295 | 294 | ||
296 | err = pci_bus_add_device(dev); | 295 | pci_bus_add_device(dev); |
297 | if (err) { | ||
298 | i82875p_printk(KERN_ERR, | ||
299 | "%s(): pci_bus_add_device() Failed\n", | ||
300 | __func__); | ||
301 | } | ||
302 | pci_bus_assign_resources(dev->bus); | 296 | pci_bus_assign_resources(dev->bus); |
303 | } | 297 | } |
304 | 298 | ||
diff --git a/drivers/pci/bus.c b/drivers/pci/bus.c index ba2bf55a38df..447d393725e1 100644 --- a/drivers/pci/bus.c +++ b/drivers/pci/bus.c | |||
@@ -235,7 +235,7 @@ void __weak pcibios_resource_survey_bus(struct pci_bus *bus) { } | |||
235 | * | 235 | * |
236 | * This adds add sysfs entries and start device drivers | 236 | * This adds add sysfs entries and start device drivers |
237 | */ | 237 | */ |
238 | int pci_bus_add_device(struct pci_dev *dev) | 238 | void pci_bus_add_device(struct pci_dev *dev) |
239 | { | 239 | { |
240 | int retval; | 240 | int retval; |
241 | 241 | ||
@@ -252,8 +252,6 @@ int pci_bus_add_device(struct pci_dev *dev) | |||
252 | WARN_ON(retval < 0); | 252 | WARN_ON(retval < 0); |
253 | 253 | ||
254 | dev->is_added = 1; | 254 | dev->is_added = 1; |
255 | |||
256 | return 0; | ||
257 | } | 255 | } |
258 | 256 | ||
259 | /** | 257 | /** |
@@ -266,16 +264,12 @@ void pci_bus_add_devices(const struct pci_bus *bus) | |||
266 | { | 264 | { |
267 | struct pci_dev *dev; | 265 | struct pci_dev *dev; |
268 | struct pci_bus *child; | 266 | struct pci_bus *child; |
269 | int retval; | ||
270 | 267 | ||
271 | list_for_each_entry(dev, &bus->devices, bus_list) { | 268 | list_for_each_entry(dev, &bus->devices, bus_list) { |
272 | /* Skip already-added devices */ | 269 | /* Skip already-added devices */ |
273 | if (dev->is_added) | 270 | if (dev->is_added) |
274 | continue; | 271 | continue; |
275 | retval = pci_bus_add_device(dev); | 272 | pci_bus_add_device(dev); |
276 | if (retval) | ||
277 | dev_err(&dev->dev, "Error adding device (%d)\n", | ||
278 | retval); | ||
279 | } | 273 | } |
280 | 274 | ||
281 | list_for_each_entry(dev, &bus->devices, bus_list) { | 275 | list_for_each_entry(dev, &bus->devices, bus_list) { |
diff --git a/drivers/pci/iov.c b/drivers/pci/iov.c index de7a74782f92..cb6f24740ee3 100644 --- a/drivers/pci/iov.c +++ b/drivers/pci/iov.c | |||
@@ -106,7 +106,7 @@ static int virtfn_add(struct pci_dev *dev, int id, int reset) | |||
106 | pci_device_add(virtfn, virtfn->bus); | 106 | pci_device_add(virtfn, virtfn->bus); |
107 | mutex_unlock(&iov->dev->sriov->lock); | 107 | mutex_unlock(&iov->dev->sriov->lock); |
108 | 108 | ||
109 | rc = pci_bus_add_device(virtfn); | 109 | pci_bus_add_device(virtfn); |
110 | sprintf(buf, "virtfn%u", id); | 110 | sprintf(buf, "virtfn%u", id); |
111 | rc = sysfs_create_link(&dev->dev.kobj, &virtfn->dev.kobj, buf); | 111 | rc = sysfs_create_link(&dev->dev.kobj, &virtfn->dev.kobj, buf); |
112 | if (rc) | 112 | if (rc) |
diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c index c5e082fb82fa..91ef69a52263 100644 --- a/drivers/platform/x86/asus-wmi.c +++ b/drivers/platform/x86/asus-wmi.c | |||
@@ -642,8 +642,7 @@ static void asus_rfkill_hotplug(struct asus_wmi *asus) | |||
642 | dev = pci_scan_single_device(bus, 0); | 642 | dev = pci_scan_single_device(bus, 0); |
643 | if (dev) { | 643 | if (dev) { |
644 | pci_bus_assign_resources(bus); | 644 | pci_bus_assign_resources(bus); |
645 | if (pci_bus_add_device(dev)) | 645 | pci_bus_add_device(dev); |
646 | pr_err("Unable to hotplug wifi\n"); | ||
647 | } | 646 | } |
648 | } else { | 647 | } else { |
649 | dev = pci_get_slot(bus, 0); | 648 | dev = pci_get_slot(bus, 0); |
diff --git a/drivers/platform/x86/eeepc-laptop.c b/drivers/platform/x86/eeepc-laptop.c index 399e8c562192..9b0c57cd1d4a 100644 --- a/drivers/platform/x86/eeepc-laptop.c +++ b/drivers/platform/x86/eeepc-laptop.c | |||
@@ -633,8 +633,7 @@ static void eeepc_rfkill_hotplug(struct eeepc_laptop *eeepc, acpi_handle handle) | |||
633 | dev = pci_scan_single_device(bus, 0); | 633 | dev = pci_scan_single_device(bus, 0); |
634 | if (dev) { | 634 | if (dev) { |
635 | pci_bus_assign_resources(bus); | 635 | pci_bus_assign_resources(bus); |
636 | if (pci_bus_add_device(dev)) | 636 | pci_bus_add_device(dev); |
637 | pr_err("Unable to hotplug wifi\n"); | ||
638 | } | 637 | } |
639 | } else { | 638 | } else { |
640 | dev = pci_get_slot(bus, 0); | 639 | dev = pci_get_slot(bus, 0); |
diff --git a/include/linux/pci.h b/include/linux/pci.h index 322335aaa7e1..785149a6aec1 100644 --- a/include/linux/pci.h +++ b/include/linux/pci.h | |||
@@ -764,7 +764,7 @@ int pci_scan_slot(struct pci_bus *bus, int devfn); | |||
764 | struct pci_dev *pci_scan_single_device(struct pci_bus *bus, int devfn); | 764 | struct pci_dev *pci_scan_single_device(struct pci_bus *bus, int devfn); |
765 | void pci_device_add(struct pci_dev *dev, struct pci_bus *bus); | 765 | void pci_device_add(struct pci_dev *dev, struct pci_bus *bus); |
766 | unsigned int pci_scan_child_bus(struct pci_bus *bus); | 766 | unsigned int pci_scan_child_bus(struct pci_bus *bus); |
767 | int __must_check pci_bus_add_device(struct pci_dev *dev); | 767 | void pci_bus_add_device(struct pci_dev *dev); |
768 | void pci_read_bridge_bases(struct pci_bus *child); | 768 | void pci_read_bridge_bases(struct pci_bus *child); |
769 | struct resource *pci_find_parent_resource(const struct pci_dev *dev, | 769 | struct resource *pci_find_parent_resource(const struct pci_dev *dev, |
770 | struct resource *res); | 770 | struct resource *res); |