On Thu, 8 Nov 2018 21:19:26 +0000 "Mody, Rasesh" <rasesh.m...@cavium.com> wrote:
> Use rte_log() rather than RTE_LOG() for dynamic logging. Rearrange > dynamic log types to the top and configurable log types to bottom. > Remove unused RTE_LIBRTE_BNX2X_DEBUG_TX_FREE > > Fixes: ba7eeb035a5f ("net/bnx2x: fix logging to include device name") > > Signed-off-by: Rasesh Mody <rasesh.m...@cavium.com> Looks good to me. You might also want to address the excessive number of INFO messages in bnx2x.c. There are things like debug messages at INFO level, extra new lines and banner bars. Acked-by: Stephen Hemminger <step...@networkplumber.org>