提交 55c06c71 编写于 作者: S Stephen M. Cameron 提交者: James Bottomley

[SCSI] hpsa: save pdev pointer in per hba structure early to avoid passing it around so much.

Signed-off-by: NStephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: NJames Bottomley <James.Bottomley@suse.de>
上级 1db90ea2
...@@ -3209,8 +3209,7 @@ static int find_PCI_BAR_index(struct pci_dev *pdev, unsigned long pci_bar_addr) ...@@ -3209,8 +3209,7 @@ static int find_PCI_BAR_index(struct pci_dev *pdev, unsigned long pci_bar_addr)
* controllers that are capable. If not, we use IO-APIC mode. * controllers that are capable. If not, we use IO-APIC mode.
*/ */
static void __devinit hpsa_interrupt_mode(struct ctlr_info *h, static void __devinit hpsa_interrupt_mode(struct ctlr_info *h, u32 board_id)
struct pci_dev *pdev, u32 board_id)
{ {
#ifdef CONFIG_PCI_MSI #ifdef CONFIG_PCI_MSI
int err; int err;
...@@ -3223,9 +3222,9 @@ static void __devinit hpsa_interrupt_mode(struct ctlr_info *h, ...@@ -3223,9 +3222,9 @@ static void __devinit hpsa_interrupt_mode(struct ctlr_info *h,
(board_id == 0x40800E11) || (board_id == 0x40800E11) ||
(board_id == 0x40820E11) || (board_id == 0x40830E11)) (board_id == 0x40820E11) || (board_id == 0x40830E11))
goto default_int_mode; goto default_int_mode;
if (pci_find_capability(pdev, PCI_CAP_ID_MSIX)) { if (pci_find_capability(h->pdev, PCI_CAP_ID_MSIX)) {
dev_info(&pdev->dev, "MSIX\n"); dev_info(&h->pdev->dev, "MSIX\n");
err = pci_enable_msix(pdev, hpsa_msix_entries, 4); err = pci_enable_msix(h->pdev, hpsa_msix_entries, 4);
if (!err) { if (!err) {
h->intr[0] = hpsa_msix_entries[0].vector; h->intr[0] = hpsa_msix_entries[0].vector;
h->intr[1] = hpsa_msix_entries[1].vector; h->intr[1] = hpsa_msix_entries[1].vector;
...@@ -3235,29 +3234,29 @@ static void __devinit hpsa_interrupt_mode(struct ctlr_info *h, ...@@ -3235,29 +3234,29 @@ static void __devinit hpsa_interrupt_mode(struct ctlr_info *h,
return; return;
} }
if (err > 0) { if (err > 0) {
dev_warn(&pdev->dev, "only %d MSI-X vectors " dev_warn(&h->pdev->dev, "only %d MSI-X vectors "
"available\n", err); "available\n", err);
goto default_int_mode; goto default_int_mode;
} else { } else {
dev_warn(&pdev->dev, "MSI-X init failed %d\n", dev_warn(&h->pdev->dev, "MSI-X init failed %d\n",
err); err);
goto default_int_mode; goto default_int_mode;
} }
} }
if (pci_find_capability(pdev, PCI_CAP_ID_MSI)) { if (pci_find_capability(h->pdev, PCI_CAP_ID_MSI)) {
dev_info(&pdev->dev, "MSI\n"); dev_info(&h->pdev->dev, "MSI\n");
if (!pci_enable_msi(pdev)) if (!pci_enable_msi(h->pdev))
h->msi_vector = 1; h->msi_vector = 1;
else else
dev_warn(&pdev->dev, "MSI init failed\n"); dev_warn(&h->pdev->dev, "MSI init failed\n");
} }
default_int_mode: default_int_mode:
#endif /* CONFIG_PCI_MSI */ #endif /* CONFIG_PCI_MSI */
/* if we get here we're going to use the default interrupt mode */ /* if we get here we're going to use the default interrupt mode */
h->intr[PERF_MODE_INT] = pdev->irq; h->intr[PERF_MODE_INT] = h->pdev->irq;
} }
static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev) static int __devinit hpsa_pci_init(struct ctlr_info *h)
{ {
ushort subsystem_vendor_id, subsystem_device_id, command; ushort subsystem_vendor_id, subsystem_device_id, command;
u32 board_id, scratchpad = 0; u32 board_id, scratchpad = 0;
...@@ -3267,8 +3266,8 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev) ...@@ -3267,8 +3266,8 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev)
u32 trans_offset; u32 trans_offset;
int i, prod_index, err; int i, prod_index, err;
subsystem_vendor_id = pdev->subsystem_vendor; subsystem_vendor_id = h->pdev->subsystem_vendor;
subsystem_device_id = pdev->subsystem_device; subsystem_device_id = h->pdev->subsystem_device;
board_id = (((u32) (subsystem_device_id << 16) & 0xffff0000) | board_id = (((u32) (subsystem_device_id << 16) & 0xffff0000) |
subsystem_vendor_id); subsystem_vendor_id);
...@@ -3282,7 +3281,7 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev) ...@@ -3282,7 +3281,7 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev)
prod_index--; prod_index--;
if (subsystem_vendor_id != PCI_VENDOR_ID_HP || if (subsystem_vendor_id != PCI_VENDOR_ID_HP ||
!hpsa_allow_any) { !hpsa_allow_any) {
dev_warn(&pdev->dev, "unrecognized board ID:" dev_warn(&h->pdev->dev, "unrecognized board ID:"
" 0x%08lx, ignoring.\n", " 0x%08lx, ignoring.\n",
(unsigned long) board_id); (unsigned long) board_id);
return -ENODEV; return -ENODEV;
...@@ -3291,41 +3290,42 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev) ...@@ -3291,41 +3290,42 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev)
/* check to see if controller has been disabled /* check to see if controller has been disabled
* BEFORE trying to enable it * BEFORE trying to enable it
*/ */
(void)pci_read_config_word(pdev, PCI_COMMAND, &command); (void)pci_read_config_word(h->pdev, PCI_COMMAND, &command);
if (!(command & 0x02)) { if (!(command & 0x02)) {
dev_warn(&pdev->dev, "controller appears to be disabled\n"); dev_warn(&h->pdev->dev, "controller appears to be disabled\n");
return -ENODEV; return -ENODEV;
} }
err = pci_enable_device(pdev); err = pci_enable_device(h->pdev);
if (err) { if (err) {
dev_warn(&pdev->dev, "unable to enable PCI device\n"); dev_warn(&h->pdev->dev, "unable to enable PCI device\n");
return err; return err;
} }
err = pci_request_regions(pdev, "hpsa"); err = pci_request_regions(h->pdev, "hpsa");
if (err) { if (err) {
dev_err(&pdev->dev, "cannot obtain PCI resources, aborting\n"); dev_err(&h->pdev->dev,
"cannot obtain PCI resources, aborting\n");
return err; return err;
} }
/* If the kernel supports MSI/MSI-X we will try to enable that, /* If the kernel supports MSI/MSI-X we will try to enable that,
* else we use the IO-APIC interrupt assigned to us by system ROM. * else we use the IO-APIC interrupt assigned to us by system ROM.
*/ */
hpsa_interrupt_mode(h, pdev, board_id); hpsa_interrupt_mode(h, board_id);
/* find the memory BAR */ /* find the memory BAR */
for (i = 0; i < DEVICE_COUNT_RESOURCE; i++) { for (i = 0; i < DEVICE_COUNT_RESOURCE; i++) {
if (pci_resource_flags(pdev, i) & IORESOURCE_MEM) if (pci_resource_flags(h->pdev, i) & IORESOURCE_MEM)
break; break;
} }
if (i == DEVICE_COUNT_RESOURCE) { if (i == DEVICE_COUNT_RESOURCE) {
dev_warn(&pdev->dev, "no memory BAR found\n"); dev_warn(&h->pdev->dev, "no memory BAR found\n");
err = -ENODEV; err = -ENODEV;
goto err_out_free_res; goto err_out_free_res;
} }
h->paddr = pci_resource_start(pdev, i); /* addressing mode bits h->paddr = pci_resource_start(h->pdev, i); /* addressing mode bits
* already removed * already removed
*/ */
...@@ -3339,7 +3339,7 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev) ...@@ -3339,7 +3339,7 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev)
msleep(HPSA_BOARD_READY_POLL_INTERVAL_MSECS); msleep(HPSA_BOARD_READY_POLL_INTERVAL_MSECS);
} }
if (scratchpad != HPSA_FIRMWARE_READY) { if (scratchpad != HPSA_FIRMWARE_READY) {
dev_warn(&pdev->dev, "board not ready, timed out.\n"); dev_warn(&h->pdev->dev, "board not ready, timed out.\n");
err = -ENODEV; err = -ENODEV;
goto err_out_free_res; goto err_out_free_res;
} }
...@@ -3347,20 +3347,20 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev) ...@@ -3347,20 +3347,20 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev)
/* get the address index number */ /* get the address index number */
cfg_base_addr = readl(h->vaddr + SA5_CTCFG_OFFSET); cfg_base_addr = readl(h->vaddr + SA5_CTCFG_OFFSET);
cfg_base_addr &= (u32) 0x0000ffff; cfg_base_addr &= (u32) 0x0000ffff;
cfg_base_addr_index = find_PCI_BAR_index(pdev, cfg_base_addr); cfg_base_addr_index = find_PCI_BAR_index(h->pdev, cfg_base_addr);
if (cfg_base_addr_index == -1) { if (cfg_base_addr_index == -1) {
dev_warn(&pdev->dev, "cannot find cfg_base_addr_index\n"); dev_warn(&h->pdev->dev, "cannot find cfg_base_addr_index\n");
err = -ENODEV; err = -ENODEV;
goto err_out_free_res; goto err_out_free_res;
} }
cfg_offset = readl(h->vaddr + SA5_CTMEM_OFFSET); cfg_offset = readl(h->vaddr + SA5_CTMEM_OFFSET);
h->cfgtable = remap_pci_mem(pci_resource_start(pdev, h->cfgtable = remap_pci_mem(pci_resource_start(h->pdev,
cfg_base_addr_index) + cfg_offset, cfg_base_addr_index) + cfg_offset,
sizeof(h->cfgtable)); sizeof(h->cfgtable));
/* Find performant mode table. */ /* Find performant mode table. */
trans_offset = readl(&(h->cfgtable->TransMethodOffset)); trans_offset = readl(&(h->cfgtable->TransMethodOffset));
h->transtable = remap_pci_mem(pci_resource_start(pdev, h->transtable = remap_pci_mem(pci_resource_start(h->pdev,
cfg_base_addr_index)+cfg_offset+trans_offset, cfg_base_addr_index)+cfg_offset+trans_offset,
sizeof(*h->transtable)); sizeof(*h->transtable));
...@@ -3392,7 +3392,7 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev) ...@@ -3392,7 +3392,7 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev)
(readb(&h->cfgtable->Signature[1]) != 'I') || (readb(&h->cfgtable->Signature[1]) != 'I') ||
(readb(&h->cfgtable->Signature[2]) != 'S') || (readb(&h->cfgtable->Signature[2]) != 'S') ||
(readb(&h->cfgtable->Signature[3]) != 'S')) { (readb(&h->cfgtable->Signature[3]) != 'S')) {
dev_warn(&pdev->dev, "not a valid CISS config table\n"); dev_warn(&h->pdev->dev, "not a valid CISS config table\n");
err = -ENODEV; err = -ENODEV;
goto err_out_free_res; goto err_out_free_res;
} }
...@@ -3434,11 +3434,12 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev) ...@@ -3434,11 +3434,12 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev)
} }
#ifdef HPSA_DEBUG #ifdef HPSA_DEBUG
print_cfg_table(&pdev->dev, h->cfgtable); print_cfg_table(&h->pdev->dev, h->cfgtable);
#endif /* HPSA_DEBUG */ #endif /* HPSA_DEBUG */
if (!(readl(&(h->cfgtable->TransportActive)) & CFGTBL_Trans_Simple)) { if (!(readl(&(h->cfgtable->TransportActive)) & CFGTBL_Trans_Simple)) {
dev_warn(&pdev->dev, "unable to get board into simple mode\n"); dev_warn(&h->pdev->dev,
"unable to get board into simple mode\n");
err = -ENODEV; err = -ENODEV;
goto err_out_free_res; goto err_out_free_res;
} }
...@@ -3449,7 +3450,7 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev) ...@@ -3449,7 +3450,7 @@ static int __devinit hpsa_pci_init(struct ctlr_info *h, struct pci_dev *pdev)
* Deliberately omit pci_disable_device(): it does something nasty to * Deliberately omit pci_disable_device(): it does something nasty to
* Smart Array controllers that pci_enable_device does not undo * Smart Array controllers that pci_enable_device does not undo
*/ */
pci_release_regions(pdev); pci_release_regions(h->pdev);
return err; return err;
} }
...@@ -3507,17 +3508,17 @@ static int __devinit hpsa_init_one(struct pci_dev *pdev, ...@@ -3507,17 +3508,17 @@ static int __devinit hpsa_init_one(struct pci_dev *pdev,
if (!h) if (!h)
return -ENOMEM; return -ENOMEM;
h->pdev = pdev;
h->busy_initializing = 1; h->busy_initializing = 1;
INIT_HLIST_HEAD(&h->cmpQ); INIT_HLIST_HEAD(&h->cmpQ);
INIT_HLIST_HEAD(&h->reqQ); INIT_HLIST_HEAD(&h->reqQ);
rc = hpsa_pci_init(h, pdev); rc = hpsa_pci_init(h);
if (rc != 0) if (rc != 0)
goto clean1; goto clean1;
sprintf(h->devname, "hpsa%d", number_of_controllers); sprintf(h->devname, "hpsa%d", number_of_controllers);
h->ctlr = number_of_controllers; h->ctlr = number_of_controllers;
number_of_controllers++; number_of_controllers++;
h->pdev = pdev;
/* configure PCI DMA stuff */ /* configure PCI DMA stuff */
rc = pci_set_dma_mask(pdev, DMA_BIT_MASK(64)); rc = pci_set_dma_mask(pdev, DMA_BIT_MASK(64));
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册