scsi: drop bus reset for wd33c93-compatible boards
authorHannes Reinecke <hare@suse.de>
Fri, 25 Aug 2017 11:57:06 +0000 (13:57 +0200)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 25 Aug 2017 21:21:10 +0000 (17:21 -0400)
The bus reset function is just a wrapper calling host reset under the
host lock. So move taking of the host lock into the host reset function
and drop bus reset.

Signed-off-by: Hannes Reinecke <hare@suse.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/a2091.c
drivers/scsi/a3000.c
drivers/scsi/gvp11.c
drivers/scsi/mvme147.c
drivers/scsi/sgiwd93.c
drivers/scsi/wd33c93.c

index 9176bfbd574586ab04a62786253271b1a1636006..61aadc7acb4998a58f27cb1872d883d7e771378d 100644 (file)
@@ -147,22 +147,6 @@ static void dma_stop(struct Scsi_Host *instance, struct scsi_cmnd *SCpnt,
        }
 }
 
-static int a2091_bus_reset(struct scsi_cmnd *cmd)
-{
-       struct Scsi_Host *instance = cmd->device->host;
-
-       /* FIXME perform bus-specific reset */
-
-       /* FIXME 2: kill this function, and let midlayer fall back
-          to the same action, calling wd33c93_host_reset() */
-
-       spin_lock_irq(instance->host_lock);
-       wd33c93_host_reset(cmd);
-       spin_unlock_irq(instance->host_lock);
-
-       return SUCCESS;
-}
-
 static struct scsi_host_template a2091_scsi_template = {
        .module                 = THIS_MODULE,
        .name                   = "Commodore A2091/A590 SCSI",
@@ -171,7 +155,6 @@ static struct scsi_host_template a2091_scsi_template = {
        .proc_name              = "A2901",
        .queuecommand           = wd33c93_queuecommand,
        .eh_abort_handler       = wd33c93_abort,
-       .eh_bus_reset_handler   = a2091_bus_reset,
        .eh_host_reset_handler  = wd33c93_host_reset,
        .can_queue              = CAN_QUEUE,
        .this_id                = 7,
index e6375b4de79e58e05ba935c51de8aa088435b792..2427a85412471f39d10b159c25154c3bbf37ae85 100644 (file)
@@ -162,22 +162,6 @@ static void dma_stop(struct Scsi_Host *instance, struct scsi_cmnd *SCpnt,
        }
 }
 
-static int a3000_bus_reset(struct scsi_cmnd *cmd)
-{
-       struct Scsi_Host *instance = cmd->device->host;
-
-       /* FIXME perform bus-specific reset */
-
-       /* FIXME 2: kill this entire function, which should
-          cause mid-layer to call wd33c93_host_reset anyway? */
-
-       spin_lock_irq(instance->host_lock);
-       wd33c93_host_reset(cmd);
-       spin_unlock_irq(instance->host_lock);
-
-       return SUCCESS;
-}
-
 static struct scsi_host_template amiga_a3000_scsi_template = {
        .module                 = THIS_MODULE,
        .name                   = "Amiga 3000 built-in SCSI",
@@ -186,7 +170,6 @@ static struct scsi_host_template amiga_a3000_scsi_template = {
        .proc_name              = "A3000",
        .queuecommand           = wd33c93_queuecommand,
        .eh_abort_handler       = wd33c93_abort,
-       .eh_bus_reset_handler   = a3000_bus_reset,
        .eh_host_reset_handler  = wd33c93_host_reset,
        .can_queue              = CAN_QUEUE,
        .this_id                = 7,
index 3b6f83ffddc4e87c59057d0f7eb8a4ded51f11f4..a27fc49ebd3acd5ed7063e8254d6736fca5ca4e6 100644 (file)
@@ -171,23 +171,6 @@ static void dma_stop(struct Scsi_Host *instance, struct scsi_cmnd *SCpnt,
        }
 }
 
-static int gvp11_bus_reset(struct scsi_cmnd *cmd)
-{
-       struct Scsi_Host *instance = cmd->device->host;
-
-       /* FIXME perform bus-specific reset */
-
-       /* FIXME 2: shouldn't we no-op this function (return
-          FAILED), and fall back to host reset function,
-          wd33c93_host_reset ? */
-
-       spin_lock_irq(instance->host_lock);
-       wd33c93_host_reset(cmd);
-       spin_unlock_irq(instance->host_lock);
-
-       return SUCCESS;
-}
-
 static struct scsi_host_template gvp11_scsi_template = {
        .module                 = THIS_MODULE,
        .name                   = "GVP Series II SCSI",
@@ -196,7 +179,6 @@ static struct scsi_host_template gvp11_scsi_template = {
        .proc_name              = "GVP11",
        .queuecommand           = wd33c93_queuecommand,
        .eh_abort_handler       = wd33c93_abort,
-       .eh_bus_reset_handler   = gvp11_bus_reset,
        .eh_host_reset_handler  = wd33c93_host_reset,
        .can_queue              = CAN_QUEUE,
        .this_id                = 7,
index e7f6661a8862112a7a126e2ee4ec07c369c7dd5a..4f515700bdc349511cf5df90035f589b3a9e8039 100644 (file)
@@ -121,21 +121,6 @@ err_out:
        return 0;
 }
 
-static int mvme147_bus_reset(struct scsi_cmnd *cmd)
-{
-       /* FIXME perform bus-specific reset */
-
-       /* FIXME 2: kill this function, and let midlayer fallback to
-          the same result, calling wd33c93_host_reset() */
-
-       spin_lock_irq(cmd->device->host->host_lock);
-       wd33c93_host_reset(cmd);
-       spin_unlock_irq(cmd->device->host->host_lock);
-
-       return SUCCESS;
-}
-
-
 static struct scsi_host_template driver_template = {
        .proc_name              = "MVME147",
        .name                   = "MVME147 built-in SCSI",
@@ -143,7 +128,6 @@ static struct scsi_host_template driver_template = {
        .release                = mvme147_release,
        .queuecommand           = wd33c93_queuecommand,
        .eh_abort_handler       = wd33c93_abort,
-       .eh_bus_reset_handler   = mvme147_bus_reset,
        .eh_host_reset_handler  = wd33c93_host_reset,
        .can_queue              = CAN_QUEUE,
        .this_id                = 7,
index 80cfa93e407c3aab151c0165196dc5e8751b5c1f..5ed696dc9bbd53a5bf9cd063b6e3915069bde473 100644 (file)
@@ -192,20 +192,6 @@ static inline void init_hpc_chain(struct ip22_hostdata *hdata)
        hcp->desc.pnext = hdata->dma;
 }
 
-static int sgiwd93_bus_reset(struct scsi_cmnd *cmd)
-{
-       /* FIXME perform bus-specific reset */
-
-       /* FIXME 2: kill this function, and let midlayer fallback
-          to the same result, calling wd33c93_host_reset() */
-
-       spin_lock_irq(cmd->device->host->host_lock);
-       wd33c93_host_reset(cmd);
-       spin_unlock_irq(cmd->device->host->host_lock);
-
-       return SUCCESS;
-}
-
 /*
  * Kludge alert - the SCSI code calls the abort and reset method with int
  * arguments not with pointers.  So this is going to blow up beautyfully
@@ -217,7 +203,6 @@ static struct scsi_host_template sgiwd93_template = {
        .name                   = "SGI WD93",
        .queuecommand           = wd33c93_queuecommand,
        .eh_abort_handler       = wd33c93_abort,
-       .eh_bus_reset_handler   = sgiwd93_bus_reset,
        .eh_host_reset_handler  = wd33c93_host_reset,
        .can_queue              = 16,
        .this_id                = 7,
index 9e09da412b92299c4f903764e4c0cebf71a5b845..74be04f2357c0f9ca9110bf6f8b05f147b4370b0 100644 (file)
@@ -1578,6 +1578,7 @@ wd33c93_host_reset(struct scsi_cmnd * SCpnt)
        int i;
 
        instance = SCpnt->device->host;
+       spin_lock_irq(instance->host_lock);
        hostdata = (struct WD33C93_hostdata *) instance->hostdata;
 
        printk("scsi%d: reset. ", instance->host_no);
@@ -1603,6 +1604,7 @@ wd33c93_host_reset(struct scsi_cmnd * SCpnt)
        reset_wd33c93(instance);
        SCpnt->result = DID_RESET << 16;
        enable_irq(instance->irq);
+       spin_unlock_irq(instance->host_lock);
        return SUCCESS;
 }