diff options
| author | Bart Van Assche | 2021-10-07 13:29:02 -0700 |
|---|---|---|
| committer | Martin K. Petersen | 2021-10-16 21:31:41 -0400 |
| commit | da65bc05cf9166478c38b63ec05b8c9fad2f7962 (patch) | |
| tree | 72ea9123dd578311bba97e19eba1e0c01c208034 | |
| parent | fdcfbd6517d9bff7afe31119a9ee7f386cf19b89 (diff) | |
scsi: qlogicfas408: Call scsi_done() directly
Conditional statements are faster than indirect calls. Hence call
scsi_done() directly.
Link: https://lore.kernel.org/r/20211007202923.2174984-68-bvanassche@acm.org
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
| -rw-r--r-- | drivers/scsi/qlogicfas408.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/qlogicfas408.c b/drivers/scsi/qlogicfas408.c index 3bbe0b5545d9..5471c046a4b7 100644 --- a/drivers/scsi/qlogicfas408.c +++ b/drivers/scsi/qlogicfas408.c @@ -442,7 +442,7 @@ static void ql_ihandl(void *dev_id) * If result is CHECK CONDITION done calls qcommand to request * sense */ - (icmd->scsi_done) (icmd); + scsi_done(icmd); } irqreturn_t qlogicfas408_ihandl(int irq, void *dev_id) @@ -473,7 +473,6 @@ static int qlogicfas408_queuecommand_lck(struct scsi_cmnd *cmd, return 0; } - cmd->scsi_done = done; /* wait for the last command's interrupt to finish */ while (priv->qlcmd != NULL) { barrier(); |