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 after PRIx32? Should I remove it when pulling?
> The title should be about removing superfluous empty line?

I'm quite sure answers are "yes", so I move forward.

Also please rename log functions in all bbdev PMDs as they are defined inside 
the drivers.
For instance you should rename rte_bbdev_log to acc_log in this driver.
Same for rte_bbdev_log_debug.


Reply via email to