Commit fc65df48 authored by Colin Ian King's avatar Colin Ian King Committed by Martin K. Petersen
Browse files

scsi: qedf: Remove redundant variable op 

The variable 'op' is assigned a value and is never read. The variable is
not used and is redundant, remove it.

Link: https://lore.kernel.org/r/20220517092518.93159-1-colin.i.king@gmail.com


Signed-off-by: default avatarColin Ian King <colin.i.king@gmail.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 6c6ac8b7

drivers/scsi/qedf/qedf_io.c

+1 −2
Original line number Diff line number Diff line
@@ -804,7 +804,6 @@ static void qedf_trace_io(struct qedf_rport *fcport, struct qedf_ioreq *io_req, 
	struct qedf_io_log *io_log;

	struct scsi_cmnd *sc_cmd = io_req->sc_cmd;

	unsigned long flags;

	uint8_t op;



	spin_lock_irqsave(&qedf->io_trace_lock, flags);
@@ -813,7 +812,7 @@ static void qedf_trace_io(struct qedf_rport *fcport, struct qedf_ioreq *io_req, 
	io_log->task_id = io_req->xid;

	io_log->port_id = fcport->rdata->ids.port_id;

	io_log->lun = sc_cmd->device->lun;

	io_log->op = op = sc_cmd->cmnd[0];

	io_log->op = sc_cmd->cmnd[0];

	io_log->lba[0] = sc_cmd->cmnd[2];

	io_log->lba[1] = sc_cmd->cmnd[3];

	io_log->lba[2] = sc_cmd->cmnd[4];