RE: [EXTERNAL] [PATCH v2 3/3] compress/uadk: support burst enqueue/dequeue

[Akhil Goyal <gakhil@marvell.com>]

> -----Original Message-----
> From: Zhangfei Gao <zhangfei.gao@linaro.org>
> Sent: Friday, June 7, 2024 7:24 PM
> To: Akhil Goyal <gakhil@marvell.com>
> Cc: Fan Zhang <fanzhang.oss@gmail.com>; Ashish Gupta
> <ashishg@marvell.com>; dev@dpdk.org
> Subject: Re: [EXTERNAL] [PATCH v2 3/3] compress/uadk: support burst
> enqueue/dequeue
> 
> Hi, Akhil
> 
> On Fri, 7 Jun 2024 at 19:49, Akhil Goyal <gakhil@marvell.com> wrote:
> >
> > > This commit adds the burst enqueue and dequeue operations,
> > > and adds compressdev uadk info to doc
> > >
> > > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org>
> > > ---
> > >  MAINTAINERS                               |  6 ++
> > >  doc/guides/compressdevs/index.rst         |  1 +
> > >  doc/guides/compressdevs/uadk.rst          | 98 +++++++++++++++++++++++
> > >  doc/guides/rel_notes/release_24_07.rst    |  5 ++
> > >  drivers/compress/uadk/uadk_compress_pmd.c | 88 +++++++++++++++++++-
> > >  5 files changed, 196 insertions(+), 2 deletions(-)
> > >  create mode 100644 doc/guides/compressdevs/uadk.rst
> > >
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index c9adff9846..8e541d23e5 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -1213,6 +1213,12 @@ F: drivers/compress/octeontx/
> > >  F: doc/guides/compressdevs/octeontx.rst
> > >  F: doc/guides/compressdevs/features/octeontx.ini
> > >
> > > +HiSilicon UADK compress
> > > +M: Zhangfei Gao <zhangfei.gao@linaro.org>
> > > +F: drivers/compress/uadk/
> >
> > Above 3 lines should be part of patch 1 as compress/uadk is added in that.
> 
> How about
> 
> patch 1:
>  MAINTAINERS
>  doc/guides/compressdevs/index.rst
>  doc/guides/compressdevs/uadk.rst
>  doc/guides/compressdevs/features/uadk.ini
> 
> Last patch3
> doc/guides/rel_notes/release_24_07.rst

You can do that. But your implementation is completed in patch 3.
Hence details about test etc will need to be in patch 3.