scsi: qla4xxx: Remove redundant check on drvr_wait

The check for a non-zero drvr_wait is redundant as the same check is performed
earlier in the outer while loop, the inner check will always be true if we
reached this point inside the while loop.  Remove the redundant if check.

Detected by cppcheck:
(warning) Identical inner 'if' condition is always true.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Manish Rangankar <Manish.Rangankar@cavium.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/scsi/qla4xxx/ql4_init.c b/drivers/scsi/qla4xxx/ql4_init.c
index 52b1a0bc93c9ddcb6c00d73d334079284dc089ec..1ef74aa2d00ac7cacb4c7232c619352fd33869f1 100644 (file)
--- a/drivers/scsi/qla4xxx/ql4_init.c
+++ b/drivers/scsi/qla4xxx/ql4_init.c
@@ -766,12 +766,10 @@ int ql4xxx_lock_drvr_wait(struct scsi_qla_host *a)
        while (drvr_wait) {
                if (ql4xxx_lock_drvr(a) == 0) {
                        ssleep(QL4_LOCK_DRVR_SLEEP);
-                       if (drvr_wait) {
-                               DEBUG2(printk("scsi%ld: %s: Waiting for "
-                                             "Global Init Semaphore(%d)...\n",
-                                             a->host_no,
-                                             __func__, drvr_wait));
-                       }
+                       DEBUG2(printk("scsi%ld: %s: Waiting for "
+                                     "Global Init Semaphore(%d)...\n",
+                                     a->host_no,
+                                     __func__, drvr_wait));
                        drvr_wait -= QL4_LOCK_DRVR_SLEEP;
                } else {
                        DEBUG2(printk("scsi%ld: %s: Global Init Semaphore "