On 8/3/16, 11:42 AM, "Dan Carpenter" <[email protected]> wrote:
>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 <[email protected]> > >diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c >index 2674f4c..1dd8650 100644 >--- a/drivers/scsi/qla2xxx/qla_os.c >+++ b/drivers/scsi/qla2xxx/qla_os.c >@@ -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); > >- 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); > } > } Looks Good. Acked-by: Himanshu Madhani <[email protected]> >
