[SCSI] arcmsr: irq handler fixes, cleanups, micro-opts
authorJeff Garzik <jeff@garzik.org>
Fri, 27 Jul 2007 17:01:15 +0000 (13:01 -0400)
committerJames Bottomley <jejb@mulgrave.localdomain>
Fri, 12 Oct 2007 18:50:51 +0000 (14:50 -0400)
* 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>
drivers/scsi/arcmsr/arcmsr_hba.c

index 7832a10dc47f5319591cb9c4469852ef5f665815..f4d2d523db441d5b5d21d40bbc0a3fa3c78a4d94 100644 (file)
@@ -422,7 +422,7 @@ static int arcmsr_probe(struct pci_dev *pdev,
                goto out_release_regions;
 
        error = request_irq(pdev->irq, arcmsr_do_interrupt,
-                               IRQF_SHARED, "arcmsr", acb);
+                           IRQF_SHARED, "arcmsr", acb);
        if (error)
                goto out_free_ccb_pool;