Hi Nithin,
<snip>
> > To: Nithin Kumar Dabilpuram <ndabilpu...@marvell.com>; Lu, Wenzhuo
> > <wenzhuo...@intel.com>; Wu, Jingjing <jingjing...@intel.com>;
> > Iremonger, Bernard <bernard.iremon...@intel.com>
> > Cc: dev@dpdk.org; Wang, Xiao W <xiao.w.w...@intel.com>
> > Subject: Re: [dpdk-dev] [PATCH v2 1/2] app/testpmd: fix tx vlan and
> > qinq dependency
> >
> > On 3/18/2019 9:56 AM, Nithin Kumar Dabilpuram wrote:
> > > Tx VLAN & QinQ insert enable need not depend on Rx VLAN offload
> > > ETH_VLAN_EXTEND_OFFLOAD.
> > >
> > > 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>
> >
> > <...>
> >
> > > @@ -2963,11 +2962,6 @@ tx_vlan_set(portid_t port_id, uint16_t vlan_id)
> > > if (vlan_id_is_invalid(vlan_id))
> > > return;
> > >
> > > - vlan_offload = rte_eth_dev_get_vlan_offload(port_id);
> > > - if (vlan_offload & ETH_VLAN_EXTEND_OFFLOAD) {
> > > - printf("Error, as QinQ has been enabled.\n");
> > > - return;
> > > - }
> >
> > It looks like you didn't take account comment on this in previous
> > version, can you please check it?
>
> ./dpdk/devtools/check-git-log.sh -1
> Wrong headline lowercase:
> app/testpmd: fix tx vlan and qinq dependency
>
> otherwise
> acked-by: Bernard Iremonger <bernard.iremon...@intel.com>
Please drop my ack until the comment on the v1 patch is addressed.
