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.


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

Reply via email to