Thanks for the review. I added a comment to explain why lldp is disabled in the code, and sent the v2 patch.
Thanks, Zijie ------------------ Original ------------------ From: "Zhang, Qi Z"<qi.z.zh...@intel.com>; Date: Tue, May 22, 2018 09:00 PM To: "Zijie Pan"<zijie....@6wind.com>; "dev@dpdk.org"<dev@dpdk.org>; Cc: "Xing, Beilei"<beilei.x...@intel.com>; "Laurent Hardy"<laurent.ha...@6wind.com>; Subject: RE: [PATCH] net/i40e: add a specific API to control the LLDP agent > -----Original Message----- > From: Zijie Pan [mailto:zijie....@6wind.com] > Sent: Friday, May 18, 2018 6:14 PM > To: dev@dpdk.org > Cc: Xing, Beilei <beilei.x...@intel.com>; Zhang, Qi Z <qi.z.zh...@intel.com>; > Laurent Hardy <laurent.ha...@6wind.com> > Subject: [PATCH] net/i40e: add a specific API to control the LLDP agent > > Add a new API rte_pmd_i40e_set_lldp_cmd to control LLDP agent for i40e. > It supports the following i40e debug lldp commands: > - start/stop of the LLDP agent. > - get local/remote of the LLDP MIB (Management Information Base). > > Signed-off-by: Laurent Hardy <laurent.ha...@6wind.com> > Signed-off-by: Zijie Pan <zijie....@6wind.com> Acked-by: Qi Zhang <qi.z.zh...@intel.com>