On 4/5/2019 1:04 PM, Nithin Dabilpuram wrote: > From: Nithin Dabilpuram <ndabilpu...@marvell.com> > > 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: xiao.w.w...@intel.com > > Signed-off-by: Nithin Dabilpuram <ndabilpu...@marvell.com> > Acked-by: Bernard Iremonger <bernard.iremon...@intel.com> > --- > 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?