On Thu, May 23, 2024 at 04:30:30PM +0000, Konstantin Ananyev 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.

+1

>  

Reply via email to