dmx3191d: Use NO_IRQ
authorFinn Thain <fthain@telegraphics.com.au>
Wed, 12 Nov 2014 05:12:03 +0000 (16:12 +1100)
committerChristoph Hellwig <hch@lst.de>
Thu, 20 Nov 2014 08:11:11 +0000 (09:11 +0100)
Testing shows that the Domex 3191D card never asserts its IRQ. Hence it is
non-functional with Linux (worse, the EH bugs in the core driver are fatal
but that's a problem for another patch). Perhaps the DT-536 chip needs
special setup? I can't find documentation for it. The NetBSD driver uses
polling apparently because of this issue.

Set host->irq = NO_IRQ so the core driver will prevent targets from
disconnecting. Don't request host->irq.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>
drivers/scsi/dmx3191d.c

index 159434a5cfdb72f552bd0eeb710b12f5b459eec3..3e088125a8be6a339757dc1380f1c73418952bfc 100644 (file)
@@ -34,6 +34,8 @@
  * Definitions for the generic 5380 driver.
  */
 
+#define DONT_USE_INTR
+
 #define NCR5380_read(reg)              inb(port + reg)
 #define NCR5380_write(reg, value)      outb(value, port + reg)
 
@@ -89,32 +91,23 @@ static int dmx3191d_probe_one(struct pci_dev *pdev,
        if (!shost)
                goto out_release_region;       
        shost->io_port = io;
-       shost->irq = pdev->irq;
 
-       NCR5380_init(shost, FLAG_NO_PSEUDO_DMA | FLAG_DTC3181E);
+       /* This card does not seem to raise an interrupt on pdev->irq.
+        * Steam-powered SCSI controllers run without an IRQ anyway.
+        */
+       shost->irq = NO_IRQ;
 
-       if (request_irq(pdev->irq, NCR5380_intr, IRQF_SHARED,
-                               DMX3191D_DRIVER_NAME, shost)) {
-               /*
-                * Steam powered scsi controllers run without an IRQ anyway
-                */
-               printk(KERN_WARNING "dmx3191: IRQ %d not available - "
-                                   "switching to polled mode.\n", pdev->irq);
-               shost->irq = NO_IRQ;
-       }
+       NCR5380_init(shost, FLAG_NO_PSEUDO_DMA | FLAG_DTC3181E);
 
        pci_set_drvdata(pdev, shost);
 
        error = scsi_add_host(shost, &pdev->dev);
        if (error)
-               goto out_free_irq;
+               goto out_release_region;
 
        scsi_scan_host(shost);
        return 0;
 
- out_free_irq:
-       if (shost->irq != NO_IRQ)
-               free_irq(shost->irq, shost);
  out_release_region:
        release_region(io, DMX3191D_REGION_LEN);
  out_disable_device:
@@ -131,8 +124,6 @@ static void dmx3191d_remove_one(struct pci_dev *pdev)
 
        NCR5380_exit(shost);
 
-       if (shost->irq != NO_IRQ)
-               free_irq(shost->irq, shost);
        release_region(shost->io_port, DMX3191D_REGION_LEN);
        pci_disable_device(pdev);