[dpdk-dev] [PATCH 01/20] crypto/cnxk: add driver skeleton
Akhil Goyal
gakhil at marvell.com
Wed Jun 16 09:47:27 CEST 2021
> Hi Akhil,
>
> > >
> > > +Marvell cnxk
> > This should be Marvell cnxk crypto as we have net and event PMD with the
> same
> > name.
>
> [Anoob] Mempool & event already follows this convention for Marvell cnxk.
> Net driver (which is in pipeline) is also adding the same. Marvell OCTEON TX2
> all drivers followed the same convention as well. Just changing to 'Marvell
> cnxk crypto' here might make it stand out.
>
> I don't mind making the change here if you can confirm it's okay.
I think there is no convention followed here,
In case of Octeontx2, I see following in MAINTAINERS
Marvell OCTEON TX2 crypto
Marvell OCTEON TX2 regex
Marvell OCTEON TX2 -------------mempool missing here
Marvell OCTEON TX2 DMA
Marvell OCTEON TX2 EP
Marvell OCTEON TX2 ------------ event missing here
Marvell OCTEON TX2 ------------ net missing here.
I believe it is better to add crypto here when we have same PMD name for all
Subsystems. It is convenient that way.
@Thomas Monjalon Can you suggest?
Regards,
Akhil
More information about the dev
mailing list