[SCSI] hpsa: hpsa factor out hpsa_find_board_params
authorStephen M. Cameron <scameron@beardog.cce.hp.com>
Thu, 27 May 2010 20:13:27 +0000 (15:13 -0500)
committerJames Bottomley <James.Bottomley@suse.de>
Tue, 27 Jul 2010 17:01:08 +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 4983f34..dcbe54b 100644 (file)
@@ -3348,6 +3348,30 @@ static int __devinit hpsa_find_cfgtables(struct ctlr_info *h)
        return 0;
 }
 
+/* Interrogate the hardware for some limits:
+ * max commands, max SG elements without chaining, and with chaining,
+ * SG chain block size, etc.
+ */
+static void __devinit hpsa_find_board_params(struct ctlr_info *h)
+{
+       h->max_commands = readl(&(h->cfgtable->MaxPerformantModeCommands));
+       h->nr_cmds = h->max_commands - 4; /* Allow room for some ioctls */
+       h->maxsgentries = readl(&(h->cfgtable->MaxScatterGatherElements));
+       /*
+        * Limit in-command s/g elements to 32 save dma'able memory.
+        * Howvever spec says if 0, use 31
+        */
+       h->max_cmd_sg_entries = 31;
+       if (h->maxsgentries > 512) {
+               h->max_cmd_sg_entries = 32;
+               h->chainsize = h->maxsgentries - h->max_cmd_sg_entries + 1;
+               h->maxsgentries--; /* save one for chain pointer */
+       } else {
+               h->maxsgentries = 31; /* default to traditional values */
+               h->chainsize = 0;
+       }
+}
+
 static int __devinit hpsa_pci_init(struct ctlr_info *h)
 {
        int i, prod_index, err;
@@ -3389,27 +3413,7 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h)
        err = hpsa_find_cfgtables(h);
        if (err)
                goto err_out_free_res;
-
-       h->max_commands = readl(&(h->cfgtable->MaxPerformantModeCommands));
-       h->maxsgentries = readl(&(h->cfgtable->MaxScatterGatherElements));
-
-       /*
-        * Limit in-command s/g elements to 32 save dma'able memory.
-        * Howvever spec says if 0, use 31
-        */
-
-       h->max_cmd_sg_entries = 31;
-       if (h->maxsgentries > 512) {
-               h->max_cmd_sg_entries = 32;
-               h->chainsize = h->maxsgentries - h->max_cmd_sg_entries + 1;
-               h->maxsgentries--; /* save one for chain pointer */
-       } else {
-               h->maxsgentries = 31; /* default to traditional values */
-               h->chainsize = 0;
-       }
-
-       /* Allow room for some ioctls */
-       h->nr_cmds = h->max_commands - 4;
+       hpsa_find_board_params(h);
 
        if ((readb(&h->cfgtable->Signature[0]) != 'C') ||
            (readb(&h->cfgtable->Signature[1]) != 'I') ||