[dpdk-dev] [PATCH v3 36/36] net/i40e/base: add new link speed constants

[Loktionov, Aleksandr]

Good day dear ALL

It's a new feature /*new speed display*/ but from other point of view it's a fix /*because it was found after new feature release*/

-----Original Message-----
From: Xing, Beilei <beilei.xing at intel.com> 
Sent: Friday, December 27, 2019 6:38 AM
To: Ye, Xiaolong <xiaolong.ye at intel.com>; Zhang, Qi Z <qi.z.zhang at intel.com>
Cc: dev at dpdk.org; Stillwell Jr, Paul M <paul.m.stillwell.jr at intel.com>; Loktionov, Aleksandr <aleksandr.loktionov at intel.com>
Subject: RE: [PATCH v3 36/36] net/i40e/base: add new link speed constants



> -----Original Message-----
> From: Ye, Xiaolong
> Sent: Monday, December 16, 2019 10:44 AM
> To: Xing, Beilei <beilei.xing at intel.com>; Zhang, Qi Z 
> <qi.z.zhang at intel.com>
> Cc: dev at dpdk.org; Stillwell Jr, Paul M 
> <paul.m.stillwell.jr at intel.com>; Ye, Xiaolong <xiaolong.ye at intel.com>; 
> Loktionov, Aleksandr <aleksandr.loktionov at intel.com>
> Subject: [PATCH v3 36/36] net/i40e/base: add new link speed constants
> 
> This patch fixes 'NIC Link is Up, Unknown bps' message in dmesg for 
> 2.5Gb/5Gb speeds. This problem is fixed by adding constants for 
> VIRTCHNL_LINK_SPEED_2_5GB and VIRTCHNL_LINK_SPEED_5GB.
> 

It's a fix patch, fix line is needed? The same comment is also for patch 27/36, 25/36, 21/36, 14/36.

> Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov at intel.com>
> Signed-off-by: Xiaolong Ye <xiaolong.ye at intel.com>
> ---
>  drivers/net/i40e/base/i40e_prototype.h | 4 ++++
>  drivers/net/i40e/base/virtchnl.h       | 4 ++++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/drivers/net/i40e/base/i40e_prototype.h
> b/drivers/net/i40e/base/i40e_prototype.h
> index 0f06e3262..d8ab3ea0a 100644
> --- a/drivers/net/i40e/base/i40e_prototype.h
> +++ b/drivers/net/i40e/base/i40e_prototype.h
> @@ -505,6 +505,10 @@ i40e_virtchnl_link_speed(enum i40e_aq_link_speed
> link_speed)
>  		return VIRTCHNL_LINK_SPEED_100MB;
>  	case I40E_LINK_SPEED_1GB:
>  		return VIRTCHNL_LINK_SPEED_1GB;
> +	case I40E_LINK_SPEED_2_5GB:
> +		return VIRTCHNL_LINK_SPEED_2_5GB;
> +	case I40E_LINK_SPEED_5GB:
> +		return VIRTCHNL_LINK_SPEED_5GB;
>  	case I40E_LINK_SPEED_10GB:
>  		return VIRTCHNL_LINK_SPEED_10GB;
>  	case I40E_LINK_SPEED_40GB:
> diff --git a/drivers/net/i40e/base/virtchnl.h 
> b/drivers/net/i40e/base/virtchnl.h
> index c613d4761..92515bf34 100644
> --- a/drivers/net/i40e/base/virtchnl.h
> +++ b/drivers/net/i40e/base/virtchnl.h
> @@ -53,12 +53,14 @@ enum virtchnl_status_code {  #define 
> VIRTCHNL_ERR_PARAM VIRTCHNL_STATUS_ERR_PARAM  #define 
> VIRTCHNL_STATUS_NOT_SUPPORTED VIRTCHNL_STATUS_ERR_NOT_SUPPORTED
> 
> +#define VIRTCHNL_LINK_SPEED_2_5GB_SHIFT		0x0
>  #define VIRTCHNL_LINK_SPEED_100MB_SHIFT		0x1
>  #define VIRTCHNL_LINK_SPEED_1000MB_SHIFT	0x2
>  #define VIRTCHNL_LINK_SPEED_10GB_SHIFT		0x3
>  #define VIRTCHNL_LINK_SPEED_40GB_SHIFT		0x4
>  #define VIRTCHNL_LINK_SPEED_20GB_SHIFT		0x5
>  #define VIRTCHNL_LINK_SPEED_25GB_SHIFT		0x6
> +#define VIRTCHNL_LINK_SPEED_5GB_SHIFT		0x7
> 
>  enum virtchnl_link_speed {
>  	VIRTCHNL_LINK_SPEED_UNKNOWN	= 0,
> @@ -68,6 +70,8 @@ enum virtchnl_link_speed {
>  	VIRTCHNL_LINK_SPEED_40GB	=
> BIT(VIRTCHNL_LINK_SPEED_40GB_SHIFT),
>  	VIRTCHNL_LINK_SPEED_20GB	=
> BIT(VIRTCHNL_LINK_SPEED_20GB_SHIFT),
>  	VIRTCHNL_LINK_SPEED_25GB	=
> BIT(VIRTCHNL_LINK_SPEED_25GB_SHIFT),
> +	VIRTCHNL_LINK_SPEED_2_5GB	=
> BIT(VIRTCHNL_LINK_SPEED_2_5GB_SHIFT),
> +	VIRTCHNL_LINK_SPEED_5GB		=
> BIT(VIRTCHNL_LINK_SPEED_5GB_SHIFT),
>  };
> 
>  /* for hsplit_0 field of Rx HMC context */
> --
> 2.17.1


--------------------------------------------------------------------

Intel Technology Poland sp. z o.o.
ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.

Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek
przegladanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by
others is strictly prohibited.