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

Reply via email to