diff options
| author | Colin Ian King <colin.king@canonical.com> | 2020-07-23 16:26:14 +0200 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2020-07-25 04:09:56 +0200 |
| commit | 584d902eb10e845cf3e1828b63b184b35ac2e023 (patch) | |
| tree | 9210fce2263b6b58e20e4cbaa7514d721d57dde8 /drivers/scsi/isci | |
| parent | scsi: libcxgbi: Remove unnecessary NULL checks for 'tdata' pointer (diff) | |
| download | linux-584d902eb10e845cf3e1828b63b184b35ac2e023.tar.xz linux-584d902eb10e845cf3e1828b63b184b35ac2e023.zip | |
scsi: isci: Remove redundant initialization of variable 'status'
The variable 'status' is being initialized with a value that is never read
and it is being updated later with a new value. The initialization is
redundant and can be removed.
Link: https://lore.kernel.org/r/20200723142614.991416-1-colin.king@canonical.com
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Addresses-Coverity: ("Unused value")
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 343d24c7e788..6561a07db189 100644 --- a/drivers/scsi/isci/request.c +++ b/drivers/scsi/isci/request.c @@ -3444,7 +3444,7 @@ struct isci_request *isci_tmf_request_from_tag(struct isci_host *ihost, int isci_request_execute(struct isci_host *ihost, struct isci_remote_device *idev, struct sas_task *task, u16 tag) { - enum sci_status status = SCI_FAILURE_UNSUPPORTED_PROTOCOL; + enum sci_status status; struct isci_request *ireq; unsigned long flags; int ret = 0; |