linux-kernel - [PATCH] scsi: lpfc: lpfc_sli: Removed unnecessary 'return'

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 for Android: free password hash cracker in your pocket

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

Date:   Wed,  9 Jun 2021 16:01:51 +0800
From:   lijian_8010a29@....com
To:     james.smart@...adcom.com, dick.kennedy@...adcom.com,
        jejb@...ux.ibm.com, martin.petersen@...cle.com
Cc:     linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
        lijian <lijian@...ong.com>
Subject: [PATCH] scsi: lpfc: lpfc_sli: Removed unnecessary 'return'

From: lijian <lijian@...ong.com>

Removed unnecessary 'return'.

Signed-off-by: lijian <lijian@...ong.com>
---
 drivers/scsi/lpfc/lpfc_sli.c | 17 -----------------
 1 file changed, 17 deletions(-)

diff --git a/drivers/scsi/lpfc/lpfc_sli.c b/drivers/scsi/lpfc/lpfc_sli.c
index 368cbc10e0cf..14e1ddc5ac17 100644
--- a/drivers/scsi/lpfc/lpfc_sli.c
+++ b/drivers/scsi/lpfc/lpfc_sli.c
@@ -1544,7 +1544,6 @@ lpfc_sli_cancel_iocbs(struct lpfc_hba *phba, struct list_head *iocblist,
 			lpfc_sli_release_iocbq(phba, piocb);
 		}
 	}
-	return;
 }
 
 /**
@@ -2054,8 +2053,6 @@ lpfc_sli_resume_iocb(struct lpfc_hba *phba, struct lpfc_sli_ring *pring)
 		else
 			lpfc_sli_update_full_ring(phba, pring);
 	}
-
-	return;
 }
 
 /**
@@ -2566,7 +2563,6 @@ lpfc_sli_wake_mbox_wait(struct lpfc_hba *phba, LPFC_MBOXQ_t *pmboxq)
 	if (pmbox_done)
 		complete(pmbox_done);
 	spin_unlock_irqrestore(&phba->hbalock, drvr_flag);
-	return;
 }
 
 static void
@@ -3560,8 +3556,6 @@ lpfc_sli_rsp_pointers_error(struct lpfc_hba *phba, struct lpfc_sli_ring *pring)
 	phba->work_hs = HS_FFER3;
 
 	lpfc_worker_wake_up(phba);
-
-	return;
 }
 
 /**
@@ -3608,7 +3602,6 @@ void lpfc_poll_eratt(struct timer_list *t)
 		mod_timer(&phba->eratt_poll,
 			  jiffies +
 			  msecs_to_jiffies(1000 * phba->eratt_poll_interval));
-	return;
 }
 
 
@@ -4135,7 +4128,6 @@ lpfc_sli_handle_slow_ring_event_s3(struct lpfc_hba *phba,
 	}
 
 	spin_unlock_irqrestore(&phba->hbalock, iflag);
-	return;
 }
 
 /**
@@ -6439,8 +6431,6 @@ lpfc_set_features(struct lpfc_hba *phba, LPFC_MBOXQ_t *mbox,
 		mbox->u.mqe.un.set_feature.param_len = 4;
 		break;
 	}
-
-	return;
 }
 
 /**
@@ -8162,7 +8152,6 @@ lpfc_mbox_timeout(struct timer_list *t)
 
 	if (!tmo_posted)
 		lpfc_worker_wake_up(phba);
-	return;
 }
 
 /**
@@ -11572,7 +11561,6 @@ lpfc_sli_abort_els_cmpl(struct lpfc_hba *phba, struct lpfc_iocbq *cmdiocb,
 	}
 release_iocb:
 	lpfc_sli_release_iocbq(phba, cmdiocb);
-	return;
 }
 
 /**
@@ -11939,7 +11927,6 @@ lpfc_sli_abort_fcp_cmpl(struct lpfc_hba *phba, struct lpfc_iocbq *cmdiocb,
 			cmdiocb->iotag, rspiocb->iocb.ulpStatus,
 			rspiocb->iocb.un.ulpWord[4]);
 	lpfc_sli_release_iocbq(phba, cmdiocb);
-	return;
 }
 
 /**
@@ -12219,7 +12206,6 @@ lpfc_sli_wake_iocb_wait(struct lpfc_hba *phba,
 	if (pdone_q)
 		wake_up(pdone_q);
 	spin_unlock_irqrestore(&phba->hbalock, iflags);
-	return;
 }
 
 /**
@@ -15078,7 +15064,6 @@ lpfc_sli4_queue_free(struct lpfc_queue *queue)
 		list_del(&queue->cpu_list);
 
 	kfree(queue);
-	return;
 }
 
 /**
@@ -15306,7 +15291,6 @@ lpfc_modify_hba_eq_delay(struct lpfc_hba *phba, uint32_t startq,
 				shdr_status, shdr_add_status, rc);
 	}
 	mempool_free(mbox, phba->mbox_mem_pool);
-	return;
 }
 
 /**
@@ -19967,7 +19951,6 @@ lpfc_sli_read_link_ste(struct lpfc_hba *phba)
 
 out:
 	kfree(rgn23_data);
-	return;
 }
 
 /**
-- 
2.25.1