tg3: Use *_UNKNOWN ethtool definitions
This patch replaces tg3's private SPEED_INVALID and DUPLEX_INVALID definitions with SPEED_UNKNOWN and DUPLEX_UNKNOWN respectively. Signed-off-by: Matt Carlson <mcarlson@broadcom.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
c6700ce224
commit
e740522e6d
2 changed files with 20 additions and 23 deletions
|
@ -1819,8 +1819,8 @@ static void tg3_adjust_link(struct net_device *dev)
|
||||||
(6 << TX_LENGTHS_IPG_SHIFT) |
|
(6 << TX_LENGTHS_IPG_SHIFT) |
|
||||||
(32 << TX_LENGTHS_SLOT_TIME_SHIFT)));
|
(32 << TX_LENGTHS_SLOT_TIME_SHIFT)));
|
||||||
|
|
||||||
if ((phydev->link && tp->link_config.active_speed == SPEED_INVALID) ||
|
if ((phydev->link && tp->link_config.active_speed == SPEED_UNKNOWN) ||
|
||||||
(!phydev->link && tp->link_config.active_speed != SPEED_INVALID) ||
|
(!phydev->link && tp->link_config.active_speed != SPEED_UNKNOWN) ||
|
||||||
phydev->speed != tp->link_config.active_speed ||
|
phydev->speed != tp->link_config.active_speed ||
|
||||||
phydev->duplex != tp->link_config.active_duplex ||
|
phydev->duplex != tp->link_config.active_duplex ||
|
||||||
oldflowctrl != tp->link_config.active_flowctrl)
|
oldflowctrl != tp->link_config.active_flowctrl)
|
||||||
|
@ -3813,8 +3813,8 @@ static void tg3_aux_stat_to_speed_duplex(struct tg3 *tp, u32 val, u16 *speed, u8
|
||||||
DUPLEX_HALF;
|
DUPLEX_HALF;
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
*speed = SPEED_INVALID;
|
*speed = SPEED_UNKNOWN;
|
||||||
*duplex = DUPLEX_INVALID;
|
*duplex = DUPLEX_UNKNOWN;
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -3906,7 +3906,7 @@ static void tg3_phy_copper_begin(struct tg3 *tp)
|
||||||
|
|
||||||
tg3_phy_autoneg_cfg(tp, new_adv,
|
tg3_phy_autoneg_cfg(tp, new_adv,
|
||||||
FLOW_CTRL_TX | FLOW_CTRL_RX);
|
FLOW_CTRL_TX | FLOW_CTRL_RX);
|
||||||
} else if (tp->link_config.speed == SPEED_INVALID) {
|
} else if (tp->link_config.speed == SPEED_UNKNOWN) {
|
||||||
if (tp->phy_flags & TG3_PHYFLG_10_100_ONLY)
|
if (tp->phy_flags & TG3_PHYFLG_10_100_ONLY)
|
||||||
tp->link_config.advertising &=
|
tp->link_config.advertising &=
|
||||||
~(ADVERTISED_1000baseT_Half |
|
~(ADVERTISED_1000baseT_Half |
|
||||||
|
@ -3938,7 +3938,7 @@ static void tg3_phy_copper_begin(struct tg3 *tp)
|
||||||
}
|
}
|
||||||
|
|
||||||
if (tp->link_config.autoneg == AUTONEG_DISABLE &&
|
if (tp->link_config.autoneg == AUTONEG_DISABLE &&
|
||||||
tp->link_config.speed != SPEED_INVALID) {
|
tp->link_config.speed != SPEED_UNKNOWN) {
|
||||||
u32 bmcr, orig_bmcr;
|
u32 bmcr, orig_bmcr;
|
||||||
|
|
||||||
tp->link_config.active_speed = tp->link_config.speed;
|
tp->link_config.active_speed = tp->link_config.speed;
|
||||||
|
@ -4172,8 +4172,8 @@ static int tg3_setup_copper_phy(struct tg3 *tp, int force_reset)
|
||||||
}
|
}
|
||||||
|
|
||||||
current_link_up = 0;
|
current_link_up = 0;
|
||||||
current_speed = SPEED_INVALID;
|
current_speed = SPEED_UNKNOWN;
|
||||||
current_duplex = DUPLEX_INVALID;
|
current_duplex = DUPLEX_UNKNOWN;
|
||||||
tp->phy_flags &= ~TG3_PHYFLG_MDIX_STATE;
|
tp->phy_flags &= ~TG3_PHYFLG_MDIX_STATE;
|
||||||
tp->link_config.rmt_adv = 0;
|
tp->link_config.rmt_adv = 0;
|
||||||
|
|
||||||
|
@ -5069,8 +5069,8 @@ static int tg3_setup_fiber_phy(struct tg3 *tp, int force_reset)
|
||||||
LED_CTRL_LNKLED_OVERRIDE |
|
LED_CTRL_LNKLED_OVERRIDE |
|
||||||
LED_CTRL_1000MBPS_ON));
|
LED_CTRL_1000MBPS_ON));
|
||||||
} else {
|
} else {
|
||||||
tp->link_config.active_speed = SPEED_INVALID;
|
tp->link_config.active_speed = SPEED_UNKNOWN;
|
||||||
tp->link_config.active_duplex = DUPLEX_INVALID;
|
tp->link_config.active_duplex = DUPLEX_UNKNOWN;
|
||||||
tw32(MAC_LED_CTRL, (tp->led_ctrl |
|
tw32(MAC_LED_CTRL, (tp->led_ctrl |
|
||||||
LED_CTRL_LNKLED_OVERRIDE |
|
LED_CTRL_LNKLED_OVERRIDE |
|
||||||
LED_CTRL_TRAFFIC_OVERRIDE));
|
LED_CTRL_TRAFFIC_OVERRIDE));
|
||||||
|
@ -5118,8 +5118,8 @@ static int tg3_setup_fiber_mii_phy(struct tg3 *tp, int force_reset)
|
||||||
tg3_phy_reset(tp);
|
tg3_phy_reset(tp);
|
||||||
|
|
||||||
current_link_up = 0;
|
current_link_up = 0;
|
||||||
current_speed = SPEED_INVALID;
|
current_speed = SPEED_UNKNOWN;
|
||||||
current_duplex = DUPLEX_INVALID;
|
current_duplex = DUPLEX_UNKNOWN;
|
||||||
tp->link_config.rmt_adv = 0;
|
tp->link_config.rmt_adv = 0;
|
||||||
|
|
||||||
err |= tg3_readphy(tp, MII_BMSR, &bmsr);
|
err |= tg3_readphy(tp, MII_BMSR, &bmsr);
|
||||||
|
@ -10515,8 +10515,8 @@ static int tg3_get_settings(struct net_device *dev, struct ethtool_cmd *cmd)
|
||||||
cmd->eth_tp_mdix = ETH_TP_MDI;
|
cmd->eth_tp_mdix = ETH_TP_MDI;
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
ethtool_cmd_speed_set(cmd, SPEED_INVALID);
|
ethtool_cmd_speed_set(cmd, SPEED_UNKNOWN);
|
||||||
cmd->duplex = DUPLEX_INVALID;
|
cmd->duplex = DUPLEX_UNKNOWN;
|
||||||
cmd->eth_tp_mdix = ETH_TP_MDI_INVALID;
|
cmd->eth_tp_mdix = ETH_TP_MDI_INVALID;
|
||||||
}
|
}
|
||||||
cmd->phy_address = tp->phy_addr;
|
cmd->phy_address = tp->phy_addr;
|
||||||
|
@ -10598,8 +10598,8 @@ static int tg3_set_settings(struct net_device *dev, struct ethtool_cmd *cmd)
|
||||||
if (cmd->autoneg == AUTONEG_ENABLE) {
|
if (cmd->autoneg == AUTONEG_ENABLE) {
|
||||||
tp->link_config.advertising = (cmd->advertising |
|
tp->link_config.advertising = (cmd->advertising |
|
||||||
ADVERTISED_Autoneg);
|
ADVERTISED_Autoneg);
|
||||||
tp->link_config.speed = SPEED_INVALID;
|
tp->link_config.speed = SPEED_UNKNOWN;
|
||||||
tp->link_config.duplex = DUPLEX_INVALID;
|
tp->link_config.duplex = DUPLEX_UNKNOWN;
|
||||||
} else {
|
} else {
|
||||||
tp->link_config.advertising = 0;
|
tp->link_config.advertising = 0;
|
||||||
tp->link_config.speed = speed;
|
tp->link_config.speed = speed;
|
||||||
|
@ -13347,11 +13347,11 @@ static void __devinit tg3_phy_init_link_config(struct tg3 *tp)
|
||||||
adv |= ADVERTISED_FIBRE;
|
adv |= ADVERTISED_FIBRE;
|
||||||
|
|
||||||
tp->link_config.advertising = adv;
|
tp->link_config.advertising = adv;
|
||||||
tp->link_config.speed = SPEED_INVALID;
|
tp->link_config.speed = SPEED_UNKNOWN;
|
||||||
tp->link_config.duplex = DUPLEX_INVALID;
|
tp->link_config.duplex = DUPLEX_UNKNOWN;
|
||||||
tp->link_config.autoneg = AUTONEG_ENABLE;
|
tp->link_config.autoneg = AUTONEG_ENABLE;
|
||||||
tp->link_config.active_speed = SPEED_INVALID;
|
tp->link_config.active_speed = SPEED_UNKNOWN;
|
||||||
tp->link_config.active_duplex = DUPLEX_INVALID;
|
tp->link_config.active_duplex = DUPLEX_UNKNOWN;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int __devinit tg3_phy_probe(struct tg3 *tp)
|
static int __devinit tg3_phy_probe(struct tg3 *tp)
|
||||||
|
|
|
@ -2702,9 +2702,6 @@ struct tg3_link_config {
|
||||||
u8 active_flowctrl;
|
u8 active_flowctrl;
|
||||||
|
|
||||||
u8 active_duplex;
|
u8 active_duplex;
|
||||||
#define SPEED_INVALID 0xffff
|
|
||||||
#define DUPLEX_INVALID 0xff
|
|
||||||
#define AUTONEG_INVALID 0xff
|
|
||||||
u16 active_speed;
|
u16 active_speed;
|
||||||
u32 rmt_adv;
|
u32 rmt_adv;
|
||||||
};
|
};
|
||||||
|
|
Loading…
Reference in a new issue