diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2017-11-27 13:45:59 +0100 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2017-12-21 09:30:32 +0100 |
| commit | 9661e208f8cc94f66176db6f069a0f8adef9478d (patch) | |
| tree | e43c7b08b87af097a8165ab0208fe03961aaf741 /scsi | |
| parent | f68d98b21fa74155dc7c1fd212474379ac3c7531 (diff) | |
| download | qemu-9661e208f8cc94f66176db6f069a0f8adef9478d.zip qemu-9661e208f8cc94f66176db6f069a0f8adef9478d.tar.gz qemu-9661e208f8cc94f66176db6f069a0f8adef9478d.tar.bz2 | |
scsi: replace hex constants with #defines
Sense keys have nice #defines in scsi/constants.h, use them.
Reported-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'scsi')
| -rw-r--r-- | scsi/utils.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/scsi/utils.c b/scsi/utils.c index 61bc1a8..ddae650 100644 --- a/scsi/utils.c +++ b/scsi/utils.c @@ -339,16 +339,16 @@ int scsi_convert_sense(uint8_t *in_buf, int in_len, int scsi_sense_to_errno(int key, int asc, int ascq) { switch (key) { - case 0x00: /* NO SENSE */ - case 0x01: /* RECOVERED ERROR */ - case 0x06: /* UNIT ATTENTION */ + case NO_SENSE: + case RECOVERED_ERROR: + case UNIT_ATTENTION: /* These sense keys are not errors */ return 0; - case 0x0b: /* COMMAND ABORTED */ + case ABORTED_COMMAND: /* COMMAND ABORTED */ return ECANCELED; - case 0x02: /* NOT READY */ - case 0x05: /* ILLEGAL REQUEST */ - case 0x07: /* DATA PROTECTION */ + case NOT_READY: + case ILLEGAL_REQUEST: + case DATA_PROTECT: /* Parse ASCQ */ break; default: |