PCI: Remove pci_bus_b() and use list_for_each_entry() directly
authorYijing Wang <wangyijing@huawei.com>
Thu, 13 Feb 2014 13:14:03 +0000 (21:14 +0800)
committerBjorn Helgaas <bhelgaas@google.com>
Fri, 14 Feb 2014 19:32:26 +0000 (12:32 -0700)
Replace list_for_each() with list_for_each_entry(), which means we no
longer need pci_bus_b() and can remove it.

Signed-off-by: Yijing Wang <wangyijing@huawei.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/pci.c
drivers/pci/search.c
include/linux/pci.h

index 1febe90831b442303b7414faec770d4850ca53ed..6f5ed88ca1267612891b7517d99e5353624025df 100644 (file)
@@ -108,12 +108,12 @@ static bool pcie_ari_disabled;
  */
 unsigned char pci_bus_max_busnr(struct pci_bus* bus)
 {
-       struct list_head *tmp;
+       struct pci_bus *tmp;
        unsigned char max, n;
 
        max = bus->busn_res.end;
-       list_for_each(tmp, &bus->children) {
-               n = pci_bus_max_busnr(pci_bus_b(tmp));
+       list_for_each_entry(tmp, &bus->children, node) {
+               n = pci_bus_max_busnr(tmp);
                if(n > max)
                        max = n;
        }
index 3ff2ac7c14e235c564b3b8d9c4d11609a4696fe5..4a1b972efe7f2fbb76c33467de665a16004fda37 100644 (file)
@@ -54,14 +54,14 @@ pci_find_upstream_pcie_bridge(struct pci_dev *pdev)
 
 static struct pci_bus *pci_do_find_bus(struct pci_bus *bus, unsigned char busnr)
 {
-       struct pci_buschild;
-       struct list_head *tmp;
+       struct pci_bus *child;
+       struct pci_bus *tmp;
 
        if(bus->number == busnr)
                return bus;
 
-       list_for_each(tmp, &bus->children) {
-               child = pci_do_find_bus(pci_bus_b(tmp), busnr);
+       list_for_each_entry(tmp, &bus->children, node) {
+               child = pci_do_find_bus(tmp, busnr);
                if(child)
                        return child;
        }
@@ -111,7 +111,7 @@ pci_find_next_bus(const struct pci_bus *from)
        down_read(&pci_bus_sem);
        n = from ? from->node.next : pci_root_buses.next;
        if (n != &pci_root_buses)
-               b = pci_bus_b(n);
+               b = list_entry(n, struct pci_bus, node);
        up_read(&pci_bus_sem);
        return b;
 }
index fb57c892b214d6b3482c63a28061130f3a700a9c..e1b5752f3b1bf09f425a6ff0f60dd2664a42024e 100644 (file)
@@ -461,7 +461,6 @@ struct pci_bus {
        unsigned int            is_added:1;
 };
 
-#define pci_bus_b(n)   list_entry(n, struct pci_bus, node)
 #define to_pci_bus(n)  container_of(n, struct pci_bus, dev)
 
 /*