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 <ferruh.yi...@intel.com> wrote:
> 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?
>
>
