> 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