Hi Ferruh, Yes, our mail server had some issue and was inserting CRLF chars. So I sent the same v3 as v4 from gmail.
Thanks Nithin On Fri, Apr 5, 2019 at 5:36 PM Ferruh Yigit <[email protected]> wrote: > On 4/5/2019 1:04 PM, Nithin Dabilpuram wrote: > > From: Nithin Dabilpuram <[email protected]> > > > > Tx VLAN & QinQ insert enable need not depend on > > Rx VLAN offload ETH_VLAN_EXTEND_OFFLOAD. For Tx VLAN > > insert enable, error check is now to see if QinQ was enabled > > but only single VLAN id is set. > > > > Fixes: 6a34f91690d0 ("app/testpmd: fix error message when setting Tx > VLAN") > > Cc: [email protected] > > > > Signed-off-by: Nithin Dabilpuram <[email protected]> > > Acked-by: Bernard Iremonger <[email protected]> > > --- > > v4: > > * Resend v3 from different mailserver to avoid > > CRLF > > v3: > > * Add back error check in tx_vlan_set() to check if QinQ is > > already enabled. Also fix headline. > > v2: > > * Split change into two seperate patches as suggested. > > > > Hi Nithin, > > I just merged the v3 and about to send the mail :) > > What is different in v4? avoid CRLF? > >
