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
