> On Jun 19, 2017, at 1:09 AM, Johannes Thumshirn <jthumsh...@suse.de> wrote:
> 
> On Fri, Jun 16, 2017 at 03:47:39PM -0700, Himanshu Madhani wrote:
>> From: Duane Grigsby <duane.grig...@cavium.com>
>> 
>> Signed-off-by: Darren Trapp <darren.tr...@cavium.com>
>> Signed-off-by: Duane Grigsby <duane.grig...@cavium.com>
>> Signed-off-by: Anil Gurumurthy <anil.gurumur...@cavium.com>
>> Signed-off-by: Giridhar Malavali <giridhar.malav...@cavium.com>
>> Signed-off-by: Himanshu Madhani <himanshu.madh...@cavium.com>
>> ---
> 
> [...]
> 
>> +
>> +    if ((fcport->fw_login_state == DSC_LS_PLOGI_PEND) ||
>> +        (fcport->fw_login_state == DSC_LS_PLOGI_COMP) ||
>> +        (fcport->fw_login_state == DSC_LS_PRLI_PEND))
>> +            return rval;
> 
> Minor nit, superfluous parenthesis.
> 
> 

Will fix it in v2

> Anyways,
> Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de>
> 
> -- 
> Johannes Thumshirn                                          Storage
> jthumsh...@suse.de                                +49 911 74053 689
> SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
> GF: Felix Imendörffer, Jane Smithard, Graham Norton
> HRB 21284 (AG Nürnberg)
> Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850

Thanks,
- Himanshu

Reply via email to