[PATCH 44/59] qla2xxx: Introduce the function qla2xxx_init_sp()

[Bart Van Assche]

This patch does not change any functionality but makes the next patch
easier to read.

Cc: Himanshu Madhani <hmadh...@marvell.com>
Cc: Giridhar Malavali <gmalav...@marvell.com>
Signed-off-by: Bart Van Assche <bvanass...@acm.org>
---
 drivers/scsi/qla2xxx/qla_inline.h | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/drivers/scsi/qla2xxx/qla_inline.h 
b/drivers/scsi/qla2xxx/qla_inline.h
index bf063c664352..0c3d907af769 100644
--- a/drivers/scsi/qla2xxx/qla_inline.h
+++ b/drivers/scsi/qla2xxx/qla_inline.h
@@ -152,6 +152,18 @@ qla2x00_chip_is_down(scsi_qla_host_t *vha)
        return (qla2x00_reset_active(vha) || !vha->hw->flags.fw_started);
 }
 
+static void qla2xxx_init_sp(srb_t *sp, scsi_qla_host_t *vha,
+                           struct qla_qpair *qpair, fc_port_t *fcport)
+{
+       memset(sp, 0, sizeof(*sp));
+       sp->fcport = fcport;
+       sp->iocbs = 1;
+       sp->vha = vha;
+       sp->qpair = qpair;
+       sp->cmd_type = TYPE_SRB;
+       INIT_LIST_HEAD(&sp->elem);
+}
+
 static inline srb_t *
 qla2xxx_get_qpair_sp(scsi_qla_host_t *vha, struct qla_qpair *qpair,
     fc_port_t *fcport, gfp_t flag)
@@ -164,19 +176,9 @@ qla2xxx_get_qpair_sp(scsi_qla_host_t *vha, struct 
qla_qpair *qpair,
                return NULL;
 
        sp = mempool_alloc(qpair->srb_mempool, flag);
-       if (!sp)
-               goto done;
-
-       memset(sp, 0, sizeof(*sp));
-       sp->fcport = fcport;
-       sp->iocbs = 1;
-       sp->vha = vha;
-       sp->qpair = qpair;
-       sp->cmd_type = TYPE_SRB;
-       INIT_LIST_HEAD(&sp->elem);
-
-done:
-       if (!sp)
+       if (sp)
+               qla2xxx_init_sp(sp, vha, qpair, fcport);
+       else
                QLA_QPAIR_MARK_NOT_BUSY(qpair);
        return sp;
 }
-- 
2.22.0.770.g0f2c4a37fd-goog