Hi Nicolas, Can you add fixes lines for the commits which introduced these issues so that these can be backported as well? Also cc stable
Regards, Akhil > -----Original Message----- > From: Nicolas Chautru <nicolas.chau...@intel.com> > Sent: Monday, April 20, 2020 2:17 AM > To: dev@dpdk.org; Akhil Goyal <akhil.go...@nxp.com> > Cc: Nicolas Chautru <nicolas.chau...@intel.com> > Subject: [PATCH v1] bbdev: doxygen update only > > Several doxygen markup were incorrect in header files. > > Signed-off-by: Nicolas Chautru <nicolas.chau...@intel.com> > --- > lib/librte_bbdev/rte_bbdev.h | 16 ++++++++-------- > lib/librte_bbdev/rte_bbdev_op.h | 16 ++++++++-------- > lib/librte_bbdev/rte_bbdev_pmd.h | 14 +++++++------- > 3 files changed, 23 insertions(+), 23 deletions(-) > > diff --git a/lib/librte_bbdev/rte_bbdev.h b/lib/librte_bbdev/rte_bbdev.h > index 38d9d50..ecd95a8 100644 > --- a/lib/librte_bbdev/rte_bbdev.h > +++ b/lib/librte_bbdev/rte_bbdev.h > @@ -442,21 +442,21 @@ struct rte_bbdev_data { > * these fields, but should only write to the *_ops fields. > */ > struct __rte_cache_aligned rte_bbdev { > - /**< Enqueue encode function */ > + /** Enqueue encode function */ > rte_bbdev_enqueue_enc_ops_t enqueue_enc_ops; > - /**< Enqueue decode function */ > + /** Enqueue decode function */ > rte_bbdev_enqueue_dec_ops_t enqueue_dec_ops; > - /**< Dequeue encode function */ > + /** Dequeue encode function */ > rte_bbdev_dequeue_enc_ops_t dequeue_enc_ops; > - /**< Dequeue decode function */ > + /** Dequeue decode function */ > rte_bbdev_dequeue_dec_ops_t dequeue_dec_ops; > - /**< Enqueue encode function */ > + /** Enqueue encode function */ > rte_bbdev_enqueue_enc_ops_t enqueue_ldpc_enc_ops; > - /**< Enqueue decode function */ > + /** Enqueue decode function */ > rte_bbdev_enqueue_dec_ops_t enqueue_ldpc_dec_ops; > - /**< Dequeue encode function */ > + /** Dequeue encode function */ > rte_bbdev_dequeue_enc_ops_t dequeue_ldpc_enc_ops; > - /**< Dequeue decode function */ > + /** Dequeue decode function */ > rte_bbdev_dequeue_dec_ops_t dequeue_ldpc_dec_ops; > const struct rte_bbdev_ops *dev_ops; /**< Functions exported by PMD > */ > struct rte_bbdev_data *data; /**< Pointer to device data */ > diff --git a/lib/librte_bbdev/rte_bbdev_op.h b/lib/librte_bbdev/rte_bbdev_op.h > index 9c66721..f726d73 100644 > --- a/lib/librte_bbdev/rte_bbdev_op.h > +++ b/lib/librte_bbdev/rte_bbdev_op.h > @@ -398,12 +398,12 @@ struct rte_bbdev_op_turbo_dec { > */ > uint8_t num_maps; > > - /**< [0 - TB : 1 - CB] */ > + /** [0 - TB : 1 - CB] */ > uint8_t code_block_mode; > union { > - /**< Struct which stores Code Block specific parameters */ > + /** Struct which stores Code Block specific parameters */ > struct rte_bbdev_op_dec_turbo_cb_params cb_params; > - /**< Struct which stores Transport Block specific parameters */ > + /** Struct which stores Transport Block specific parameters */ > struct rte_bbdev_op_dec_turbo_tb_params tb_params; > }; > }; > @@ -554,7 +554,7 @@ struct rte_bbdev_op_enc_turbo_tb_params { > * the Turbo operation when r >= C-, [K:3*Kpi] > */ > uint16_t ncb_pos; > - /**< The index of the first CB in the inbound mbuf data, default is 0 */ > + /** The index of the first CB in the inbound mbuf data, default is 0 */ > uint8_t r; > }; > > @@ -751,11 +751,11 @@ enum { > > /** Structure specifying a single encode operation */ > struct rte_bbdev_enc_op { > - /**< Status of operation that was performed */ > + /** Status of operation that was performed */ > int status; > - /**< Mempool which op instance is in */ > + /** Mempool which op instance is in */ > struct rte_mempool *mempool; > - /**< Opaque pointer for user data */ > + /** Opaque pointer for user data */ > void *opaque_data; > union { > /** Contains turbo decoder specific parameters */ > @@ -792,7 +792,7 @@ struct rte_bbdev_op_cap { > } cap; /**< Operation-type specific capabilities */ > }; > > -/**< @internal Private data structure stored with operation pool. */ > +/** @internal Private data structure stored with operation pool. */ > struct rte_bbdev_op_pool_private { > enum rte_bbdev_op_type type; /**< Type of operations in a pool */ > }; > diff --git a/lib/librte_bbdev/rte_bbdev_pmd.h > b/lib/librte_bbdev/rte_bbdev_pmd.h > index 24ddcee..237e336 100644 > --- a/lib/librte_bbdev/rte_bbdev_pmd.h > +++ b/lib/librte_bbdev/rte_bbdev_pmd.h > @@ -146,18 +146,18 @@ typedef int (*rte_bbdev_queue_intr_disable_t)(struct > rte_bbdev *dev, > * fields are for non-vital operations > */ > struct rte_bbdev_ops { > - /**< Allocate and configure device memory. Optional. */ > + /** Allocate and configure device memory. Optional. */ > rte_bbdev_setup_queues_t setup_queues; > - /**< Configure interrupts. Optional. */ > + /** Configure interrupts. Optional. */ > rte_bbdev_intr_enable_t intr_enable; > - /**< Start device. Optional. */ > + /** Start device. Optional. */ > rte_bbdev_start_t start; > - /**< Stop device. Optional. */ > + /** Stop device. Optional. */ > rte_bbdev_stop_t stop; > - /**< Close device. Optional. */ > + /** Close device. Optional. */ > rte_bbdev_close_t close; > > - /**< Get device info. Required. */ > + /** Get device info. Required. */ > rte_bbdev_info_get_t info_get; > /** Get device statistics. Optional. */ > rte_bbdev_stats_get_t stats_get; > @@ -170,7 +170,7 @@ struct rte_bbdev_ops { > rte_bbdev_queue_release_t queue_release; > /** Start a queue. Optional. */ > rte_bbdev_queue_start_t queue_start; > - /**< Stop a queue pair. Optional. */ > + /** Stop a queue pair. Optional. */ > rte_bbdev_queue_stop_t queue_stop; > > /** Enable queue interrupt. Optional */ > -- > 1.8.3.1