On Fri, 24 May 2024 at 00:30, Konstantin Ananyev
<konstantin.anan...@huawei.com> wrote:
>
>
>
> > -----Original Message-----
> > From: Stephen Hemminger <step...@networkplumber.org>
> > Sent: Thursday, May 23, 2024 5:21 PM
> > To: Zhangfei Gao <zhangfei....@linaro.org>
> > Cc: Akhil Goyal <gak...@marvell.com>; Fan Zhang <fanzhang....@gmail.com>;
> > Ashish Gupta <ashish.gu...@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....@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
