On Thu, 8 Nov 2018 21:19:26 +0000
"Mody, Rasesh" <[email protected]> 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 <[email protected]>


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 <[email protected]>

Reply via email to