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