[SCSI] hpsa: factor out hpsa_find_memory_BAR
authorStephen M. Cameron <scameron@beardog.cce.hp.com>
Thu, 27 May 2010 20:13:07 +0000 (15:13 -0500)
committerJames Bottomley <James.Bottomley@suse.de>
Tue, 27 Jul 2010 17:01:05 +0000 (12:01 -0500)
Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/hpsa.c

index 66b7dcfb7276ef96cdbb5a2ec0d1229ac97533a9..0582f2fc11f0ffbc03a4441f4e95e8f3b3cf731d 100644 (file)
@@ -3285,6 +3285,23 @@ static inline bool hpsa_board_disabled(struct pci_dev *pdev)
        return ((command & PCI_COMMAND_MEMORY) == 0);
 }
 
+static int __devinit hpsa_pci_find_memory_BAR(struct ctlr_info *h,
+       unsigned long *memory_bar)
+{
+       int i;
+
+       for (i = 0; i < DEVICE_COUNT_RESOURCE; i++)
+               if (pci_resource_flags(h->pdev, i) & IORESOURCE_MEM) {
+                       /* addressing mode bits already removed */
+                       *memory_bar = pci_resource_start(h->pdev, i);
+                       dev_dbg(&h->pdev->dev, "memory BAR = %lx\n",
+                               *memory_bar);
+                       return 0;
+               }
+       dev_warn(&h->pdev->dev, "no memory BAR found\n");
+       return -ENODEV;
+}
+
 static int __devinit hpsa_pci_init(struct ctlr_info *h)
 {
        u32 scratchpad = 0;
@@ -3317,22 +3334,9 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h)
                return err;
        }
        hpsa_interrupt_mode(h);
-
-       /* find the memory BAR */
-       for (i = 0; i < DEVICE_COUNT_RESOURCE; i++) {
-               if (pci_resource_flags(h->pdev, i) & IORESOURCE_MEM)
-                       break;
-       }
-       if (i == DEVICE_COUNT_RESOURCE) {
-               dev_warn(&h->pdev->dev, "no memory BAR found\n");
-               err = -ENODEV;
+       err = hpsa_pci_find_memory_BAR(h, &h->paddr);
+       if (err)
                goto err_out_free_res;
-       }
-
-       h->paddr = pci_resource_start(h->pdev, i); /* addressing mode bits
-                                                * already removed
-                                                */
-
        h->vaddr = remap_pci_mem(h->paddr, 0x250);
 
        /* Wait for the board to become ready.  */