This cleanup makes pcibios_allocate_resources() easier to read.

Cc: Alex Chiang <[EMAIL PROTECTED]>
Cc: Grant Grundler <[EMAIL PROTECTED]>
Cc: Greg KH <[EMAIL PROTECTED]>
Cc: Ingo Molnar <[EMAIL PROTECTED]>
Cc: Jesse Barnes <[EMAIL PROTECTED]>
Cc: Matthew Wilcox <[EMAIL PROTECTED]>
Cc: Randy Dunlap <[EMAIL PROTECTED]>
Cc: Roland Dreier <[EMAIL PROTECTED]>
Signed-off-by: Yu Zhao <[EMAIL PROTECTED]>

---
 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

_______________________________________________
Virtualization mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/virtualization

Reply via email to