[PATCH] scsi: csiostor: Remove unused variable n

Colin Ian King posted 1 patch 3 years, 5 months ago
drivers/scsi/csiostor/csio_wr.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
[PATCH] scsi: csiostor: Remove unused variable n
Posted by Colin Ian King 3 years, 5 months ago
Variable n is just being incremented and it's never used anywhere else. The
variable and the increment are redundant so remove it.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
 drivers/scsi/csiostor/csio_wr.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/scsi/csiostor/csio_wr.c b/drivers/scsi/csiostor/csio_wr.c
index fe0355c964bc..a516df019c22 100644
--- a/drivers/scsi/csiostor/csio_wr.c
+++ b/drivers/scsi/csiostor/csio_wr.c
@@ -1051,7 +1051,6 @@ csio_wr_process_fl(struct csio_hw *hw, struct csio_q *q,
 	struct csio_fl_dma_buf flb;
 	struct csio_dma_buf *buf, *fbuf;
 	uint32_t bufsz, len, lastlen = 0;
-	int n;
 	struct csio_q *flq = hw->wrm.q_arr[q->un.iq.flq_idx];
 
 	CSIO_DB_ASSERT(flq != NULL);
@@ -1071,7 +1070,7 @@ csio_wr_process_fl(struct csio_hw *hw, struct csio_q *q,
 	flb.totlen = len;
 
 	/* Consume all freelist buffers used for len bytes */
-	for (n = 0, fbuf = flb.flbufs; ; n++, fbuf++) {
+	for (fbuf = flb.flbufs; ; fbuf++) {
 		buf = &flq->un.fl.bufs[flq->cidx];
 		bufsz = csio_wr_fl_bufsz(sge, buf);
 
-- 
2.37.3
Re: [PATCH] scsi: csiostor: Remove unused variable n
Posted by Martin K. Petersen 3 years, 4 months ago
On Tue, 1 Nov 2022 12:32:35 +0000, Colin Ian King wrote:

> Variable n is just being incremented and it's never used anywhere else. The
> variable and the increment are redundant so remove it.
> 
> 

Applied to 6.2/scsi-queue, thanks!

[1/1] scsi: csiostor: Remove unused variable n
      https://git.kernel.org/mkp/scsi/c/0aa46eba2955

-- 
Martin K. Petersen	Oracle Linux Engineering
Re: [PATCH] scsi: csiostor: Remove unused variable n
Posted by Martin K. Petersen 3 years, 5 months ago
Colin,

> Variable n is just being incremented and it's never used anywhere
> else. The variable and the increment are redundant so remove it.

Applied to 6.2/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering