| Message ID | YJ2mMHNqAgTNVVj+@fedora (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | scsi: be2iscsi: Remove redundant initialization | expand |
On Thu, 13 May 2021 17:20:32 -0500, Nigel Christian wrote: > The nested for loop variables i and j in beiscsi_free_mem() are > initialized twice. The values outside of the loops are redundant > and can be removed. Applied to 5.14/scsi-queue, thanks! [1/1] scsi: be2iscsi: Remove redundant initialization https://git.kernel.org/mkp/scsi/c/0edca4fc633c
diff --git a/drivers/scsi/be2iscsi/be_main.c b/drivers/scsi/be2iscsi/be_main.c index 22cf7f4b8d8c..c15cc6c164d9 100644 --- a/drivers/scsi/be2iscsi/be_main.c +++ b/drivers/scsi/be2iscsi/be_main.c @@ -3858,8 +3858,6 @@ static void beiscsi_free_mem(struct beiscsi_hba *phba) int i, j; mem_descr = phba->init_mem; - i = 0; - j = 0; for (i = 0; i < SE_MEM_MAX; i++) { for (j = mem_descr->num_elements; j > 0; j--) { dma_free_coherent(&phba->pcidev->dev,
The nested for loop variables i and j in beiscsi_free_mem() are initialized twice. The values outside of the loops are redundant and can be removed. Addresses-Coverity: ("Unused value") Signed-off-by: Nigel Christian <nigel.l.christian@gmail.com> --- drivers/scsi/be2iscsi/be_main.c | 2 -- 1 file changed, 2 deletions(-)