]> git.baikalelectronics.ru Git - kernel.git/commitdiff
USB: UAS: use macro for reporting results
authorOliver Neukum <oneukum@suse.com>
Wed, 16 Sep 2020 09:40:26 +0000 (11:40 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 16 Sep 2020 10:43:35 +0000 (12:43 +0200)
The SCSI layer has introduced a new macro for recording the result
of a command. Use it.

Signed-off-by: Oliver Neukum <oneukum@suse.com>
Link: https://lore.kernel.org/r/20200916094026.30085-3-oneukum@suse.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/storage/uas.c

index abf7e3d6423425e3689cf7f5855163cb81e1e37e..c8a577309e8fbe027a103effdd9f01e156a8ca17 100644 (file)
@@ -279,17 +279,17 @@ static bool uas_evaluate_response_iu(struct response_iu *riu, struct scsi_cmnd *
 
        switch (response_code) {
        case RC_INCORRECT_LUN:
-               cmnd->result = DID_BAD_TARGET << 16;
+               set_host_byte(cmnd, DID_BAD_TARGET);
                break;
        case RC_TMF_SUCCEEDED:
-               cmnd->result = DID_OK << 16;
+               set_host_byte(cmnd, DID_OK);
                break;
        case RC_TMF_NOT_SUPPORTED:
-               cmnd->result = DID_TARGET_FAILURE << 16;
+               set_host_byte(cmnd, DID_TARGET_FAILURE);
                break;
        default:
                uas_log_cmd_state(cmnd, "response iu", response_code);
-               cmnd->result = DID_ERROR << 16;
+               set_host_byte(cmnd, DID_ERROR);
                break;
        }
 
@@ -660,7 +660,7 @@ static int uas_queuecommand_lck(struct scsi_cmnd *cmnd,
        spin_lock_irqsave(&devinfo->lock, flags);
 
        if (devinfo->resetting) {
-               cmnd->result = DID_ERROR << 16;
+               set_host_byte(cmnd, DID_ERROR);
                cmnd->scsi_done(cmnd);
                goto zombie;
        }
@@ -704,7 +704,7 @@ static int uas_queuecommand_lck(struct scsi_cmnd *cmnd,
         * of queueing, no matter how fatal the error
         */
        if (err == -ENODEV) {
-               cmnd->result = DID_ERROR << 16;
+               set_host_byte(cmnd, DID_ERROR);
                cmnd->scsi_done(cmnd);
                goto zombie;
        }