ncr5380: Remove redundant AUTOSENSE macro
authorFinn Thain <fthain@telegraphics.com.au>
Wed, 12 Nov 2014 05:11:54 +0000 (16:11 +1100)
committerChristoph Hellwig <hch@lst.de>
Thu, 20 Nov 2014 08:11:06 +0000 (09:11 +0100)
Every NCR5380 driver sets AUTOSENSE so it need not be optional (and the
mid-layer expects it). Remove this redundant macro to improve readability.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Tested-by: Michael Schmitz <schmitzmic@gmail.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
14 files changed:
drivers/scsi/NCR5380.c
drivers/scsi/NCR5380.h
drivers/scsi/arm/cumana_1.c
drivers/scsi/arm/oak.c
drivers/scsi/atari_NCR5380.c
drivers/scsi/atari_scsi.c
drivers/scsi/dmx3191d.c
drivers/scsi/dtc.c
drivers/scsi/g_NCR5380.c
drivers/scsi/mac_scsi.c
drivers/scsi/pas16.c
drivers/scsi/sun3_NCR5380.c
drivers/scsi/sun3_scsi.c
drivers/scsi/t128.c

index 0a8786acddc3ad2ef700a9af43ee5da12d2d6ec0..e17e64e3e0d9657f157ecf6c50ee7a03ddb9930f 100644 (file)
@@ -627,9 +627,6 @@ NCR5380_print_options(struct Scsi_Host *instance)
 #ifdef AUTOPROBE_IRQ
               " AUTOPROBE_IRQ"
 #endif
-#ifdef AUTOSENSE
-              " AUTOSENSE"
-#endif
 #ifdef DIFFERENTIAL
               " DIFFERENTIAL"
 #endif
@@ -857,12 +854,6 @@ static int NCR5380_init(struct Scsi_Host *instance, int flags)
        hostdata->host = instance;
        hostdata->time_expires = 0;
 
-#ifndef AUTOSENSE
-       if ((instance->cmd_per_lun > 1) || instance->can_queue > 1)
-                   printk(KERN_WARNING "scsi%d : WARNING : support for multiple outstanding commands enabled\n" "         without AUTOSENSE option, contingent allegiance conditions may\n"
-                          "         be incorrectly cleared.\n", instance->host_no);
-#endif                         /* def AUTOSENSE */
-
        NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE);
        NCR5380_write(MODE_REG, MR_BASE);
        NCR5380_write(TARGET_COMMAND_REG, 0);
@@ -2260,7 +2251,6 @@ static void NCR5380_information_transfer(struct Scsi_Host *instance) {
                                        else if (status_byte(cmd->SCp.Status) != GOOD)
                                                cmd->result = (cmd->result & 0x00ffff) | (DID_ERROR << 16);
 
-#ifdef AUTOSENSE
                                        if ((cmd->cmnd[0] == REQUEST_SENSE) &&
                                                hostdata->ses.cmd_len) {
                                                scsi_eh_restore_cmnd(cmd, &hostdata->ses);
@@ -2277,9 +2267,7 @@ static void NCR5380_information_transfer(struct Scsi_Host *instance) {
                                                    hostdata->issue_queue;
                                                hostdata->issue_queue = (Scsi_Cmnd *) cmd;
                                                dprintk(NDEBUG_QUEUES, "scsi%d : REQUEST SENSE added to head of issue queue\n", instance->host_no);
-                                       } else
-#endif                         /* def AUTOSENSE */
-                                       {
+                                       } else {
                                                collect_stats(hostdata, cmd);
                                                cmd->scsi_done(cmd);
                                        }
index b36e3b4a9d437d05cf649543ecc023bf632c9d8e..f09d560fb6b27201c295064c2f7c12fbca538cfb 100644 (file)
 #define NCR5380_H
 
 #include <linux/interrupt.h>
-
-#ifdef AUTOSENSE
 #include <scsi/scsi_eh.h>
-#endif
 
 #define NCR5380_PUBLIC_RELEASE 7
 #define NCR53C400_PUBLIC_RELEASE 2
@@ -281,9 +278,7 @@ struct NCR5380_hostdata {
        unsigned pendingr;
        unsigned pendingw;
 #endif
-#ifdef AUTOSENSE
        struct scsi_eh_save ses;
-#endif
 };
 
 #ifdef __KERNEL__
index 8ef810a4476ec15f163069005ceae6b3f6b0f089..8266039b60eea2c0a7bfdd7c2f85c7a60242ec26 100644 (file)
@@ -18,7 +18,6 @@
 
 #include <scsi/scsicam.h>
 
-#define AUTOSENSE
 #define PSEUDO_DMA
 
 #define CUMANASCSI_PUBLIC_RELEASE 1
index 5ba3db6050006dd75eef5e04cf51ecb06877f6c9..f62cc904a47b86e621a9805d7982925aa7d8cd9a 100644 (file)
@@ -17,7 +17,6 @@
 #include "../scsi.h"
 #include <scsi/scsi_host.h>
 
-#define AUTOSENSE
 /*#define PSEUDO_DMA*/
 
 #define OAKSCSI_PUBLIC_RELEASE 1
index 3cfd39c583ad54277cd43bf9a42dcc16e2d19395..f69d36327a7d0884db4ce7db66cadf0d1eb52979 100644 (file)
@@ -683,9 +683,6 @@ static inline void NCR5380_all_init(void)
 static void __init NCR5380_print_options(struct Scsi_Host *instance)
 {
        printk(" generic options"
-#ifdef AUTOSENSE
-              " AUTOSENSE"
-#endif
 #ifdef REAL_DMA
               " REAL DMA"
 #endif
@@ -842,13 +839,6 @@ static int __init NCR5380_init(struct Scsi_Host *instance, int flags)
                first_instance = instance;
        }
 
-#ifndef AUTOSENSE
-       if ((instance->cmd_per_lun > 1) || (instance->can_queue > 1))
-               printk("scsi%d: WARNING : support for multiple outstanding commands enabled\n"
-                      "        without AUTOSENSE option, contingent allegiance conditions may\n"
-                      "        be incorrectly cleared.\n", HOSTNO);
-#endif /* def AUTOSENSE */
-
        NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE);
        NCR5380_write(MODE_REG, MR_BASE);
        NCR5380_write(TARGET_COMMAND_REG, 0);
@@ -2199,7 +2189,6 @@ static void NCR5380_information_transfer(struct Scsi_Host *instance)
                                        else if (status_byte(cmd->SCp.Status) != GOOD)
                                                cmd->result = (cmd->result & 0x00ffff) | (DID_ERROR << 16);
 
-#ifdef AUTOSENSE
                                        if ((cmd->cmnd[0] == REQUEST_SENSE) &&
                                                hostdata->ses.cmd_len) {
                                                scsi_eh_restore_cmnd(cmd, &hostdata->ses);
@@ -2219,9 +2208,7 @@ static void NCR5380_information_transfer(struct Scsi_Host *instance)
                                                local_irq_restore(flags);
                                                dprintk(NDEBUG_QUEUES, "scsi%d: REQUEST SENSE added to head of "
                                                          "issue queue\n", H_NO(cmd));
-                                       } else
-#endif /* def AUTOSENSE */
-                                       {
+                                       } else {
 #ifdef NCR5380_STATS
                                                collect_stats(hostdata, cmd);
 #endif
index a68d6257bc681c46c4a3ff37756fd319e630d6bc..9f4d58ca3647a1c93111afbc779f862ddd74afa8 100644 (file)
@@ -67,7 +67,6 @@
 
 #include <linux/module.h>
 
-#define AUTOSENSE
 /* For the Atari version, use only polled IO or REAL_DMA */
 #define        REAL_DMA
 /* Support tagged queuing? (on devices that are able to... :-) */
index 6d534b9ffc916a48cae4695c433616a117dbd516..5101328656b895feea0d960c48ddd59f542d9523 100644 (file)
@@ -33,7 +33,6 @@
 /*
  * Definitions for the generic 5380 driver.
  */
-#define AUTOSENSE
 
 #define NCR5380_read(reg)              inb(port + reg)
 #define NCR5380_write(reg, value)      outb(value, port + reg)
index e890b08efbd8b64f2233012a9c451ee922d31679..975abbfe0bba72fab025ada305add2da618cea7a 100644 (file)
@@ -1,5 +1,4 @@
 
-#define AUTOSENSE
 #define PSEUDO_DMA
 #define DONT_USE_INTR
 #define UNSAFE                 /* Leave interrupts enabled during pseudo-dma I/O */
@@ -30,9 +29,6 @@
 
 /*
  * Options : 
- * AUTOSENSE - if defined, REQUEST SENSE will be performed automatically
- *      for commands that return with a CHECK CONDITION status. 
- *
  * PSEUDO_DMA - enables PSEUDO-DMA hardware, should give a 3-4X performance
  * increase compared to polled I/O.
  *
index c10466939a5228da1774a7068469ad5ad7651dc1..eda978f29d9d7684c468121bf9010d005bdbac89 100644 (file)
@@ -80,8 +80,6 @@
 #define USLEEP_WAITLONG        500
 
 #define AUTOPROBE_IRQ
-#define AUTOSENSE
-
 
 #ifdef CONFIG_SCSI_GENERIC_NCR53C400
 #define NCR53C400_PSEUDO_DMA 1
index aa372ec3850749262f25028aba5a8ed75b1f33a0..3f125838a29d74433cd0b4fb7d3bcd32b28f10d3 100644 (file)
@@ -48,8 +48,6 @@
 #include <scsi/scsi_host.h>
 #include "mac_scsi.h"
 
-/* These control the behaviour of the generic 5380 core */
-#define AUTOSENSE
 #define PSEUDO_DMA
 
 #include "NCR5380.h"
index ffa399e7860adc0cbca84fc7a339d50c81f4e4b8..7621205e4aa02b4b7987c1d14319874c84365a41 100644 (file)
@@ -1,4 +1,3 @@
-#define AUTOSENSE
 #define PSEUDO_DMA
 #define UNSAFE  /* Not unsafe for PAS16 -- use it */
 #define PDEBUG 0
@@ -39,9 +38,6 @@
 
 /*
  * Options : 
- * AUTOSENSE - if defined, REQUEST SENSE will be performed automatically
- *      for commands that return with a CHECK CONDITION status. 
- *
  * LIMIT_TRANSFERSIZE - if defined, limit the pseudo-dma transfers to 512
  *      bytes at a time.  Since interrupts are disabled by default during
  *      these transfers, we might need this to give reasonable interrupt
index 3586fecdc8bbbce06fa8da2f9b20614aa1ac66eb..f52bda71dfb917829ae3a5b15e48e62558cf5520 100644 (file)
@@ -629,9 +629,6 @@ static inline void NCR5380_all_init (void)
 static void __init NCR5380_print_options (struct Scsi_Host *instance)
 {
     printk(" generic options"
-#ifdef AUTOSENSE 
-    " AUTOSENSE"
-#endif
 #ifdef REAL_DMA
     " REAL DMA"
 #endif
@@ -788,14 +785,6 @@ static int __init NCR5380_init(struct Scsi_Host *instance, int flags)
        first_instance = instance;
     }
        
-
-#ifndef AUTOSENSE
-    if ((instance->cmd_per_lun > 1) || (instance->can_queue > 1))
-        printk("scsi%d: WARNING : support for multiple outstanding commands enabled\n"
-               "        without AUTOSENSE option, contingent allegiance conditions may\n"
-               "        be incorrectly cleared.\n", HOSTNO);
-#endif /* def AUTOSENSE */
-
     NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE);
     NCR5380_write(MODE_REG, MR_BASE);
     NCR5380_write(TARGET_COMMAND_REG, 0);
@@ -2161,7 +2150,6 @@ static void NCR5380_information_transfer (struct Scsi_Host *instance)
                    else if (status_byte(cmd->SCp.Status) != GOOD)
                        cmd->result = (cmd->result & 0x00ffff) | (DID_ERROR << 16);
                    
-#ifdef AUTOSENSE
                    if ((cmd->cmnd[0] == REQUEST_SENSE) &&
                                                hostdata->ses.cmd_len) {
                        scsi_eh_restore_cmnd(cmd, &hostdata->ses);
@@ -2185,9 +2173,7 @@ static void NCR5380_information_transfer (struct Scsi_Host *instance)
                        local_irq_restore(flags);
                        dprintk(NDEBUG_QUEUES, "scsi%d: REQUEST SENSE added to head of "
                                  "issue queue\n", H_NO(cmd));
-                  } else
-#endif /* def AUTOSENSE */
-                  {
+                  } else {
 #ifdef NCR5380_STATS
                       collect_stats(hostdata, cmd);
 #endif
index 30353f6b599c4d2b512d4b54134bcc4409898d85..a5e58a7373beb4457baee90f803b1c6850516e96 100644 (file)
@@ -45,8 +45,6 @@
  * PARITY - enable parity checking.  Not supported.
  */
 
-#define AUTOSENSE
-
 #include <linux/types.h>
 #include <linux/stddef.h>
 #include <linux/ctype.h>
index f06ad91d737dd30b7afc8d64bf435055be12acc6..4253fe97c2104f3c9805233962da4053f4bc83bf 100644 (file)
@@ -1,4 +1,3 @@
-#define AUTOSENSE
 #define PSEUDO_DMA
 
 /*
@@ -40,9 +39,6 @@
 
 /*
  * Options : 
- * AUTOSENSE - if defined, REQUEST SENSE will be performed automatically
- *      for commands that return with a CHECK CONDITION status. 
- *
  * PSEUDO_DMA - enables PSEUDO-DMA hardware, should give a 3-4X performance
  * increase compared to polled I/O.
  *