diff options
| author | Finn Thain <fthain@telegraphics.com.au> | 2017-10-26 01:51:50 -0400 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2017-10-31 12:28:04 -0400 |
| commit | 39bef87c140aa9586ba55ede3bb856261c5e9d57 (patch) | |
| tree | 845366f3e9048fe250b5b036b853ba2c53df0a18 | |
| parent | d84c198f43c50c6c0bd57571acbf0f000165bd56 (diff) | |
scsi: NCR5380: Suppress SDTR and WDTR message logging
The 5380 drivers only support asynchronous transfers and the 5380
controllers only have narrow busses. Hence, the core driver will reject
any SDTR and WDTR messages from target devices. Don't log this, it's
expected behaviour. Also, fix the off-by-one array indices in the
arguments to scmd_printk().
Tested-by: Stan Johnson <userm57@yahoo.com>
Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
| -rw-r--r-- | drivers/scsi/NCR5380.c | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/drivers/scsi/NCR5380.c b/drivers/scsi/NCR5380.c index 8a0812221d72..ccd912e7b63e 100644 --- a/drivers/scsi/NCR5380.c +++ b/drivers/scsi/NCR5380.c | |||
| @@ -1907,8 +1907,6 @@ static void NCR5380_information_transfer(struct Scsi_Host *instance) | |||
| 1907 | switch (extended_msg[2]) { | 1907 | switch (extended_msg[2]) { |
| 1908 | case EXTENDED_SDTR: | 1908 | case EXTENDED_SDTR: |
| 1909 | case EXTENDED_WDTR: | 1909 | case EXTENDED_WDTR: |
| 1910 | case EXTENDED_MODIFY_DATA_POINTER: | ||
| 1911 | case EXTENDED_EXTENDED_IDENTIFY: | ||
| 1912 | tmp = 0; | 1910 | tmp = 0; |
| 1913 | } | 1911 | } |
| 1914 | } else if (len) { | 1912 | } else if (len) { |
| @@ -1931,18 +1929,14 @@ static void NCR5380_information_transfer(struct Scsi_Host *instance) | |||
| 1931 | * reject it. | 1929 | * reject it. |
| 1932 | */ | 1930 | */ |
| 1933 | default: | 1931 | default: |
| 1934 | if (!tmp) { | 1932 | if (tmp == EXTENDED_MESSAGE) |
| 1935 | shost_printk(KERN_ERR, instance, "rejecting message "); | ||
| 1936 | spi_print_msg(extended_msg); | ||
| 1937 | printk("\n"); | ||
| 1938 | } else if (tmp != EXTENDED_MESSAGE) | ||
| 1939 | scmd_printk(KERN_INFO, cmd, | ||
| 1940 | "rejecting unknown message %02x\n", | ||
| 1941 | tmp); | ||
| 1942 | else | ||
| 1943 | scmd_printk(KERN_INFO, cmd, | 1933 | scmd_printk(KERN_INFO, cmd, |
| 1944 | "rejecting unknown extended message code %02x, length %d\n", | 1934 | "rejecting unknown extended message code %02x, length %d\n", |
| 1945 | extended_msg[1], extended_msg[0]); | 1935 | extended_msg[2], extended_msg[1]); |
| 1936 | else if (tmp) | ||
| 1937 | scmd_printk(KERN_INFO, cmd, | ||
| 1938 | "rejecting unknown message code %02x\n", | ||
| 1939 | tmp); | ||
| 1946 | 1940 | ||
| 1947 | msgout = MESSAGE_REJECT; | 1941 | msgout = MESSAGE_REJECT; |
| 1948 | NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE | ICR_ASSERT_ATN); | 1942 | NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE | ICR_ASSERT_ATN); |