x86/PCI: Drop return value of pcibios_scan_root()
authorBjorn Helgaas <bhelgaas@google.com>
Tue, 28 Jan 2014 23:40:36 +0000 (16:40 -0700)
committerBjorn Helgaas <bhelgaas@google.com>
Mon, 3 Feb 2014 17:38:29 +0000 (10:38 -0700)
Nobody really uses the return value of pcibios_scan_root() (one place uses
it to control a printk, but the printk is not very useful).  This converts
pcibios_scan_root() to a void function.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
arch/x86/include/asm/pci.h
arch/x86/pci/common.c
arch/x86/pci/irq.c

index b52f9c0ec5fbd23bf3fc3a4b1ea51c5b2dc6bf23..96ae4f4040bb359f8f3588f82f6595e41edbc501 100644 (file)
@@ -65,7 +65,7 @@ extern unsigned long pci_mem_start;
 
 extern int pcibios_enabled;
 void pcibios_config_init(void);
-struct pci_bus *pcibios_scan_root(int bus);
+void pcibios_scan_root(int bus);
 
 void pcibios_set_master(struct pci_dev *dev);
 void pcibios_penalize_isa_irq(int irq, int active);
index 52ad00c2e1987fc3b9862743236f7c6758144cdb..600e5925f1a40a3fbb2868ee06422bd478d0386c 100644 (file)
@@ -456,7 +456,7 @@ void __init dmi_check_pciprobe(void)
        dmi_check_system(pciprobe_dmi_table);
 }
 
-struct pci_bus *pcibios_scan_root(int busnum)
+void pcibios_scan_root(int busnum)
 {
        struct pci_bus *bus;
        struct pci_sysdata *sd;
@@ -465,7 +465,7 @@ struct pci_bus *pcibios_scan_root(int busnum)
        sd = kzalloc(sizeof(*sd), GFP_KERNEL);
        if (!sd) {
                printk(KERN_ERR "PCI: OOM, skipping PCI bus %02x\n", busnum);
-               return NULL;
+               return;
        }
        sd->node = get_mp_bus_to_node(busnum);
        x86_pci_root_bus_resources(busnum, &resources);
@@ -475,8 +475,6 @@ struct pci_bus *pcibios_scan_root(int busnum)
                pci_free_resource_list(&resources);
                kfree(sd);
        }
-
-       return bus;
 }
 
 void __init pcibios_set_cache_line_size(void)
index 8658874165c276124f870c98518d95b0afd26a6c..84112f55dd7a95d83232d9416b7e9196a55dee30 100644 (file)
@@ -138,9 +138,7 @@ static void __init pirq_peer_trick(void)
        for (i = 1; i < 256; i++) {
                if (!busmap[i] || pci_find_bus(0, i))
                        continue;
-               if (pcibios_scan_root(i))
-                       printk(KERN_INFO "PCI: Discovered primary peer "
-                              "bus %02x [IRQ]\n", i);
+               pcibios_scan_root(i);
        }
        pcibios_last_bus = -1;
 }