Re: [PATCH] common/cnxk: update NIX and NPA dump function

[Jerin Jacob <jerinjacobk@gmail.com>]

On Mon, Dec 20, 2021 at 6:56 PM Rahul Bhansali <rbhansali@marvell.com> wrote:
>
> Updates nix_dump and npa_dump to use plt_dump function.
>
> Signed-off-by: Rahul Bhansali <rbhansali@marvell.com>

Acked-by: Jerin Jacob <jerinj@marvell.com>
Applied to dpdk-next-net-mrvl/for-next-net. Thanks
> ---
>  drivers/common/cnxk/roc_nix_debug.c | 4 ++--
>  drivers/common/cnxk/roc_npa_debug.c | 2 +-
>  drivers/common/cnxk/roc_platform.h  | 1 +
>  3 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/common/cnxk/roc_nix_debug.c b/drivers/common/cnxk/roc_nix_debug.c
> index 266935a6c5..7466f782c3 100644
> --- a/drivers/common/cnxk/roc_nix_debug.c
> +++ b/drivers/common/cnxk/roc_nix_debug.c
> @@ -5,14 +5,14 @@
>  #include "roc_api.h"
>  #include "roc_priv.h"
>
> -#define nix_dump(fmt, ...) fprintf(stderr, fmt "\n", ##__VA_ARGS__)
> +#define nix_dump plt_dump
>  #define NIX_REG_INFO(reg)                                                      \
>         {                                                                      \
>                 reg, #reg                                                      \
>         }
>  #define NIX_REG_NAME_SZ 48
>
> -#define nix_dump_no_nl(fmt, ...) fprintf(stderr, fmt, ##__VA_ARGS__)
> +#define nix_dump_no_nl plt_dump_no_nl
>
>  struct nix_lf_reg_info {
>         uint32_t offset;
> diff --git a/drivers/common/cnxk/roc_npa_debug.c b/drivers/common/cnxk/roc_npa_debug.c
> index 421c2af8a5..798ace291c 100644
> --- a/drivers/common/cnxk/roc_npa_debug.c
> +++ b/drivers/common/cnxk/roc_npa_debug.c
> @@ -5,7 +5,7 @@
>  #include "roc_api.h"
>  #include "roc_priv.h"
>
> -#define npa_dump(fmt, ...) fprintf(stderr, fmt "\n", ##__VA_ARGS__)
> +#define npa_dump plt_dump
>
>  static inline void
>  npa_pool_dump(__io struct npa_pool_s *pool)
> diff --git a/drivers/common/cnxk/roc_platform.h b/drivers/common/cnxk/roc_platform.h
> index 61d4781209..13b23cec03 100644
> --- a/drivers/common/cnxk/roc_platform.h
> +++ b/drivers/common/cnxk/roc_platform.h
> @@ -204,6 +204,7 @@ extern int cnxk_logtype_tm;
>  #define plt_warn(fmt, args...) RTE_LOG(WARNING, PMD, fmt "\n", ##args)
>  #define plt_print(fmt, args...) RTE_LOG(INFO, PMD, fmt "\n", ##args)
>  #define plt_dump(fmt, ...)      fprintf(stderr, fmt "\n", ##__VA_ARGS__)
> +#define plt_dump_no_nl(fmt, ...) fprintf(stderr, fmt, ##__VA_ARGS__)
>
>  /**
>   * Log debug message if given subsystem logging is enabled.
> --
> 2.25.1
>