On 1/11/17, 11:51 AM, "Bart Van Assche" <bart.vanassche@xxxxxxxxxxx> wrote: >On 12/23/2016 08:23 PM, Himanshu Madhani wrote: >> -#define QLA_TGT_MODE_ENABLED() (ql2x_ini_mode != QLA2XXX_INI_MODE_ENABLED) >> +#define QLA_TGT_MODE_ENABLED() \ >> + ((ql2x_ini_mode != QLA2XXX_INI_MODE_ENABLED) || \ >> + (ql2x_ini_mode == QLA2XXX_INI_MODE_DUAL)) > >Hello Himanshu and Quinn, > >Is this change needed? It looks redundant to me. For all possible >ql2x_ini_mode values both the old and the new expression will yield the >same value. Agree. This is redundant. I’ll remove it. > >Anyway, dual mode is a welcome improvement! > >Bart. ��.n��������+%������w��{.n�����{������ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f