summaryrefslogtreecommitdiffstats
path: root/drivers/net/gianfar.c
diff options
context:
space:
mode:
authorLi Yang <leoli@freescale.com>2009-01-06 14:08:10 -0800
committerDavid S. Miller <davem@davemloft.net>2009-01-06 14:08:10 -0800
commitf430e49e9e0cf821c090abedb399b80ebf0cb4c6 (patch)
tree50b0d36acc55dbe17cce4157c1ef2ddeb2c81b4c /drivers/net/gianfar.c
parent1fa17d4ba43d7e5aab5e90777b07da06524f6748 (diff)
downloadop-kernel-dev-f430e49e9e0cf821c090abedb399b80ebf0cb4c6.zip
op-kernel-dev-f430e49e9e0cf821c090abedb399b80ebf0cb4c6.tar.gz
gianfar: ensure ECNTRL[R100] is cleared on link state change
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>
Diffstat (limited to 'drivers/net/gianfar.c')
-rw-r--r--drivers/net/gianfar.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c
index c672ecf..7fcc3af 100644
--- a/drivers/net/gianfar.c
+++ b/drivers/net/gianfar.c
@@ -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:
OpenPOWER on IntegriCloud