qla2xxx: small cleanup in qla2x00_wait_for_hba_ready()
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 3 Aug 2016 18:42:32 +0000 (21:42 +0300)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 9 Aug 2016 01:28:43 +0000 (21:28 -0400)
The "if (test_bit(UNLOADING..." line was indented one tab more than it
should have been.  There was an extra parenthesis around the
qla2x00_reset_active() function call.  I lined up the conditions a bit
so that it shows how they group together.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Himanshu Madhani <himanshu.madhani@qlogic.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qla2xxx/qla_os.c

index 2674f4c..1dd8650 100644 (file)
@@ -899,12 +899,12 @@ qla2x00_wait_for_hba_ready(scsi_qla_host_t *vha)
        struct qla_hw_data *ha = vha->hw;
        scsi_qla_host_t *base_vha = pci_get_drvdata(ha->pdev);
 
        struct qla_hw_data *ha = vha->hw;
        scsi_qla_host_t *base_vha = pci_get_drvdata(ha->pdev);
 
-       while (((qla2x00_reset_active(vha)) || ha->dpc_active ||
-           ha->flags.mbox_busy) ||
-               test_bit(FX00_RESET_RECOVERY, &vha->dpc_flags) ||
-               test_bit(FX00_TARGET_SCAN, &vha->dpc_flags)) {
-                       if (test_bit(UNLOADING, &base_vha->dpc_flags))
-                               break;
+       while ((qla2x00_reset_active(vha) || ha->dpc_active ||
+               ha->flags.mbox_busy) ||
+              test_bit(FX00_RESET_RECOVERY, &vha->dpc_flags) ||
+              test_bit(FX00_TARGET_SCAN, &vha->dpc_flags)) {
+               if (test_bit(UNLOADING, &base_vha->dpc_flags))
+                       break;
                msleep(1000);
        }
 }
                msleep(1000);
        }
 }