* [PATCH] sky2: enable rx/tx in sky2_phy_reinit() @ 2010-06-17 2:21 Brandon Philips 2010-06-22 14:57 ` Mike McCormack 2010-06-23 21:37 ` David Miller 0 siblings, 2 replies; 6+ messages in thread From: Brandon Philips @ 2010-06-17 2:21 UTC (permalink / raw) To: Stephen Hemminger; +Cc: netdev, davem sky2_phy_reinit is called by the ethtool helpers sky2_set_settings, sky2_nway_reset and sky2_set_pauseparam when netif_running. However, at the end of sky2_phy_init GM_GP_CTRL has GM_GPCR_RX_ENA and GM_GPCR_TX_ENA cleared. So, doing these commands causes the device to stop working: $ ethtool -r eth0 $ ethtool -A eth0 autoneg off Fix this issue by enabling Rx/Tx after running sky2_phy_init in sky2_phy_reinit. Signed-off-by: Brandon Philips <bphilips@suse.de> Tested-by: Brandon Philips <bphilips@suse.de> Cc: stable@kernel.org --- drivers/net/sky2.c | 19 ++++++++++++++----- 1 files changed, 14 insertions(+), 5 deletions(-) diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c index 2111c7b..7985165 100644 --- a/drivers/net/sky2.c +++ b/drivers/net/sky2.c @@ -717,11 +717,24 @@ static void sky2_phy_power_down(struct sky2_hw *hw, unsigned port) sky2_write8(hw, B2_TST_CTRL1, TST_CFG_WRITE_OFF); } +/* Enable Rx/Tx */ +static void sky2_enable_rx_tx(struct sky2_port *sky2) +{ + struct sky2_hw *hw = sky2->hw; + unsigned port = sky2->port; + u16 reg; + + reg = gma_read16(hw, port, GM_GP_CTRL); + reg |= GM_GPCR_RX_ENA | GM_GPCR_TX_ENA; + gma_write16(hw, port, GM_GP_CTRL, reg); +} + /* Force a renegotiation */ static void sky2_phy_reinit(struct sky2_port *sky2) { spin_lock_bh(&sky2->phy_lock); sky2_phy_init(sky2->hw, sky2->port); + sky2_enable_rx_tx(sky2); spin_unlock_bh(&sky2->phy_lock); } @@ -2040,7 +2053,6 @@ static void sky2_link_up(struct sky2_port *sky2) { struct sky2_hw *hw = sky2->hw; unsigned port = sky2->port; - u16 reg; static const char *fc_name[] = { [FC_NONE] = "none", [FC_TX] = "tx", @@ -2048,10 +2060,7 @@ static void sky2_link_up(struct sky2_port *sky2) [FC_BOTH] = "both", }; - /* enable Rx/Tx */ - reg = gma_read16(hw, port, GM_GP_CTRL); - reg |= GM_GPCR_RX_ENA | GM_GPCR_TX_ENA; - gma_write16(hw, port, GM_GP_CTRL, reg); + sky2_enable_rx_tx(sky2); gm_phy_write(hw, port, PHY_MARV_INT_MASK, PHY_M_DEF_MSK); -- 1.7.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] sky2: enable rx/tx in sky2_phy_reinit() 2010-06-17 2:21 [PATCH] sky2: enable rx/tx in sky2_phy_reinit() Brandon Philips @ 2010-06-22 14:57 ` Mike McCormack 2010-06-23 17:00 ` Brandon Philips 2010-06-23 21:37 ` David Miller 1 sibling, 1 reply; 6+ messages in thread From: Mike McCormack @ 2010-06-22 14:57 UTC (permalink / raw) To: Brandon Philips; +Cc: Stephen Hemminger, netdev, davem Tested, and verified that it fixes the bug reported. Tested-by: Mike McCormack <mikem@ring3k.org> On 17 June 2010 11:21, Brandon Philips <brandon@ifup.org> wrote: > sky2_phy_reinit is called by the ethtool helpers sky2_set_settings, > sky2_nway_reset and sky2_set_pauseparam when netif_running. > > However, at the end of sky2_phy_init GM_GP_CTRL has GM_GPCR_RX_ENA and > GM_GPCR_TX_ENA cleared. So, doing these commands causes the device to > stop working: > > $ ethtool -r eth0 > $ ethtool -A eth0 autoneg off > > Fix this issue by enabling Rx/Tx after running sky2_phy_init in > sky2_phy_reinit. > > Signed-off-by: Brandon Philips <bphilips@suse.de> > Tested-by: Brandon Philips <bphilips@suse.de> > Cc: stable@kernel.org > > --- > drivers/net/sky2.c | 19 ++++++++++++++----- > 1 files changed, 14 insertions(+), 5 deletions(-) > > diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c > index 2111c7b..7985165 100644 > --- a/drivers/net/sky2.c > +++ b/drivers/net/sky2.c > @@ -717,11 +717,24 @@ static void sky2_phy_power_down(struct sky2_hw *hw, unsigned port) > sky2_write8(hw, B2_TST_CTRL1, TST_CFG_WRITE_OFF); > } > > +/* Enable Rx/Tx */ > +static void sky2_enable_rx_tx(struct sky2_port *sky2) > +{ > + struct sky2_hw *hw = sky2->hw; > + unsigned port = sky2->port; > + u16 reg; > + > + reg = gma_read16(hw, port, GM_GP_CTRL); > + reg |= GM_GPCR_RX_ENA | GM_GPCR_TX_ENA; > + gma_write16(hw, port, GM_GP_CTRL, reg); > +} > + > /* Force a renegotiation */ > static void sky2_phy_reinit(struct sky2_port *sky2) > { > spin_lock_bh(&sky2->phy_lock); > sky2_phy_init(sky2->hw, sky2->port); > + sky2_enable_rx_tx(sky2); > spin_unlock_bh(&sky2->phy_lock); > } > > @@ -2040,7 +2053,6 @@ static void sky2_link_up(struct sky2_port *sky2) > { > struct sky2_hw *hw = sky2->hw; > unsigned port = sky2->port; > - u16 reg; > static const char *fc_name[] = { > [FC_NONE] = "none", > [FC_TX] = "tx", > @@ -2048,10 +2060,7 @@ static void sky2_link_up(struct sky2_port *sky2) > [FC_BOTH] = "both", > }; > > - /* enable Rx/Tx */ > - reg = gma_read16(hw, port, GM_GP_CTRL); > - reg |= GM_GPCR_RX_ENA | GM_GPCR_TX_ENA; > - gma_write16(hw, port, GM_GP_CTRL, reg); > + sky2_enable_rx_tx(sky2); > > gm_phy_write(hw, port, PHY_MARV_INT_MASK, PHY_M_DEF_MSK); > > -- > 1.7.1 > > > -- > 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 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] sky2: enable rx/tx in sky2_phy_reinit() 2010-06-22 14:57 ` Mike McCormack @ 2010-06-23 17:00 ` Brandon Philips 2010-07-12 11:04 ` Brandon Philips 0 siblings, 1 reply; 6+ messages in thread From: Brandon Philips @ 2010-06-23 17:00 UTC (permalink / raw) To: Mike McCormack; +Cc: Stephen Hemminger, netdev, davem On 23:57 Tue 22 Jun 2010, Mike McCormack wrote: > Tested, and verified that it fixes the bug reported. Thanks for testing Mike. While testing this I found a new unrelated bug. If I turn off speed autoneg the interface stops sending packets until I turn off pause autoneg also. To illustrate start pinging some machine and run this: $ ethtool --change eth0 speed 100 duplex full autoneg off # At this point the ping stops $ ethtool -A eth0 autoneg off # Ping starts up again When I tried capturing packets nothing was coming from the device. 0ea065e52eb6a0f029b5fa5ed2f142be1b66a153 implemeneted the behaviour of having seperate pause and speed ethtool controls for sky2. Reverting this (see quick ugly revert below) obviously fixes the issue. Any ideas on how to fix this in a proper way though? Cheers, Brandon >From 8de50faa1911933dc545f663a24f32d0caeea3b4 Mon Sep 17 00:00:00 2001 From: Brandon Philips <brandon@ifup.org> Date: Wed, 23 Jun 2010 09:56:20 -0700 Subject: [PATCH] sky2: revert "fix pause negotiation" Revert 0ea065e52eb6a0f029b5fa5ed2f142be1b66a153 --- drivers/net/sky2.c | 31 ++++++++++++------------------- drivers/net/sky2.h | 2 ++ 2 files changed, 14 insertions(+), 19 deletions(-) diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c index 7985165..a638171 100644 --- a/drivers/net/sky2.c +++ b/drivers/net/sky2.c @@ -333,7 +333,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) struct sky2_port *sky2 = netdev_priv(hw->dev[port]); u16 ctrl, ct1000, adv, pg, ledctrl, ledover, reg; - if ( (sky2->flags & SKY2_FLAG_AUTO_SPEED) && + if (sky2->autoneg == AUTONEG_ENABLE && !(hw->flags & SKY2_HW_NEWER_PHY)) { u16 ectrl = gm_phy_read(hw, port, PHY_MARV_EXT_CTRL); @@ -375,7 +375,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) ctrl |= PHY_M_PC_MDI_XMODE(PHY_M_PC_ENA_AUTO); /* downshift on PHY 88E1112 and 88E1149 is changed */ - if ( (sky2->flags & SKY2_FLAG_AUTO_SPEED) && + if (sky2->autoneg == AUTONEG_ENABLE && (hw->flags & SKY2_HW_NEWER_PHY)) { /* set downshift counter to 3x and enable downshift */ ctrl &= ~PHY_M_PC_DSC_MSK; @@ -420,7 +420,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) adv = PHY_AN_CSMA; reg = 0; - if (sky2->flags & SKY2_FLAG_AUTO_SPEED) { + if (sky2->autoneg == AUTONEG_ENABLE) { if (sky2_is_copper(hw)) { if (sky2->advertising & ADVERTISED_1000baseT_Full) ct1000 |= PHY_M_1000C_AFD; @@ -435,11 +435,14 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) if (sky2->advertising & ADVERTISED_10baseT_Half) adv |= PHY_M_AN_10_HD; + adv |= copper_fc_adv[sky2->flow_mode]; } else { /* special defines for FIBER (88E1040S only) */ if (sky2->advertising & ADVERTISED_1000baseT_Full) adv |= PHY_M_AN_1000X_AFD; if (sky2->advertising & ADVERTISED_1000baseT_Half) adv |= PHY_M_AN_1000X_AHD; + + adv |= fiber_fc_adv[sky2->flow_mode]; } /* Restart Auto-negotiation */ @@ -448,8 +451,8 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) /* forced speed/duplex settings */ ct1000 = PHY_M_1000C_MSE; - /* Disable auto update for duplex flow control and duplex */ - reg |= GM_GPCR_AU_DUP_DIS | GM_GPCR_AU_SPD_DIS; + /* Disable auto update for duplex flow control and speed */ + reg |= GM_GPCR_AU_ALL_DIS; switch (sky2->speed) { case SPEED_1000: @@ -467,15 +470,8 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) ctrl |= PHY_CT_DUP_MD; } else if (sky2->speed < SPEED_1000) sky2->flow_mode = FC_NONE; - } - if (sky2->flags & SKY2_FLAG_AUTO_PAUSE) { - if (sky2_is_copper(hw)) - adv |= copper_fc_adv[sky2->flow_mode]; - else - adv |= fiber_fc_adv[sky2->flow_mode]; - } else { - reg |= GM_GPCR_AU_FCT_DIS; + reg |= gm_fc_disable[sky2->flow_mode]; /* Forward pause packets to GMAC? */ @@ -620,8 +616,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) /* no effect on Yukon-XL */ gm_phy_write(hw, port, PHY_MARV_LED_CTRL, ledctrl); - if (!(sky2->flags & SKY2_FLAG_AUTO_SPEED) || - sky2->speed == SPEED_100) { + if (sky2->autoneg == AUTONEG_DISABLE || sky2->speed == SPEED_100) { /* turn on 100 Mbps LED (LED_LINK100) */ ledover |= PHY_M_LED_MO_100(MO_LED_ON); } @@ -632,7 +627,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) } /* Enable phy interrupt on auto-negotiation complete (or link up) */ - if (sky2->flags & SKY2_FLAG_AUTO_SPEED) + if (sky2->autoneg == AUTONEG_ENABLE) gm_phy_write(hw, port, PHY_MARV_INT_MASK, PHY_M_IS_AN_COMPL); else gm_phy_write(hw, port, PHY_MARV_INT_MASK, PHY_M_DEF_MSK); @@ -688,9 +683,7 @@ static void sky2_phy_power_down(struct sky2_hw *hw, unsigned port) /* setup General Purpose Control Register */ gma_write16(hw, port, GM_GP_CTRL, - GM_GPCR_FL_PASS | GM_GPCR_SPEED_100 | - GM_GPCR_AU_DUP_DIS | GM_GPCR_AU_FCT_DIS | - GM_GPCR_AU_SPD_DIS); + GM_GPCR_FL_PASS | GM_GPCR_SPEED_100 | GM_GPCR_AU_ALL_DIS); if (hw->chip_id != CHIP_ID_YUKON_EC) { if (hw->chip_id == CHIP_ID_YUKON_EC_U) { diff --git a/drivers/net/sky2.h b/drivers/net/sky2.h index 084eff2..db0b2ad 100644 --- a/drivers/net/sky2.h +++ b/drivers/net/sky2.h @@ -1755,6 +1755,7 @@ enum { }; #define GM_GPCR_SPEED_1000 (GM_GPCR_GIGS_ENA | GM_GPCR_SPEED_100) +#define GM_GPCR_AU_ALL_DIS (GM_GPCR_AU_DUP_DIS | GM_GPCR_AU_FCT_DIS|GM_GPCR_AU_SPD_DIS) /* GM_TX_CTRL 16 bit r/w Transmit Control Register */ enum { @@ -2247,6 +2248,7 @@ struct sky2_port { u16 speed; /* SPEED_1000, SPEED_100, ... */ u8 wol; /* WAKE_ bits */ u8 duplex; /* DUPLEX_HALF, DUPLEX_FULL */ + u8 autoneg; /* AUTONEG_ENABLE, AUTONEG_DISABLE */ u16 flags; #define SKY2_FLAG_RX_CHECKSUM 0x0001 #define SKY2_FLAG_AUTO_SPEED 0x0002 -- 1.7.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] sky2: enable rx/tx in sky2_phy_reinit() 2010-06-23 17:00 ` Brandon Philips @ 2010-07-12 11:04 ` Brandon Philips 2010-07-12 15:48 ` Stephen Hemminger 0 siblings, 1 reply; 6+ messages in thread From: Brandon Philips @ 2010-07-12 11:04 UTC (permalink / raw) To: Stephen Hemminger; +Cc: netdev, davem, Mike McCormack On 10:00 Wed 23 Jun 2010, Brandon Philips wrote: > While testing this I just noticed that if I turn off speed autoneg the > interface stops sending packets until I turn off pause autoneg too. > > To illustrate start pinging some machine and run this: > > $ ethtool --change eth0 speed 100 duplex full autoneg off > # At this point the ping stops > > $ ethtool -A eth0 autoneg off > # Ping starts up again > > When I tried capturing packets nothing was coming from the device. > > 0ea065e52eb6a0f029b5fa5ed2f142be1b66a153 implemeneted the behaviour of > having seperate pause and speed ethtool controls for sky2. Reverting > this (see quick ugly revert below) obviously fixes the issue. > > Any ideas on how to fix this in a proper way though? Hello Stephen- Any ideas on how to fix this properly? Thanks, Brandon > From 8de50faa1911933dc545f663a24f32d0caeea3b4 Mon Sep 17 00:00:00 2001 > From: Brandon Philips <brandon@ifup.org> > Date: Wed, 23 Jun 2010 09:56:20 -0700 > Subject: [PATCH] sky2: revert "fix pause negotiation" > > Revert 0ea065e52eb6a0f029b5fa5ed2f142be1b66a153 > --- > drivers/net/sky2.c | 31 ++++++++++++------------------- > drivers/net/sky2.h | 2 ++ > 2 files changed, 14 insertions(+), 19 deletions(-) > > diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c > index 7985165..a638171 100644 > --- a/drivers/net/sky2.c > +++ b/drivers/net/sky2.c > @@ -333,7 +333,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) > struct sky2_port *sky2 = netdev_priv(hw->dev[port]); > u16 ctrl, ct1000, adv, pg, ledctrl, ledover, reg; > > - if ( (sky2->flags & SKY2_FLAG_AUTO_SPEED) && > + if (sky2->autoneg == AUTONEG_ENABLE && > !(hw->flags & SKY2_HW_NEWER_PHY)) { > u16 ectrl = gm_phy_read(hw, port, PHY_MARV_EXT_CTRL); > > @@ -375,7 +375,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) > ctrl |= PHY_M_PC_MDI_XMODE(PHY_M_PC_ENA_AUTO); > > /* downshift on PHY 88E1112 and 88E1149 is changed */ > - if ( (sky2->flags & SKY2_FLAG_AUTO_SPEED) && > + if (sky2->autoneg == AUTONEG_ENABLE && > (hw->flags & SKY2_HW_NEWER_PHY)) { > /* set downshift counter to 3x and enable downshift */ > ctrl &= ~PHY_M_PC_DSC_MSK; > @@ -420,7 +420,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) > adv = PHY_AN_CSMA; > reg = 0; > > - if (sky2->flags & SKY2_FLAG_AUTO_SPEED) { > + if (sky2->autoneg == AUTONEG_ENABLE) { > if (sky2_is_copper(hw)) { > if (sky2->advertising & ADVERTISED_1000baseT_Full) > ct1000 |= PHY_M_1000C_AFD; > @@ -435,11 +435,14 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) > if (sky2->advertising & ADVERTISED_10baseT_Half) > adv |= PHY_M_AN_10_HD; > > + adv |= copper_fc_adv[sky2->flow_mode]; > } else { /* special defines for FIBER (88E1040S only) */ > if (sky2->advertising & ADVERTISED_1000baseT_Full) > adv |= PHY_M_AN_1000X_AFD; > if (sky2->advertising & ADVERTISED_1000baseT_Half) > adv |= PHY_M_AN_1000X_AHD; > + > + adv |= fiber_fc_adv[sky2->flow_mode]; > } > > /* Restart Auto-negotiation */ > @@ -448,8 +451,8 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) > /* forced speed/duplex settings */ > ct1000 = PHY_M_1000C_MSE; > > - /* Disable auto update for duplex flow control and duplex */ > - reg |= GM_GPCR_AU_DUP_DIS | GM_GPCR_AU_SPD_DIS; > + /* Disable auto update for duplex flow control and speed */ > + reg |= GM_GPCR_AU_ALL_DIS; > > switch (sky2->speed) { > case SPEED_1000: > @@ -467,15 +470,8 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) > ctrl |= PHY_CT_DUP_MD; > } else if (sky2->speed < SPEED_1000) > sky2->flow_mode = FC_NONE; > - } > > - if (sky2->flags & SKY2_FLAG_AUTO_PAUSE) { > - if (sky2_is_copper(hw)) > - adv |= copper_fc_adv[sky2->flow_mode]; > - else > - adv |= fiber_fc_adv[sky2->flow_mode]; > - } else { > - reg |= GM_GPCR_AU_FCT_DIS; > + > reg |= gm_fc_disable[sky2->flow_mode]; > > /* Forward pause packets to GMAC? */ > @@ -620,8 +616,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) > /* no effect on Yukon-XL */ > gm_phy_write(hw, port, PHY_MARV_LED_CTRL, ledctrl); > > - if (!(sky2->flags & SKY2_FLAG_AUTO_SPEED) || > - sky2->speed == SPEED_100) { > + if (sky2->autoneg == AUTONEG_DISABLE || sky2->speed == SPEED_100) { > /* turn on 100 Mbps LED (LED_LINK100) */ > ledover |= PHY_M_LED_MO_100(MO_LED_ON); > } > @@ -632,7 +627,7 @@ static void sky2_phy_init(struct sky2_hw *hw, unsigned port) > } > > /* Enable phy interrupt on auto-negotiation complete (or link up) */ > - if (sky2->flags & SKY2_FLAG_AUTO_SPEED) > + if (sky2->autoneg == AUTONEG_ENABLE) > gm_phy_write(hw, port, PHY_MARV_INT_MASK, PHY_M_IS_AN_COMPL); > else > gm_phy_write(hw, port, PHY_MARV_INT_MASK, PHY_M_DEF_MSK); > @@ -688,9 +683,7 @@ static void sky2_phy_power_down(struct sky2_hw *hw, unsigned port) > > /* setup General Purpose Control Register */ > gma_write16(hw, port, GM_GP_CTRL, > - GM_GPCR_FL_PASS | GM_GPCR_SPEED_100 | > - GM_GPCR_AU_DUP_DIS | GM_GPCR_AU_FCT_DIS | > - GM_GPCR_AU_SPD_DIS); > + GM_GPCR_FL_PASS | GM_GPCR_SPEED_100 | GM_GPCR_AU_ALL_DIS); > > if (hw->chip_id != CHIP_ID_YUKON_EC) { > if (hw->chip_id == CHIP_ID_YUKON_EC_U) { > diff --git a/drivers/net/sky2.h b/drivers/net/sky2.h > index 084eff2..db0b2ad 100644 > --- a/drivers/net/sky2.h > +++ b/drivers/net/sky2.h > @@ -1755,6 +1755,7 @@ enum { > }; > > #define GM_GPCR_SPEED_1000 (GM_GPCR_GIGS_ENA | GM_GPCR_SPEED_100) > +#define GM_GPCR_AU_ALL_DIS (GM_GPCR_AU_DUP_DIS | GM_GPCR_AU_FCT_DIS|GM_GPCR_AU_SPD_DIS) > > /* GM_TX_CTRL 16 bit r/w Transmit Control Register */ > enum { > @@ -2247,6 +2248,7 @@ struct sky2_port { > u16 speed; /* SPEED_1000, SPEED_100, ... */ > u8 wol; /* WAKE_ bits */ > u8 duplex; /* DUPLEX_HALF, DUPLEX_FULL */ > + u8 autoneg; /* AUTONEG_ENABLE, AUTONEG_DISABLE */ > u16 flags; > #define SKY2_FLAG_RX_CHECKSUM 0x0001 > #define SKY2_FLAG_AUTO_SPEED 0x0002 > -- > 1.7.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] sky2: enable rx/tx in sky2_phy_reinit() 2010-07-12 11:04 ` Brandon Philips @ 2010-07-12 15:48 ` Stephen Hemminger 0 siblings, 0 replies; 6+ messages in thread From: Stephen Hemminger @ 2010-07-12 15:48 UTC (permalink / raw) To: Brandon Philips; +Cc: netdev, davem, Mike McCormack On Mon, 12 Jul 2010 04:04:10 -0700 Brandon Philips <brandon@ifup.org> wrote: > On 10:00 Wed 23 Jun 2010, Brandon Philips wrote: > > While testing this I just noticed that if I turn off speed autoneg the > > interface stops sending packets until I turn off pause autoneg too. > > > > To illustrate start pinging some machine and run this: > > > > $ ethtool --change eth0 speed 100 duplex full autoneg off > > # At this point the ping stops > > > > $ ethtool -A eth0 autoneg off > > # Ping starts up again > > > > When I tried capturing packets nothing was coming from the device. > > > > 0ea065e52eb6a0f029b5fa5ed2f142be1b66a153 implemeneted the behaviour of > > having seperate pause and speed ethtool controls for sky2. Reverting > > this (see quick ugly revert below) obviously fixes the issue. > > > > Any ideas on how to fix this in a proper way though? No quick ideas; try doing walkthrough with debugging to see what PHY interrupt status is. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] sky2: enable rx/tx in sky2_phy_reinit() 2010-06-17 2:21 [PATCH] sky2: enable rx/tx in sky2_phy_reinit() Brandon Philips 2010-06-22 14:57 ` Mike McCormack @ 2010-06-23 21:37 ` David Miller 1 sibling, 0 replies; 6+ messages in thread From: David Miller @ 2010-06-23 21:37 UTC (permalink / raw) To: brandon; +Cc: shemminger, netdev From: Brandon Philips <brandon@ifup.org> Date: Wed, 16 Jun 2010 19:21:58 -0700 > sky2_phy_reinit is called by the ethtool helpers sky2_set_settings, > sky2_nway_reset and sky2_set_pauseparam when netif_running. > > However, at the end of sky2_phy_init GM_GP_CTRL has GM_GPCR_RX_ENA and > GM_GPCR_TX_ENA cleared. So, doing these commands causes the device to > stop working: > > $ ethtool -r eth0 > $ ethtool -A eth0 autoneg off > > Fix this issue by enabling Rx/Tx after running sky2_phy_init in > sky2_phy_reinit. > > Signed-off-by: Brandon Philips <bphilips@suse.de> > Tested-by: Brandon Philips <bphilips@suse.de> > Cc: stable@kernel.org Applied, thanks for fixing this bug. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2010-07-12 15:49 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2010-06-17 2:21 [PATCH] sky2: enable rx/tx in sky2_phy_reinit() Brandon Philips 2010-06-22 14:57 ` Mike McCormack 2010-06-23 17:00 ` Brandon Philips 2010-07-12 11:04 ` Brandon Philips 2010-07-12 15:48 ` Stephen Hemminger 2010-06-23 21:37 ` David Miller
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).