> On Jun 19, 2017, at 1:09 AM, Johannes Thumshirn <[email protected]> wrote: > > On Fri, Jun 16, 2017 at 03:47:39PM -0700, Himanshu Madhani wrote: >> From: Duane Grigsby <[email protected]> >> >> Signed-off-by: Darren Trapp <[email protected]> >> Signed-off-by: Duane Grigsby <[email protected]> >> Signed-off-by: Anil Gurumurthy <[email protected]> >> Signed-off-by: Giridhar Malavali <[email protected]> >> Signed-off-by: Himanshu Madhani <[email protected]> >> --- > > [...] > >> + >> + 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 <[email protected]> > > -- > Johannes Thumshirn Storage > [email protected] +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
