09/11/2021 05:39, Gagandeep Singh: > The DPAA DMA driver is an implementation of the dmadev APIs, > that provide means to initiate a DMA transaction from CPU. > The initiated DMA is performed without CPU being involved > in the actual DMA transaction. This is achieved via using > the QDMA controller of DPAA SoC. > > Signed-off-by: Gagandeep Singh <g.si...@nxp.com> > --- > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1377,6 +1377,17 @@ F: drivers/raw/dpaa2_qdma/ > F: doc/guides/rawdevs/dpaa2_qdma.rst > > > + > +Dmadev Drivers > +-------------- > + > +NXP DPAA DMA > +M: Gagandeep Singh <g.si...@nxp.com> > +M: Nipun Gupta <nipun.gu...@nxp.com> > +F: drivers/dma/dpaa/ > +F: doc/guides/dmadevs/dpaa.rst > + > +
There is already a section for DMA drivers. > Packet processing > ----------------- > [...] > --- /dev/null > +++ b/doc/guides/dmadevs/dpaa.rst > @@ -0,0 +1,54 @@ > +.. SPDX-License-Identifier: BSD-3-Clause > + Copyright 2021 NXP > + > +NXP DPAA DMA Driver > +===================== There are other occurences in this patch of a lack of attention. Doing underlining of correct length is a minimum when starting a doc. > --- a/doc/guides/rel_notes/release_21_11.rst > +++ b/doc/guides/rel_notes/release_21_11.rst > @@ -20,6 +20,9 @@ DPDK Release 21.11 > ninja -C build doc > xdg-open build/doc/guides/html/rel_notes/release_21_11.html > > +* **Added NXP DPAA DMA driver.** > + > + * Added a new dmadev driver for NXP DPAA platform. > > New Features > ------------ The new features should be inside this section and well sorted. There are a lot of other details that I won't report. All of this look to be on purpose, or a big negligence. In any case, it looks to me as sending some junk work to my face. I am too kind, so I will fix all, and will merge after the deadline passed. But please remember that I don't forget such attitude, and I think other maintainers won't forget as well.
