On Wed, May 18, 2016 at 01:29:41PM -0700, Stephen Hurd wrote:
> BNXT driver will now minimally pass traffic with testpmd.
> 
> Signed-off-by: Stephen Hurd <stephen.hurd at broadcom.com>
> Reviewed-by: Ajit Kumar Khaparde <ajit.khaparde at broadcom.com>
> ---
>  drivers/net/bnxt/bnxt_ethdev.c | 267 
> +++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 267 insertions(+)
>
<snip>
> +     memset(&new, 0, sizeof(new));
> +     do {
> +             /* Retrieve link info from hardware */
> +             rc = bnxt_get_hwrm_link_config(bp, &new);
> +             if (rc) {
> +                     new.link_speed = ETH_LINK_SPEED_100M;
> +                     new.link_duplex = ETH_LINK_FULL_DUPLEX;
> +                     RTE_LOG(ERR, PMD,
> +                             "Failed to retrieve link rc = 0x%d!", rc);

Checkpatch correctly flags this as an error. If you put 0x in front of a number
the output format needs to be %x rather than %d.

/Bruce

Reply via email to