diff options
| author | Bart Van Assche <[email protected]> | 2021-04-15 15:08:23 -0700 |
|---|---|---|
| committer | Martin K. Petersen <[email protected]> | 2021-04-15 22:44:41 -0400 |
| commit | 15df85e0d63d870e67fbd39c416f1d9815a107d0 (patch) | |
| tree | 38df94deee83cf92435ff0df46ce6a7b996db3f8 | |
| parent | 41e70e3006f63b89ae3b0d49557ac9f620bac524 (diff) | |
scsi: target: Compare explicitly with SAM_STAT_GOOD
Instead of leaving it implicit that SAM_STAT_GOOD == 0, compare explicitly
with SAM_STAT_GOOD.
Link: https://lore.kernel.org/r/[email protected]
Reviewed-by: Mike Christie <[email protected]>
Signed-off-by: Bart Van Assche <[email protected]>
Signed-off-by: Martin K. Petersen <[email protected]>
| -rw-r--r-- | drivers/target/target_core_pscsi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/target/target_core_pscsi.c b/drivers/target/target_core_pscsi.c index 1c9aeab93477..dac44caf77a3 100644 --- a/drivers/target/target_core_pscsi.c +++ b/drivers/target/target_core_pscsi.c @@ -1046,7 +1046,7 @@ static void pscsi_req_done(struct request *req, blk_status_t status) int result = scsi_req(req)->result; u8 scsi_status = status_byte(result) << 1; - if (scsi_status) { + if (scsi_status != SAM_STAT_GOOD) { pr_debug("PSCSI Status Byte exception at cmd: %p CDB:" " 0x%02x Result: 0x%08x\n", cmd, pt->pscsi_cdb[0], result); |