Skip to content

Commit

Permalink
gianfar: ensure ECNTRL[R100] is cleared on link state change
Browse files Browse the repository at this point in the history
When changing the link between 100Mbps and 1Gbps in SGMII mode it was
found out that the link would stop working.  The issue is that ECNTRL[R100]
needs to be cleared when in 1Gbps mode.  Older reference manuals didn't
require the explicitly clearing but has since been found it that it is
needed.

Signed-off-by: Li Yang <leoli@freescale.com>
Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
Acked-by: Andy Fleming <afleming@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Li Yang authored and David S. Miller committed Jan 6, 2009
1 parent 1fa17d4 commit f430e49
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions drivers/net/gianfar.c
Original file line number Diff line number Diff line change
Expand Up @@ -1973,6 +1973,8 @@ static void adjust_link(struct net_device *dev)
case 1000:
tempval =
((tempval & ~(MACCFG2_IF)) | MACCFG2_GMII);

ecntrl &= ~(ECNTRL_R100);
break;
case 100:
case 10:
Expand Down

0 comments on commit f430e49

Please sign in to comment.