[dpdk-dev] [PATCH v11 5/8] ethdev: add speed capabilities

[Chen, Jing D]

Hi,

Best Regards,
Mark


> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Friday, March 18, 2016 2:09 AM
> To: marcdevel at gmail.com; Richardson, Bruce; Doherty, Declan; Ananyev,
> Konstantin; Lu, Wenzhuo; Zhang, Helin; Chen, Jing D; harish.patil at qlogic.com;
> rahul.lakkireddy at chelsio.com; johndale at cisco.com; vido at cesnet.cz;
> adrien.mazarguil at 6wind.com; alejandro.lucero at netronome.com
> Cc: dev at dpdk.org
> Subject: [PATCH v11 5/8] ethdev: add speed capabilities
> 
> From: Marc Sune <marcdevel at gmail.com>
> 
> The speed capabilities of a device can be retrieved with
> rte_eth_dev_info_get().
> 
> The new field speed_capa is initialized in the drivers without
> taking care of device characteristics in this patch.
> When the capabilities of a driver are accurate, the table in
> overview.rst must be filled.
> 
> Signed-off-by: Marc Sune <marcdevel at gmail.com>
> ---
>  doc/guides/nics/overview.rst           |  1 +
>  doc/guides/rel_notes/release_16_04.rst |  8 ++++++++
>  drivers/net/bnx2x/bnx2x_ethdev.c       |  1 +
>  drivers/net/cxgbe/cxgbe_ethdev.c       |  1 +
>  drivers/net/e1000/em_ethdev.c          |  4 ++++
>  drivers/net/e1000/igb_ethdev.c         |  4 ++++
>  drivers/net/fm10k/fm10k_ethdev.c       |  4 ++++
>  drivers/net/i40e/i40e_ethdev.c         |  8 ++++++++
>  drivers/net/ixgbe/ixgbe_ethdev.c       |  8 ++++++++
>  drivers/net/mlx4/mlx4.c                |  2 ++
>  drivers/net/mlx5/mlx5_ethdev.c         |  3 +++
>  drivers/net/nfp/nfp_net.c              |  2 ++
>  lib/librte_ether/rte_ethdev.h          | 21 +++++++++++++++++++++
>  13 files changed, 67 insertions(+)
> 
> 
>  static void
> diff --git a/drivers/net/fm10k/fm10k_ethdev.c
> b/drivers/net/fm10k/fm10k_ethdev.c
> index edc8c11..2a1c222 100644
> --- a/drivers/net/fm10k/fm10k_ethdev.c
> +++ b/drivers/net/fm10k/fm10k_ethdev.c
> @@ -1410,6 +1410,10 @@ fm10k_dev_infos_get(struct rte_eth_dev *dev,
>  		.nb_min = FM10K_MIN_TX_DESC,
>  		.nb_align = FM10K_MULT_TX_DESC,
>  	};
> +
> +	dev_info->speed_capa = ETH_LINK_SPEED_1G |
> ETH_LINK_SPEED_2_5G |
> +			ETH_LINK_SPEED_10G | ETH_LINK_SPEED_25G |
> +			ETH_LINK_SPEED_40G;
>  }
> 

Fm10k has 100G capability, we'd better to add ETH_LINK_SPEED_100G here.