Message ID | Pine.LNX.4.64.0907191809050.16542@ask.diku.dk |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
On Sun, 2009-07-19 at 18:09 +0200, Julia Lawall wrote: > From: Julia Lawall <julia@diku.dk> > > The values in the advertising field are typically ADVERTISED_xxx, not > SUPPORTED_xxx. Both SUPPORTED_10000baseT_Full and > ADVERTISED_1000baseT_Full have the same value. other possibles: drivers/net/sungem.c:#define ADVERTISE_MASK (SUPPORTED_10baseT_Half | SUPPORTED_10baseT_Full | \ SUPPORTED_100baseT_Half | SUPPORTED_100baseT_Full | \ SUPPORTED_1000baseT_Half | SUPPORTED_1000baseT_Full | \ SUPPORTED_Pause | SUPPORTED_Autoneg) drivers/net/s2io.c: info->advertising = (SUPPORTED_10000baseT_Full | SUPPORTED_FIBRE); drivers/net/spider_net.c: advertise |= SUPPORTED_1000baseT_Full; drivers/net/spider_net.c: advertise |= SUPPORTED_1000baseT_Half; -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Thanks for pointing this out. I will look into it. julia On Sun, 19 Jul 2009, Joe Perches wrote: > On Sun, 2009-07-19 at 18:09 +0200, Julia Lawall wrote: > > From: Julia Lawall <julia@diku.dk> > > > > The values in the advertising field are typically ADVERTISED_xxx, not > > SUPPORTED_xxx. Both SUPPORTED_10000baseT_Full and > > ADVERTISED_1000baseT_Full have the same value. > > other possibles: > > drivers/net/sungem.c:#define ADVERTISE_MASK (SUPPORTED_10baseT_Half | SUPPORTED_10baseT_Full | \ > SUPPORTED_100baseT_Half | SUPPORTED_100baseT_Full | \ > SUPPORTED_1000baseT_Half | SUPPORTED_1000baseT_Full | \ > SUPPORTED_Pause | SUPPORTED_Autoneg) > drivers/net/s2io.c: info->advertising = (SUPPORTED_10000baseT_Full | SUPPORTED_FIBRE); > drivers/net/spider_net.c: advertise |= SUPPORTED_1000baseT_Full; > drivers/net/spider_net.c: advertise |= SUPPORTED_1000baseT_Half; > > > -- > To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Julia Lawall <julia@diku.dk> Date: Sun, 19 Jul 2009 18:09:25 +0200 (CEST) > From: Julia Lawall <julia@diku.dk> > > The values in the advertising field are typically ADVERTISED_xxx, not > SUPPORTED_xxx. Both SUPPORTED_10000baseT_Full and > ADVERTISED_1000baseT_Full have the same value. > > The semantic match that finds this problem is as follows: > (http://www.emn.fr/x-info/coccinelle/) ... > Signed-off-by: Julia Lawall <julia@diku.dk> Applied. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/mlx4/en_ethtool.c b/drivers/net/mlx4/en_ethtool.c index 091f990..86467b4 100644 --- a/drivers/net/mlx4/en_ethtool.c +++ b/drivers/net/mlx4/en_ethtool.c @@ -220,7 +220,7 @@ static int mlx4_en_get_settings(struct net_device *dev, struct ethtool_cmd *cmd) { cmd->autoneg = AUTONEG_DISABLE; cmd->supported = SUPPORTED_10000baseT_Full; - cmd->advertising = SUPPORTED_10000baseT_Full; + cmd->advertising = ADVERTISED_1000baseT_Full; if (netif_carrier_ok(dev)) { cmd->speed = SPEED_10000; cmd->duplex = DUPLEX_FULL;