[SCSI] mpt2sas: use correct pci_resource_flag for comparison
authorRichard A Lary <rlary@linux.vnet.ibm.com>
Fri, 12 Mar 2010 23:27:06 +0000 (15:27 -0800)
committerJames Bottomley <James.Bottomley@suse.de>
Sun, 11 Apr 2010 14:23:33 +0000 (09:23 -0500)
This patch replaces incorrect base address space flag with correct IO
resource flag.  Also, performs check of memory resource to validate
resource before using.

Signed-off-by: Richard A Lary <rlary@linux.vnet.ibm.com>
Acked-by: "Desai, Kashyap" <Kashyap.Desai@lsi.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/mpt2sas/mpt2sas_base.c

index da4bfbf95a7a9bdb9e882a5d7e71223b6d67abc0..fb886d099baf0ea8108b5eaacb56f1180576066e 100644 (file)
@@ -1253,7 +1253,7 @@ mpt2sas_base_map_resources(struct MPT2SAS_ADAPTER *ioc)
        }
 
        for (i = 0, memap_sz = 0, pio_sz = 0 ; i < DEVICE_COUNT_RESOURCE; i++) {
-               if (pci_resource_flags(pdev, i) & PCI_BASE_ADDRESS_SPACE_IO) {
+               if (pci_resource_flags(pdev, i) & IORESOURCE_IO) {
                        if (pio_sz)
                                continue;
                        pio_chip = (u64)pci_resource_start(pdev, i);
@@ -1261,15 +1261,18 @@ mpt2sas_base_map_resources(struct MPT2SAS_ADAPTER *ioc)
                } else {
                        if (memap_sz)
                                continue;
-                       ioc->chip_phys = pci_resource_start(pdev, i);
-                       chip_phys = (u64)ioc->chip_phys;
-                       memap_sz = pci_resource_len(pdev, i);
-                       ioc->chip = ioremap(ioc->chip_phys, memap_sz);
-                       if (ioc->chip == NULL) {
-                               printk(MPT2SAS_ERR_FMT "unable to map adapter "
-                                   "memory!\n", ioc->name);
-                               r = -EINVAL;
-                               goto out_fail;
+                       /* verify memory resource is valid before using */
+                       if (pci_resource_flags(pdev, i) & IORESOURCE_MEM) {
+                               ioc->chip_phys = pci_resource_start(pdev, i);
+                               chip_phys = (u64)ioc->chip_phys;
+                               memap_sz = pci_resource_len(pdev, i);
+                               ioc->chip = ioremap(ioc->chip_phys, memap_sz);
+                               if (ioc->chip == NULL) {
+                                       printk(MPT2SAS_ERR_FMT "unable to map "
+                                           "adapter memory!\n", ioc->name);
+                                       r = -EINVAL;
+                                       goto out_fail;
+                               }
                        }
                }
        }