diff options
author | Robert Elliott <elliott@hp.com> | 2015-01-23 17:42:32 -0500 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2015-02-02 12:57:38 -0500 |
commit | 9ee61794879dad68ec273f3e4eaad41285e6ff3c (patch) | |
tree | 5b570f1eaad7d062741fa4bca9015d03de350b52 /drivers/scsi/hpsa.c | |
parent | cd3c81c4a746ffca4e5a863ce05a1e0aa6766356 (diff) |
hpsa: rename hpsa_request_irq to hpsa_request_irqs
Make the function name more descriptive. We use more than
one interrupt.
Reviewed-by: Scott Teel <scott.teel@pmcs.com>
Signed-off-by: Robert Elliott <elliott@hp.com>
Signed-off-by: Don Brace <don.brace@pmcs.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/scsi/hpsa.c')
-rw-r--r-- | drivers/scsi/hpsa.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 354e7f8831a4..f29f569e0a5b 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c | |||
@@ -6526,7 +6526,8 @@ static void hpsa_free_irqs(struct ctlr_info *h) | |||
6526 | h->q[i] = 0; | 6526 | h->q[i] = 0; |
6527 | } | 6527 | } |
6528 | 6528 | ||
6529 | static int hpsa_request_irq(struct ctlr_info *h, | 6529 | /* returns 0 on success; cleans up and returns -Enn on error */ |
6530 | static int hpsa_request_irqs(struct ctlr_info *h, | ||
6530 | irqreturn_t (*msixhandler)(int, void *), | 6531 | irqreturn_t (*msixhandler)(int, void *), |
6531 | irqreturn_t (*intxhandler)(int, void *)) | 6532 | irqreturn_t (*intxhandler)(int, void *)) |
6532 | { | 6533 | { |
@@ -6934,7 +6935,7 @@ reinit_after_soft_reset: | |||
6934 | /* make sure the board interrupts are off */ | 6935 | /* make sure the board interrupts are off */ |
6935 | h->access.set_intr_mask(h, HPSA_INTR_OFF); | 6936 | h->access.set_intr_mask(h, HPSA_INTR_OFF); |
6936 | 6937 | ||
6937 | if (hpsa_request_irq(h, do_hpsa_intr_msi, do_hpsa_intr_intx)) | 6938 | if (hpsa_request_irqs(h, do_hpsa_intr_msi, do_hpsa_intr_intx)) |
6938 | goto clean2; | 6939 | goto clean2; |
6939 | dev_info(&pdev->dev, "%s: <0x%x> at IRQ %d%s using DAC\n", | 6940 | dev_info(&pdev->dev, "%s: <0x%x> at IRQ %d%s using DAC\n", |
6940 | h->devname, pdev->device, | 6941 | h->devname, pdev->device, |
@@ -6970,11 +6971,11 @@ reinit_after_soft_reset: | |||
6970 | h->access.set_intr_mask(h, HPSA_INTR_OFF); | 6971 | h->access.set_intr_mask(h, HPSA_INTR_OFF); |
6971 | spin_unlock_irqrestore(&h->lock, flags); | 6972 | spin_unlock_irqrestore(&h->lock, flags); |
6972 | hpsa_free_irqs(h); | 6973 | hpsa_free_irqs(h); |
6973 | rc = hpsa_request_irq(h, hpsa_msix_discard_completions, | 6974 | rc = hpsa_request_irqs(h, hpsa_msix_discard_completions, |
6974 | hpsa_intx_discard_completions); | 6975 | hpsa_intx_discard_completions); |
6975 | if (rc) { | 6976 | if (rc) { |
6976 | dev_warn(&h->pdev->dev, "Failed to request_irq after " | 6977 | dev_warn(&h->pdev->dev, |
6977 | "soft reset.\n"); | 6978 | "Failed to request_irq after soft reset.\n"); |
6978 | goto clean4; | 6979 | goto clean4; |
6979 | } | 6980 | } |
6980 | 6981 | ||