On Fri, Jun 16, 2017 at 03:47:39PM -0700, Himanshu Madhani wrote: > From: Duane Grigsby <duane.grigsby@xxxxxxxxxx> > > Signed-off-by: Darren Trapp <darren.trapp@xxxxxxxxxx> > Signed-off-by: Duane Grigsby <duane.grigsby@xxxxxxxxxx> > Signed-off-by: Anil Gurumurthy <anil.gurumurhty@xxxxxxxxxx> > Signed-off-by: Giridhar Malavali <giridhar.malavali@xxxxxxxxxx> > Signed-off-by: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx> > --- [...] > + > + 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 <jthumshirn@xxxxxxx> -- Johannes Thumshirn Storage jthumshirn@xxxxxxx +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
