> -----Original Message----- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Jerin Jacob > Sent: Tuesday, October 15, 2019 7:27 AM > To: Kiran Kumar K <kirankum...@marvell.com> > Cc: Adrien Mazarguil <adrien.mazarg...@6wind.com>; Lu, Wenzhuo > <wenzhuo...@intel.com>; Wu, Jingjing <jingjing...@intel.com>; > Iremonger, Bernard <bernard.iremon...@intel.com>; Mcnamara, John > <john.mcnam...@intel.com>; Kovacevic, Marko > <marko.kovace...@intel.com>; Thomas Monjalon <tho...@monjalon.net>; Yigit, > Ferruh <ferruh.yi...@intel.com>; Andrew Rybchenko > <arybche...@solarflare.com>; Olivier Matz <olivier.m...@6wind.com>; dpdk-dev > <dev@dpdk.org>; Ajit Khaparde > <ajit.khapa...@broadcom.com> > Subject: Re: [dpdk-dev] [PATCH] ethdev: add HIGIG2 key field to flow API > > On Tue, Oct 15, 2019 at 11:51 AM <kirankum...@marvell.com> wrote: > > > > From: Kiran Kumar K <kirankum...@marvell.com> > > > > Add new rte_flow_item_higig2_hdr in order to match higig2 header. > > It is a layer 2.5 protocol and used in broadcom switches. > > Header format is based on the following document. > > http://read.pudn.com/downloads558/doc/comm/2301468/HiGig_protocol.pdf > > > > Signed-off-by: Kiran Kumar K <kirankum...@marvell.com> > > --- > > > > V2 Changes:
If it is a v2, then I think it has to be reflected in subject, i.e: [PATCH v2] .... > > * Added support in testpmd to parse the higig2 item > > * Moved the higig2 header to new file > > * Added indentation in doc > > > > > > > include $(RTE_SDK)/mk/rte.lib.mk > > diff --git a/lib/librte_net/rte_higig.h b/lib/librte_net/rte_higig.h > > new file mode 100644 > > index 000000000..121c0a850 > > --- /dev/null > > +++ b/lib/librte_net/rte_higig.h > > @@ -0,0 +1,138 @@ > > + > > +/* SPDX-License-Identifier: BSD-3-Clause > > + * Copyright 2016 6WIND S.A. > > Copy and paste error.
