Re: [PATCH 22/43] qla2xxx: Add switch command to simplify fabric discovery

[Madhani, Himanshu]

Hi Ewan, 

> On Dec 20, 2017, at 12:09 PM, Ewan D. Milne <emi...@redhat.com> wrote:
> 
> On Tue, 2017-12-19 at 22:56 -0800, Himanshu Madhani wrote:
>> @@ -4915,12 +4941,32 @@ static int qlt_24xx_handle_els(struct scsi_qla_host 
>> *vha,
>>              }
>> 
>>              if (sess != NULL) {
>> +                    bool delete = false;
>>                      spin_lock_irqsave(&tgt->ha->tgt.sess_lock, flags);
>>                      switch (sess->fw_login_state) {
>> +                    case DSC_LS_PLOGI_PEND:
>>                      case DSC_LS_PLOGI_COMP:
>>                      case DSC_LS_PRLI_COMP:
>>                              break;
>>                      default:
>> +                            delete = true;
> 
> Nitpick:  git quiltimport complains about the whitespace error
> (space before tab) on the above line.
> 
> 

Thanks for review. Will fix in v2

Thanks,
- Himanshu