[dpdk-dev] [PATCH 11/11] doc: add documentation for OCTEON TX2 crypto PMD

[Anoob Joseph]

Hi Thomas,

Please see inline.

Thanks,
Anoob

> -----Original Message-----
> From: dev <dev-bounces at dpdk.org> On Behalf Of Thomas Monjalon
> Sent: Tuesday, September 3, 2019 11:12 PM
> To: Anoob Joseph <anoobj at marvell.com>; Jerin Jacob Kollanukkaran
> <jerinj at marvell.com>
> Cc: Akhil Goyal <akhil.goyal at nxp.com>; Pablo de Lara
> <pablo.de.lara.guarch at intel.com>; Narayana Prasad Raju Athreya
> <pathreya at marvell.com>; Ankur Dwivedi <adwivedi at marvell.com>;
> Tejasree Kondoj <ktejasree at marvell.com>; dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 11/11] doc: add documentation for OCTEON
> TX2 crypto PMD
> 
> 03/09/2019 13:18, Anoob Joseph:
> > From: Jerin Jacob Kollanukkaran <jerinj at marvell.com>
> > > > +Marvell OCTEON TX2 crypto
> > > > +M: Ankur Dwivedi <adwivedi at marvell.com>
> > > > +M: Anoob Joseph <anoobj at marvell.com>
> > > > +F: drivers/crypto/octeontx2/
> > > > +F: doc/guides/cryptodevs/octeontx2.rst
> > > > +F: doc/guides/cryptodevs/features/octeontx2.ini
> > >
> > >
> > > Move this section, next to OCTEON TX.
> >
> > [Anoob] All the entries are sorted with the "first name", which happens to
> be Marvell here. OCTEON TX2 networking driver is also added following the
> same logic.
> 
> It raises the question why OCTEON TX is still "Cavium" while OCTEON TX2 is
> "Marvell"?

[Anoob] We are awaiting legal team confirmation on whether existing "Cavium OCTEON TX" can be updated to "Marvell OCTEON TX". Will update once we get a go ahead.