[PATCH 17/34] qla2xxx: Simplify qlt_send_term_imm_notif()

[Bart Van Assche]

All qlt_send_term_imm_notif() callers pass '1' as second argument to this
function. Hence remove the (broken) code that depends on that second
argument having another value. Add a pr_debug() statement that prints rc
to avoid that the compiler would complain that rc has been set but is not
used.

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_target.c | 26 ++------------------------
 1 file changed, 2 insertions(+), 24 deletions(-)

diff --git a/drivers/scsi/qla2xxx/qla_target.c 
b/drivers/scsi/qla2xxx/qla_target.c
index f25535fb30f5..69fc29d7e59d 100644
--- a/drivers/scsi/qla2xxx/qla_target.c
+++ b/drivers/scsi/qla2xxx/qla_target.c
@@ -3651,33 +3651,11 @@ static int __qlt_send_term_imm_notif(struct 
scsi_qla_host *vha,
 static void qlt_send_term_imm_notif(struct scsi_qla_host *vha,
        struct imm_ntfy_from_isp *imm, int ha_locked)
 {
-       unsigned long flags = 0;
        int rc;
 
-       if (ha_locked) {
-               rc = __qlt_send_term_imm_notif(vha, imm);
-
-#if 0  /* Todo  */
-               if (rc == -ENOMEM)
-                       qlt_alloc_qfull_cmd(vha, imm, 0, 0);
-#else
-               if (rc) {
-               }
-#endif
-               goto done;
-       }
-
-       spin_lock_irqsave(&vha->hw->hardware_lock, flags);
+       WARN_ON_ONCE(!ha_locked);
        rc = __qlt_send_term_imm_notif(vha, imm);
-
-#if 0  /* Todo */
-       if (rc == -ENOMEM)
-               qlt_alloc_qfull_cmd(vha, imm, 0, 0);
-#endif
-
-done:
-       if (!ha_locked)
-               spin_unlock_irqrestore(&vha->hw->hardware_lock, flags);
+       pr_debug("rc = %d\n", rc);
 }
 
 /*
-- 
2.21.0.196.g041f5ea1cf98