Hi Wenzhuo, > -----Original Message----- > From: Lu, Wenzhuo > Sent: Wednesday, March 29, 2017 2:53 AM > To: Iremonger, Bernard <bernard.iremon...@intel.com>; dev@dpdk.org; > Xing, Beilei <beilei.x...@intel.com>; Wu, Jingjing <jingjing...@intel.com> > Cc: Zhang, Helin <helin.zh...@intel.com> > Subject: RE: [PATCH v3 4/5] net/i40e: initialise L3 MAP register > > Hi Bernard, > > > -----Original Message----- > > From: Iremonger, Bernard > > Sent: Wednesday, March 29, 2017 12:21 AM > > To: dev@dpdk.org; Xing, Beilei; Wu, Jingjing > > Cc: Zhang, Helin; Lu, Wenzhuo; Iremonger, Bernard > > Subject: [PATCH v3 4/5] net/i40e: initialise L3 MAP register > > > > The L3 MAP register is initialised to support QinQ cloud filters. > > > > Signed-off-by: Bernard Iremonger <bernard.iremon...@intel.com> > > --- > > drivers/net/i40e/i40e_ethdev.c | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > Is it better to adjust the sequence? Put it before the patch 1 or even merge > it > into patch 1?
I think it should be a separate patch. It was added later that is why it is the fourth patch, the first three patches compile independently. I can put it as the first patch if you like, however the sequence is ok as it is. Regards, Bernard.