linux-kernel - [PATCH] scsi: isci: fix unneeded variable

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:   Sat, 25 Jan 2020 22:58:54 +0530
From:   Saurav Girepunje <saurav.girepunje@...il.com>
To:     intel-linux-scu@...el.com, artur.paszkiewicz@...el.com,
        jejb@...ux.ibm.com, martin.petersen@...cle.com,
        linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org
Cc:     saurav.girepunje@...mail.com
Subject: [PATCH] scsi: isci: fix unneeded variable

remove unneeded variable status in request.c file.

Signed-off-by: Saurav Girepunje <saurav.girepunje@...il.com>
---
  drivers/scsi/isci/request.c | 8 +++-----
  1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/drivers/scsi/isci/request.c b/drivers/scsi/isci/request.c
index 343d24c..9849366 100644
--- a/drivers/scsi/isci/request.c
+++ b/drivers/scsi/isci/request.c
@@ -1480,7 +1480,6 @@ static enum sci_status sci_stp_request_pio_data_in_copy_data(
  stp_request_pio_await_h2d_completion_tc_event(struct isci_request *ireq,
  					      u32 completion_code)
  {
-	enum sci_status status = SCI_SUCCESS;
  
  	switch (SCU_GET_COMPLETION_TL_STATUS(completion_code)) {
  	case SCU_MAKE_COMPLETION_STATUS(SCU_TASK_DONE_GOOD):
@@ -1500,7 +1499,7 @@ static enum sci_status sci_stp_request_pio_data_in_copy_data(
  		break;
  	}
  
-	return status;
+	return SCI_SUCCESS;
  }
  
  static enum sci_status
@@ -2148,13 +2147,12 @@ static enum sci_status stp_request_udma_await_tc_event(struct isci_request *ireq
  		break;
  	}
  
-	return status;
+	return SCI_SUCCESS;
  }
  
  static enum sci_status atapi_raw_completion(struct isci_request *ireq, u32 completion_code,
  						  enum sci_base_request_states next)
  {
-	enum sci_status status = SCI_SUCCESS;
  
  	switch (SCU_GET_COMPLETION_TL_STATUS(completion_code)) {
  	case SCU_MAKE_COMPLETION_STATUS(SCU_TASK_DONE_GOOD):
@@ -2174,7 +2172,7 @@ static enum sci_status atapi_raw_completion(struct isci_request *ireq, u32 compl
  		break;
  	}
  
-	return status;
+	return SCI_SUCCESS;
  }
  
  static enum sci_status atapi_data_tc_completion_handler(struct isci_request *ireq,
-- 
1.9.1