[dpdk-dev] [PATCH v2 05/40] bnxt: add dev configure operation

Bruce Richardson bruce.richardson at intel.com
Wed May 25 17:25:55 CEST 2016


On Fri, May 13, 2016 at 03:45:54PM -0700, Stephen Hurd wrote:
> This adds the bnxt_hwrm_port_phy_cfg() HWRM call, and copies required
> information into the new struct bnxt_link_info.
> 
> Signed-off-by: Stephen Hurd <stephen.hurd at broadcom.com>
> Reviewed-by: Ajit Kumar Khaparde <ajit.khaparde at broadcom.com>
> ---
>  drivers/net/bnxt/bnxt.h                |  32 +++
>  drivers/net/bnxt/bnxt_ethdev.c         |  24 ++
>  drivers/net/bnxt/bnxt_hwrm.c           | 232 +++++++++++++++-
>  drivers/net/bnxt/bnxt_hwrm.h           |   1 +
>  drivers/net/bnxt/hsi_struct_def_dpdk.h | 470 +++++++++++++++++++++++++++++++++
>  5 files changed, 758 insertions(+), 1 deletion(-)
> 

Again, a minor issue: checkpatch flags a couple of instances of "unsigned" instead of 
"unsigned int" in this patch. Maybe just scan the rest of the set with checkpatch
to pick up on other minor niggles like this.

/Bruce


More information about the dev mailing list