[dpdk-dev] [PATCH] ethdev: add HIGIG2 key field to flow API

[Ananyev, Konstantin]

> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jerin Jacob
> Sent: Tuesday, October 15, 2019 7:27 AM
> To: Kiran Kumar K <kirankumark at marvell.com>
> Cc: Adrien Mazarguil <adrien.mazarguil at 6wind.com>; Lu, Wenzhuo <wenzhuo.lu at intel.com>; Wu, Jingjing <jingjing.wu at intel.com>;
> Iremonger, Bernard <bernard.iremonger at intel.com>; Mcnamara, John <john.mcnamara at intel.com>; Kovacevic, Marko
> <marko.kovacevic at intel.com>; Thomas Monjalon <thomas at monjalon.net>; Yigit, Ferruh <ferruh.yigit at intel.com>; Andrew Rybchenko
> <arybchenko at solarflare.com>; Olivier Matz <olivier.matz at 6wind.com>; dpdk-dev <dev at dpdk.org>; Ajit Khaparde
> <ajit.khaparde at broadcom.com>
> Subject: Re: [dpdk-dev] [PATCH] ethdev: add HIGIG2 key field to flow API
> 
> On Tue, Oct 15, 2019 at 11:51 AM <kirankumark at marvell.com> wrote:
> >
> > From: Kiran Kumar K <kirankumark at 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 <kirankumark at 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.