Messages in this thread Patch in this message | | | Date | Wed, 22 Oct 2008 16:42:41 +0800 | From | Yu Zhao <> | Subject | [PATCH 7/16 v6] PCI: cleanup pcibios_allocate_resources() |
| |
This cleanup makes pcibios_allocate_resources() easier to read.
Cc: Alex Chiang <achiang@hp.com> Cc: Grant Grundler <grundler@parisc-linux.org> Cc: Greg KH <greg@kroah.com> Cc: Ingo Molnar <mingo@elte.hu> Cc: Jesse Barnes <jbarnes@virtuousgeek.org> Cc: Matthew Wilcox <matthew@wil.cx> Cc: Randy Dunlap <randy.dunlap@oracle.com> Cc: Roland Dreier <rdreier@cisco.com> Signed-off-by: Yu Zhao <yu.zhao@intel.com>
--- arch/x86/pci/i386.c | 28 ++++++++++++++-------------- 1 files changed, 14 insertions(+), 14 deletions(-)
diff --git a/arch/x86/pci/i386.c b/arch/x86/pci/i386.c index 844df0c..8729bde 100644 --- a/arch/x86/pci/i386.c +++ b/arch/x86/pci/i386.c @@ -147,7 +147,7 @@ static void __init pcibios_allocate_bus_resources(struct list_head *bus_list) static void __init pcibios_allocate_resources(int pass) { struct pci_dev *dev = NULL; - int idx, disabled; + int idx, enabled; u16 command; struct resource *r, *pr; @@ -160,22 +160,22 @@ static void __init pcibios_allocate_resources(int pass) if (!r->start) /* Address not assigned at all */ continue; if (r->flags & IORESOURCE_IO) - disabled = !(command & PCI_COMMAND_IO); + enabled = command & PCI_COMMAND_IO; else - disabled = !(command & PCI_COMMAND_MEMORY); - if (pass == disabled) { - dev_dbg(&dev->dev, "resource %#08llx-%#08llx (f=%lx, d=%d, p=%d)\n", + enabled = command & PCI_COMMAND_MEMORY; + if (pass == enabled) + continue; + dev_dbg(&dev->dev, "resource %#08llx-%#08llx (f=%lx, d=%d, p=%d)\n", (unsigned long long) r->start, (unsigned long long) r->end, - r->flags, disabled, pass); - pr = pci_find_parent_resource(dev, r); - if (!pr || request_resource(pr, r) < 0) { - dev_err(&dev->dev, "BAR %d: can't allocate resource\n", idx); - /* We'll assign a new address later */ - r->end -= r->start; - r->start = 0; - } - } + r->flags, enabled, pass); + pr = pci_find_parent_resource(dev, r); + if (pr && !request_resource(pr, r)) + continue; + dev_err(&dev->dev, "BAR %d: can't allocate resource\n", idx); + /* We'll assign a new address later */ + r->end -= r->start; + r->start = 0; } if (!pass) { r = &dev->resource[PCI_ROM_RESOURCE]; -- 1.5.6.4
| |