On 4/5/2019 2:10 PM, Nithin Kumar D wrote: > Hi Ferruh, > > Yes, our mail server had some issue and was inserting CRLF chars. So I sent > the same v3 as v4 from gmail.
If the content is same I will continue with v3, thanks. > > 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? >> >>
