Re: [PATCH 1/3] compress/uadk: Introduce UADK compression driver

[Zhangfei Gao <zhangfei.gao@linaro.org>]

On Fri, 24 May 2024 at 00:30, Konstantin Ananyev
<konstantin.ananyev@huawei.com> wrote:
>
>
>
> > -----Original Message-----
> > From: Stephen Hemminger <stephen@networkplumber.org>
> > Sent: Thursday, May 23, 2024 5:21 PM
> > To: Zhangfei Gao <zhangfei.gao@linaro.org>
> > Cc: Akhil Goyal <gakhil@marvell.com>; Fan Zhang <fanzhang.oss@gmail.com>; Ashish Gupta <ashish.gupta@marvell.com>;
> > dev@dpdk.org
> > Subject: Re: [PATCH 1/3] compress/uadk: Introduce UADK compression driver
> >
> > On Mon, 22 Apr 2024 14:31:00 +0000
> > Zhangfei Gao <zhangfei.gao@linaro.org> wrote:
> >
> > > +static struct rte_compressdev_ops uadk_compress_pmd_ops = {
> > > +           .dev_configure          = NULL,
> > > +           .dev_start              = NULL,
> > > +           .dev_stop               = NULL,
> > > +           .dev_close              = NULL,
> > > +           .stats_get              = NULL,
> > > +           .stats_reset            = NULL,
> > > +           .dev_infos_get          = NULL,
> > > +           .queue_pair_setup       = NULL,
> > > +           .queue_pair_release     = NULL,
> > > +           .private_xform_create   = NULL,
> > > +           .private_xform_free     = NULL,
> > > +           .stream_create          = NULL,
> > > +           .stream_free            = NULL,
> > > +};
> >
> > If you just used { } then all the elements are initialized to NULL anyway
>
> Or even
> static struct rte_compressdev_ops uadk_compress_pmd_ops;
> should do the same.

Yes, good idea
Thanks Konstantin