Re: [PATCH v2 5/5] baseband/acc: cosmetic log changes

2024-06-14 Thread Thomas Monjalon
14/06/2024 23:04, Thomas Monjalon: > 02/05/2024 22:45, Hernan Vargas: > > - rte_bbdev_log(WARNING, "DMA Response %#"PRIx32"\n", > > desc->req.word0); > > + rte_bbdev_log(WARNING, "DMA Response %#"PRIx32"", > > desc->req.word0); > > Why keeping an empty string

Re: [PATCH v2 5/5] baseband/acc: cosmetic log changes

2024-06-14 Thread Thomas Monjalon
02/05/2024 22:45, Hernan Vargas: > - rte_bbdev_log(WARNING, "DMA Response %#"PRIx32"\n", > desc->req.word0); > + rte_bbdev_log(WARNING, "DMA Response %#"PRIx32"", > desc->req.word0); Why keeping an empty string after PRIx32? Should I remove it when pulling

[PATCH v2 5/5] baseband/acc: cosmetic log changes

2024-05-02 Thread Hernan Vargas
Minor cosmetic log change. No functional impact. Signed-off-by: Hernan Vargas Reviewed-by: Maxime Coquelin --- drivers/baseband/acc/rte_acc100_pmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/baseband/acc/rte_acc100_pmd.c b/drivers/baseband/acc/rte_acc100_pmd.c