diff mbox

[2/5] scsi: Add VENDOR_SPECIFIC sense code definitions

Message ID 20180119151113.80768-3-hare@suse.de (mailing list archive)
State Accepted
Headers show

Commit Message

Hannes Reinecke Jan. 19, 2018, 3:11 p.m. UTC
Some older devices will return vendor specific sense codes, so
we should be adding a definition for it.

Signed-off-by: Hannes Reinecke <hare@suse.com>
---
 include/scsi/scsi_proto.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Bart Van Assche Jan. 19, 2018, 6:47 p.m. UTC | #1
On Fri, 2018-01-19 at 16:11 +0100, Hannes Reinecke wrote:
> Some older devices will return vendor specific sense codes, so

> we should be adding a definition for it.

> 

> Signed-off-by: Hannes Reinecke <hare@suse.com>

> ---

>  include/scsi/scsi_proto.h | 1 +

>  1 file changed, 1 insertion(+)

> 

> diff --git a/include/scsi/scsi_proto.h b/include/scsi/scsi_proto.h

> index 1df8efb0ee01..c36860111932 100644

> --- a/include/scsi/scsi_proto.h

> +++ b/include/scsi/scsi_proto.h

> @@ -236,6 +236,7 @@ struct scsi_varlen_cdb_hdr {

>  #define UNIT_ATTENTION      0x06

>  #define DATA_PROTECT        0x07

>  #define BLANK_CHECK         0x08

> +#define VENDOR_SPECIFIC     0x09

>  #define COPY_ABORTED        0x0a

>  #define ABORTED_COMMAND     0x0b

>  #define VOLUME_OVERFLOW     0x0d


Reviewed-by: Bart Van Assche <bart.vanassche@wdc.com>
Martin K. Petersen Jan. 23, 2018, 12:27 a.m. UTC | #2
Hannes,

> Some older devices will return vendor specific sense codes, so
> we should be adding a definition for it.

Applied to 4.16/scsi-queue. Thanks!
diff mbox

Patch

diff --git a/include/scsi/scsi_proto.h b/include/scsi/scsi_proto.h
index 1df8efb0ee01..c36860111932 100644
--- a/include/scsi/scsi_proto.h
+++ b/include/scsi/scsi_proto.h
@@ -236,6 +236,7 @@  struct scsi_varlen_cdb_hdr {
 #define UNIT_ATTENTION      0x06
 #define DATA_PROTECT        0x07
 #define BLANK_CHECK         0x08
+#define VENDOR_SPECIFIC     0x09
 #define COPY_ABORTED        0x0a
 #define ABORTED_COMMAND     0x0b
 #define VOLUME_OVERFLOW     0x0d