diff options
author | Jeff Garzik <jeff@garzik.org> | 2007-07-27 13:01:15 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.localdomain> | 2007-10-12 14:50:51 -0400 |
commit | 488a5c8a9a3b67ae117784cd0d73bef53a73d57d (patch) | |
tree | 14fc9dd53a6208c42a3cb1e3726d86224413dd68 /drivers/scsi/arcmsr | |
parent | 24430458bb924e371ff894e26bfa9f73707f53fb (diff) |
[SCSI] arcmsr: irq handler fixes, cleanups, micro-opts
* Remove IRQF_DISABLED, it is clearly wrong for this driver.
* Remove wasteful spin_lock_irqsave() in interrupt handler.
The lighter-weight spin_lock() is all that's needed.
* Annotate with FIXME where arcmsr_interrupt() is called
without any spinlock being acquired.
* Eliminate pointless cast from void pointer in arcmsr_do_interrupt()
[jejb: conflict resolution]
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Acked-by: Nick Cheng <nick.cheng@areca.com.tw>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi/arcmsr')
-rw-r--r-- | drivers/scsi/arcmsr/arcmsr_hba.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c index 7832a10dc47f..f4d2d523db44 100644 --- a/drivers/scsi/arcmsr/arcmsr_hba.c +++ b/drivers/scsi/arcmsr/arcmsr_hba.c | |||
@@ -422,7 +422,7 @@ static int arcmsr_probe(struct pci_dev *pdev, | |||
422 | goto out_release_regions; | 422 | goto out_release_regions; |
423 | 423 | ||
424 | error = request_irq(pdev->irq, arcmsr_do_interrupt, | 424 | error = request_irq(pdev->irq, arcmsr_do_interrupt, |
425 | IRQF_SHARED, "arcmsr", acb); | 425 | IRQF_SHARED, "arcmsr", acb); |
426 | if (error) | 426 | if (error) |
427 | goto out_free_ccb_pool; | 427 | goto out_free_ccb_pool; |
428 | 428 | ||