diff options
| author | Matthew Wilcox <willy@linux.intel.com> | 2009-06-17 16:33:33 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-06-17 14:04:42 -0700 |
| commit | cebd78a8c5624b5cf04c39c3335a5fc8670a7b69 (patch) | |
| tree | 426b2ef97ee950e11e231fff1286ae06cf1f3423 /drivers | |
| parent | 7cc47662197b3050a974a6c6aed52291bb38ded0 (diff) | |
| download | linux-cebd78a8c5624b5cf04c39c3335a5fc8670a7b69.tar.gz linux-cebd78a8c5624b5cf04c39c3335a5fc8670a7b69.tar.bz2 linux-cebd78a8c5624b5cf04c39c3335a5fc8670a7b69.zip | |
Fix pci_claim_resource
Instead of starting from the iomem or ioport roots, start from the
parent bus' resources. This fixes a bug where child resources would
appear above their parents resources if they had the same size.
Signed-off-by: Matthew Wilcox <willy@linux.intel.com>
Tested-by: Andrew Patterson <andrew.patterson@hp.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/pci/setup-res.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/pci/setup-res.c b/drivers/pci/setup-res.c index 3039fcb86afc..12403516776a 100644 --- a/drivers/pci/setup-res.c +++ b/drivers/pci/setup-res.c @@ -99,11 +99,11 @@ void pci_update_resource(struct pci_dev *dev, int resno) int pci_claim_resource(struct pci_dev *dev, int resource) { struct resource *res = &dev->resource[resource]; - struct resource *root = NULL; + struct resource *root; char *dtype = resource < PCI_BRIDGE_RESOURCES ? "device" : "bridge"; int err; - root = pcibios_select_root(dev, res); + root = pci_find_parent_resource(dev, res); err = -EINVAL; if (root != NULL) |