PCI: replace struct pci_bus secondary/subordinate with busn_res

Replace the struct pci_bus secondary/subordinate members with the
struct resource busn_res.  Later we'll build a resource tree of these
bus numbers.

[bhelgaas: changelog]
Signed-off-by: Yinghai Lu <yinghai@kernel.org>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>

9 files changed, 59 insertions, 59 deletions

diff --git a/drivers/pci/hotplug/acpiphp_glue.c b/drivers/pci/hotplug/acpiphp_glue.c
index 806c44fa645a..62d0ae4dfcad 100644
--- a/drivers/pci/hotplug/acpiphp_glue.c
+++ b/drivers/pci/hotplug/acpiphp_glue.c
@@ -100,11 +100,11 @@ static int post_dock_fixups(struct notifier_block *nb, unsigned long val,
                         PCI_PRIMARY_BUS,
                         &buses);
 
-        if (((buses >> 8) & 0xff) != bus->secondary) {
+        if (((buses >> 8) & 0xff) != bus->busn_res.start) {
                 buses = (buses & 0xff000000)
                         | ((unsigned int)(bus->primary)     <<  0)
-                        | ((unsigned int)(bus->secondary)   <<  8)
-                        | ((unsigned int)(bus->subordinate) << 16);
+                        | ((unsigned int)(bus->busn_res.start)   <<  8)
+                        | ((unsigned int)(bus->busn_res.end) << 16);
                 pci_write_config_dword(bus->self, PCI_PRIMARY_BUS, buses);
         }
         return NOTIFY_OK;
@@ -692,7 +692,7 @@ static unsigned char acpiphp_max_busnr(struct pci_bus *bus)
          * bus->subordinate value because it could have
          * padding in it.
          */
-        max = bus->secondary;
+        max = bus->busn_res.start;
 
         list_for_each(tmp, &bus->children) {
                 n = pci_bus_max_busnr(pci_bus_b(tmp));
diff --git a/drivers/pci/hotplug/cpci_hotplug_pci.c b/drivers/pci/hotplug/cpci_hotplug_pci.c
index ae853ccd0cd5..42f3a61db87c 100644
--- a/drivers/pci/hotplug/cpci_hotplug_pci.c
+++ b/drivers/pci/hotplug/cpci_hotplug_pci.c
@@ -292,8 +292,8 @@ int __ref cpci_configure_slot(struct slot *slot)
                     (dev->hdr_type == PCI_HEADER_TYPE_CARDBUS)) {
                         /* Find an unused bus number for the new bridge */
                         struct pci_bus *child;
-                        unsigned char busnr, start = parent->secondary;
-                        unsigned char end = parent->subordinate;
+                        unsigned char busnr, start = parent->busn_res.start;
+                        unsigned char end = parent->busn_res.end;
 
                         for (busnr = start; busnr <= end; busnr++) {
                                 if (!pci_find_bus(pci_domain_nr(parent),
@@ -312,7 +312,7 @@ int __ref cpci_configure_slot(struct slot *slot)
                                 pci_dev_put(dev);
                                 continue;
                         }
-                        child->subordinate = pci_do_scan_bus(child);
+                        child->busn_res.end = pci_do_scan_bus(child);
                         pci_bus_size_bridges(child);
                 }
                 pci_dev_put(dev);
diff --git a/drivers/pci/hotplug/pciehp_pci.c b/drivers/pci/hotplug/pciehp_pci.c
index 47d9dc06b109..b898f06b588d 100644
--- a/drivers/pci/hotplug/pciehp_pci.c
+++ b/drivers/pci/hotplug/pciehp_pci.c
@@ -37,8 +37,8 @@
 static int __ref pciehp_add_bridge(struct pci_dev *dev)
 {
         struct pci_bus *parent = dev->bus;
-        int pass, busnr, start = parent->secondary;
-        int end = parent->subordinate;
+        int pass, busnr, start = parent->busn_res.start;
+        int end = parent->busn_res.end;
 
         for (busnr = start; busnr <= end; busnr++) {
                 if (!pci_find_bus(pci_domain_nr(parent), busnr))
diff --git a/drivers/pci/hotplug/shpchp_pci.c b/drivers/pci/hotplug/shpchp_pci.c
index df7e4bfadae3..d021eb031b3c 100644
--- a/drivers/pci/hotplug/shpchp_pci.c
+++ b/drivers/pci/hotplug/shpchp_pci.c
@@ -64,8 +64,8 @@ int __ref shpchp_configure_device(struct slot *p_slot)
                                 (dev->hdr_type == PCI_HEADER_TYPE_CARDBUS)) {
                         /* Find an unused bus number for the new bridge */
                         struct pci_bus *child;
-                        unsigned char busnr, start = parent->secondary;
-                        unsigned char end = parent->subordinate;
+                        unsigned char busnr, start = parent->busn_res.start;
+                        unsigned char end = parent->busn_res.end;
                         for (busnr = start; busnr <= end; busnr++) {
                                 if (!pci_find_bus(pci_domain_nr(parent),
                                                         busnr))
@@ -84,7 +84,7 @@ int __ref shpchp_configure_device(struct slot *p_slot)
                                 pci_dev_put(dev);
                                 continue;
                         }
-                        child->subordinate = pci_do_scan_bus(child);
+                        child->busn_res.end = pci_do_scan_bus(child);
                         pci_bus_size_bridges(child);
                 }
                 pci_configure_slot(dev);
diff --git a/drivers/pci/hotplug/shpchp_sysfs.c b/drivers/pci/hotplug/shpchp_sysfs.c
index efa30da1ae8f..eeb23ceae4a8 100644
--- a/drivers/pci/hotplug/shpchp_sysfs.c
+++ b/drivers/pci/hotplug/shpchp_sysfs.c
@@ -73,13 +73,13 @@ static ssize_t show_ctrl (struct device *dev, struct device_attribute *attr, cha
                 }
         }
         out += sprintf(out, "Free resources: bus numbers\n");
-        for (busnr = bus->secondary; busnr <= bus->subordinate; busnr++) {
+        for (busnr = bus->busn_res.start; busnr <= bus->busn_res.end; busnr++) {
                 if (!pci_find_bus(pci_domain_nr(bus), busnr))
                         break;
         }
-        if (busnr < bus->subordinate)
+        if (busnr < bus->busn_res.end)
                 out += sprintf(out, "start = %8.8x, length = %8.8x\n",
-                                busnr, (bus->subordinate - busnr));
+                                busnr, (int)(bus->busn_res.end - busnr));
 
         return out - buf;
 }
diff --git a/drivers/pci/iov.c b/drivers/pci/iov.c
index 6554e1a0f634..e873060fb35b 100644
--- a/drivers/pci/iov.c
+++ b/drivers/pci/iov.c
@@ -47,7 +47,7 @@ static struct pci_bus *virtfn_add_bus(struct pci_bus *bus, int busnr)
         if (!child)
                 return NULL;
 
-        child->subordinate = busnr;
+        child->busn_res.end = busnr;
         child->dev.parent = bus->bridge;
         rc = pci_bus_add_child(child);
         if (rc) {
@@ -327,7 +327,7 @@ static int sriov_enable(struct pci_dev *dev, int nr_virtfn)
         iov->offset = offset;
         iov->stride = stride;
 
-        if (virtfn_bus(dev, nr_virtfn - 1) > dev->bus->subordinate) {
+        if (virtfn_bus(dev, nr_virtfn - 1) > dev->bus->busn_res.end) {
                 dev_err(&dev->dev, "SR-IOV: bus number out of range\n");
                 return -ENOMEM;
         }
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index 447e83472c01..aeda6e9c245c 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -110,7 +110,7 @@ unsigned char pci_bus_max_busnr(struct pci_bus* bus)
         struct list_head *tmp;
         unsigned char max, n;
 
-        max = bus->subordinate;
+        max = bus->busn_res.end;
         list_for_each(tmp, &bus->children) {
                 n = pci_bus_max_busnr(pci_bus_b(tmp));
                 if(n > max)
diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c
index 658ac977cb56..651b096134dc 100644
--- a/drivers/pci/probe.c
+++ b/drivers/pci/probe.c
@@ -381,8 +381,8 @@ void __devinit pci_read_bridge_bases(struct pci_bus *child)
         if (pci_is_root_bus(child))     /* It's a host bus, nothing to read */
                 return;
 
-        dev_info(&dev->dev, "PCI bridge to [bus %02x-%02x]%s\n",
-                 child->secondary, child->subordinate,
+        dev_info(&dev->dev, "PCI bridge to %pR%s\n",
+                 &child->busn_res,
                  dev->transparent ? " (subtractive decode)" : "");
 
         pci_bus_remove_resources(child);
@@ -599,9 +599,9 @@ static struct pci_bus *pci_alloc_child_bus(struct pci_bus *parent,
          * Set up the primary, secondary and subordinate
          * bus numbers.
          */
-        child->number = child->secondary = busnr;
-        child->primary = parent->secondary;
-        child->subordinate = 0xff;
+        child->number = child->busn_res.start = busnr;
+        child->primary = parent->busn_res.start;
+        child->busn_res.end = 0xff;
 
         if (!bridge)
                 return child;
@@ -643,8 +643,8 @@ static void pci_fixup_parent_subordinate_busnr(struct pci_bus *child, int max)
         if (!pcibios_assign_all_busses())
                 return;
 
-        while (parent->parent && parent->subordinate < max) {
-                parent->subordinate = max;
+        while (parent->parent && parent->busn_res.end < max) {
+                parent->busn_res.end = max;
                 pci_write_config_byte(parent->self, PCI_SUBORDINATE_BUS, max);
                 parent = parent->parent;
         }
@@ -718,15 +718,15 @@ int __devinit pci_scan_bridge(struct pci_bus *bus, struct pci_dev *dev, int max,
                         if (!child)
                                 goto out;
                         child->primary = primary;
-                        child->subordinate = subordinate;
+                        child->busn_res.end = subordinate;
                         child->bridge_ctl = bctl;
                 }
 
                 cmax = pci_scan_child_bus(child);
                 if (cmax > max)
                         max = cmax;
-                if (child->subordinate > max)
-                        max = child->subordinate;
+                if (child->busn_res.end > max)
+                        max = child->busn_res.end;
         } else {
                 /*
                  * We need to assign a number to this bus which we always
@@ -759,8 +759,8 @@ int __devinit pci_scan_bridge(struct pci_bus *bus, struct pci_dev *dev, int max,
                 }
                 buses = (buses & 0xff000000)
                       | ((unsigned int)(child->primary)     <<  0)
-                      | ((unsigned int)(child->secondary)   <<  8)
-                      | ((unsigned int)(child->subordinate) << 16);
+                      | ((unsigned int)(child->busn_res.start)   <<  8)
+                      | ((unsigned int)(child->busn_res.end) << 16);
 
                 /*
                  * yenta.c forces a secondary latency timer of 176.
@@ -805,8 +805,8 @@ int __devinit pci_scan_bridge(struct pci_bus *bus, struct pci_dev *dev, int max,
                                         break;
                                 while (parent->parent) {
                                         if ((!pcibios_assign_all_busses()) &&
-                                            (parent->subordinate > max) &&
-                                            (parent->subordinate <= max+i)) {
+                                            (parent->busn_res.end > max) &&
+                                            (parent->busn_res.end <= max+i)) {
                                                 j = 1;
                                         }
                                         parent = parent->parent;
@@ -827,7 +827,7 @@ int __devinit pci_scan_bridge(struct pci_bus *bus, struct pci_dev *dev, int max,
                 /*
                  * Set the subordinate bus number to its real value.
                  */
-                child->subordinate = max;
+                child->busn_res.end = max;
                 pci_write_config_byte(dev, PCI_SUBORDINATE_BUS, max);
         }
 
@@ -837,19 +837,19 @@ int __devinit pci_scan_bridge(struct pci_bus *bus, struct pci_dev *dev, int max,
 
         /* Has only triggered on CardBus, fixup is in yenta_socket */
         while (bus->parent) {
-                if ((child->subordinate > bus->subordinate) ||
-                    (child->number > bus->subordinate) ||
+                if ((child->busn_res.end > bus->busn_res.end) ||
+                    (child->number > bus->busn_res.end) ||
                     (child->number < bus->number) ||
-                    (child->subordinate < bus->number)) {
-                        dev_info(&child->dev, "[bus %02x-%02x] %s "
-                                "hidden behind%s bridge %s [bus %02x-%02x]\n",
-                                child->number, child->subordinate,
-                                (bus->number > child->subordinate &&
-                                 bus->subordinate < child->number) ?
+                    (child->busn_res.end < bus->number)) {
+                        dev_info(&child->dev, "%pR %s "
+                                "hidden behind%s bridge %s %pR\n",
+                                &child->busn_res,
+                                (bus->number > child->busn_res.end &&
+                                 bus->busn_res.end < child->number) ?
                                         "wholly" : "partially",
                                 bus->self->transparent ? " transparent" : "",
                                 dev_name(&bus->dev),
-                                bus->number, bus->subordinate);
+                                &bus->busn_res);
                 }
                 bus = bus->parent;
         }
@@ -1548,7 +1548,7 @@ EXPORT_SYMBOL_GPL(pcie_bus_configure_settings);
 
 unsigned int __devinit pci_scan_child_bus(struct pci_bus *bus)
 {
-        unsigned int devfn, pass, max = bus->secondary;
+        unsigned int devfn, pass, max = bus->busn_res.start;
         struct pci_dev *dev;
 
         dev_dbg(&bus->dev, "scanning bus\n");
@@ -1642,7 +1642,7 @@ struct pci_bus *pci_create_root_bus(struct device *parent, int bus,
         /* Create legacy_io and legacy_mem files for this bus */
         pci_create_legacy_files(b);
 
-        b->number = b->secondary = bus;
+        b->number = b->busn_res.start = bus;
 
         if (parent)
                 dev_info(parent, "PCI host bridge to bus %s\n", dev_name(&b->dev));
@@ -1693,7 +1693,7 @@ struct pci_bus * __devinit pci_scan_root_bus(struct device *parent, int bus,
         if (!b)
                 return NULL;
 
-        b->subordinate = pci_scan_child_bus(b);
+        b->busn_res.end = pci_scan_child_bus(b);
         pci_bus_add_devices(b);
         return b;
 }
@@ -1710,7 +1710,7 @@ struct pci_bus * __devinit pci_scan_bus_parented(struct device *parent,
         pci_add_resource(&resources, &iomem_resource);
         b = pci_create_root_bus(parent, bus, ops, sysdata, &resources);
         if (b)
-                b->subordinate = pci_scan_child_bus(b);
+                b->busn_res.end = pci_scan_child_bus(b);
         else
                 pci_free_resource_list(&resources);
         return b;
@@ -1727,7 +1727,7 @@ struct pci_bus * __devinit pci_scan_bus(int bus, struct pci_ops *ops,
         pci_add_resource(&resources, &iomem_resource);
         b = pci_create_root_bus(NULL, bus, ops, sysdata, &resources);
         if (b) {
-                b->subordinate = pci_scan_child_bus(b);
+                b->busn_res.end = pci_scan_child_bus(b);
                 pci_bus_add_devices(b);
         } else {
                 pci_free_resource_list(&resources);
diff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c
index 8fa2d4be88de..192172c87b77 100644
--- a/drivers/pci/setup-bus.c
+++ b/drivers/pci/setup-bus.c
@@ -404,8 +404,8 @@ void pci_setup_cardbus(struct pci_bus *bus)
         struct resource *res;
         struct pci_bus_region region;
 
-        dev_info(&bridge->dev, "CardBus bridge to [bus %02x-%02x]\n",
-                 bus->secondary, bus->subordinate);
+        dev_info(&bridge->dev, "CardBus bridge to %pR\n",
+                 &bus->busn_res);
 
         res = bus->resource[0];
         pcibios_resource_to_bus(bridge, &region, res);
@@ -553,8 +553,8 @@ static void __pci_setup_bridge(struct pci_bus *bus, unsigned long type)
 {
         struct pci_dev *bridge = bus->self;
 
-        dev_info(&bridge->dev, "PCI bridge to [bus %02x-%02x]\n",
-                 bus->secondary, bus->subordinate);
+        dev_info(&bridge->dev, "PCI bridge to %pR\n",
+                 &bus->busn_res);
 
         if (type & IORESOURCE_IO)
                 pci_setup_bridge_io(bus);
@@ -745,8 +745,8 @@ static void pbus_size_io(struct pci_bus *bus, resource_size_t min_size,
         if (!size0 && !size1) {
                 if (b_res->start || b_res->end)
                         dev_info(&bus->self->dev, "disabling bridge window "
-                                 "%pR to [bus %02x-%02x] (unused)\n", b_res,
-                                 bus->secondary, bus->subordinate);
+                                 "%pR to %pR (unused)\n", b_res,
+                                 &bus->busn_res);
                 b_res->flags = 0;
                 return;
         }
@@ -757,8 +757,8 @@ static void pbus_size_io(struct pci_bus *bus, resource_size_t min_size,
         if (size1 > size0 && realloc_head) {
                 add_to_list(realloc_head, bus->self, b_res, size1-size0, 4096);
                 dev_printk(KERN_DEBUG, &bus->self->dev, "bridge window "
-                                 "%pR to [bus %02x-%02x] add_size %lx\n", b_res,
-                                 bus->secondary, bus->subordinate, size1-size0);
+                                 "%pR to %pR add_size %lx\n", b_res,
+                                 &bus->busn_res, size1-size0);
         }
 }
 
@@ -863,8 +863,8 @@ static int pbus_size_mem(struct pci_bus *bus, unsigned long mask,
         if (!size0 && !size1) {
                 if (b_res->start || b_res->end)
                         dev_info(&bus->self->dev, "disabling bridge window "
-                                 "%pR to [bus %02x-%02x] (unused)\n", b_res,
-                                 bus->secondary, bus->subordinate);
+                                 "%pR to %pR (unused)\n", b_res,
+                                 &bus->busn_res);
                 b_res->flags = 0;
                 return 1;
         }
@@ -874,8 +874,8 @@ static int pbus_size_mem(struct pci_bus *bus, unsigned long mask,
         if (size1 > size0 && realloc_head) {
                 add_to_list(realloc_head, bus->self, b_res, size1-size0, min_align);
                 dev_printk(KERN_DEBUG, &bus->self->dev, "bridge window "
-                                 "%pR to [bus %02x-%02x] add_size %llx\n", b_res,
-                                 bus->secondary, bus->subordinate, (unsigned long long)size1-size0);
+                                 "%pR to %pR add_size %llx\n", b_res,
+                                 &bus->busn_res, (unsigned long long)size1-size0);
         }
         return 1;
 }