diff options
| author | Su Hui <suhui@nfschina.com> | 2024-01-12 05:19:27 +0100 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2024-01-17 20:43:41 +0100 |
| commit | 658365c6b0857e6a306436e315a8633937e3af42 (patch) | |
| tree | 4f309ebbff91bc366142165b55c05fb2bf5cc458 /drivers/scsi/isci | |
| parent | scsi: target: core: Add TMF to tmr_list handling (diff) | |
| download | linux-658365c6b0857e6a306436e315a8633937e3af42.tar.xz linux-658365c6b0857e6a306436e315a8633937e3af42.zip | |
scsi: isci: Fix an error code problem in isci_io_request_build()
Clang static complains that Value stored to 'status' is never read. Return
'status' rather than 'SCI_SUCCESS'.
Fixes: f1f52e75939b ("isci: uplevel request infrastructure")
Signed-off-by: Su Hui <suhui@nfschina.com>
Link: https://lore.kernel.org/r/20240112041926.3924315-1-suhui@nfschina.com
Reviewed-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/scsi/isci')
| -rw-r--r-- | drivers/scsi/isci/request.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/isci/request.c b/drivers/scsi/isci/request.c index 71f711cb0628..355a0bc0828e 100644 --- a/drivers/scsi/isci/request.c +++ b/drivers/scsi/isci/request.c @@ -3387,7 +3387,7 @@ static enum sci_status isci_io_request_build(struct isci_host *ihost, return SCI_FAILURE; } - return SCI_SUCCESS; + return status; } static struct isci_request *isci_request_from_tag(struct isci_host *ihost, u16 tag) |