diff options
author | Bart Van Assche <bvanassche@acm.org> | 2021-10-07 22:29:03 +0200 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2021-10-17 03:31:41 +0200 |
commit | c33a2dca9853584c97588bc4eb2ca10a770d0a28 (patch) | |
tree | 868f10f27fead40a2e29d283fba0852a36046c15 /drivers/scsi/qlogicpti.c | |
parent | scsi: qlogicfas408: Call scsi_done() directly (diff) | |
download | linux-c33a2dca9853584c97588bc4eb2ca10a770d0a28.tar.xz linux-c33a2dca9853584c97588bc4eb2ca10a770d0a28.zip |
scsi: qlogicpti: Call scsi_done() directly
Conditional statements are faster than indirect calls. Hence call
scsi_done() directly.
Link: https://lore.kernel.org/r/20211007202923.2174984-69-bvanassche@acm.org
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/scsi/qlogicpti.c')
-rw-r--r-- | drivers/scsi/qlogicpti.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/scsi/qlogicpti.c b/drivers/scsi/qlogicpti.c index 8e7e833a36cc..30b5e98b5de0 100644 --- a/drivers/scsi/qlogicpti.c +++ b/drivers/scsi/qlogicpti.c @@ -1021,8 +1021,6 @@ static int qlogicpti_queuecommand_lck(struct scsi_cmnd *Cmnd, void (*done)(struc u_int out_ptr; int in_ptr; - Cmnd->scsi_done = done; - in_ptr = qpti->req_in_ptr; cmd = (struct Command_Entry *) &qpti->req_cpu[in_ptr]; out_ptr = sbus_readw(qpti->qregs + MBOX4); @@ -1214,7 +1212,7 @@ static irqreturn_t qpti_intr(int irq, void *dev_id) struct scsi_cmnd *next; next = (struct scsi_cmnd *) dq->host_scribble; - dq->scsi_done(dq); + scsi_done(dq); dq = next; } while (dq != NULL); } |