[dpdk-dev] [PATCH v3] net/qede: fix advertising link speed capability

[Thomas Monjalon]

2016-10-31 11:35, Rasesh Mody:
> From: Harish Patil <harish.patil at qlogic.com>
> 
> Fix to advertise device's link speed capability based on NVM
> port configuration instead of returning driver supported speeds.
> 
> Fixes: 95e67b479506 ("net/qede: add 100G link speed capability")
> 
> Signed-off-by: Harish Patil <harish.patil at qlogic.com>
[...]
> +	/* Fill up the native advertised speed */
> +	switch (params.speed.advertised_speeds) {
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_10G:
> +		adv_speed = 10000;
> +	break;
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_25G:
> +		adv_speed = 25000;
> +	break;
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_40G:
> +		adv_speed = 40000;
> +	break;
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_50G:
> +		adv_speed = 50000;
> +	break;
> +	case NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_BB_100G:
> +		adv_speed = 100000;
> +	break;
> +	default:
> +		DP_NOTICE(hwfn, false, "Unknown speed\n");
> +		adv_speed = 0;
> +	}
> +	if_link->adv_speed = adv_speed;

The qede devices support only one speed?
I guess it is wrong but it is a step in right direction so it
will be enough for 16.11.

Applied