diff options
author | Jonas Gorski <jogo@openwrt.org> | 2011-10-08 11:37:14 +0000 |
---|---|---|
committer | Jonas Gorski <jogo@openwrt.org> | 2011-10-08 11:37:14 +0000 |
commit | bf2012d91a826b96e015dbbfdc1ca55bcb8819d1 (patch) | |
tree | 0a323f2e0138faa47bb19400723dd7efe79e5022 /target/linux/ar71xx/files | |
parent | 99b500e98937b94f58352a0d3e4bd9c2ae36fca0 (diff) | |
download | master-187ad058-bf2012d91a826b96e015dbbfdc1ca55bcb8819d1.tar.gz master-187ad058-bf2012d91a826b96e015dbbfdc1ca55bcb8819d1.tar.bz2 master-187ad058-bf2012d91a826b96e015dbbfdc1ca55bcb8819d1.zip |
ag71xx: close a race between the phy state machine and link state
A fast stop/start cycle could leave the ag71xx interrupts and tx engine
disabled when using a phy driver with a fixed link and the start/stop
happens between two phy state machine polls.
Prevent this by always forcing the link down on stop regardless of phy
state and having a phy connected.
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@28380 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'target/linux/ar71xx/files')
-rw-r--r-- | target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_phy.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_phy.c b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_phy.c index 75500277f4..85f17b9feb 100644 --- a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_phy.c +++ b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_phy.c @@ -59,15 +59,19 @@ void ag71xx_phy_start(struct ag71xx *ag) void ag71xx_phy_stop(struct ag71xx *ag) { struct ag71xx_platform_data *pdata = ag71xx_get_pdata(ag); + unsigned long flags; - if (ag->phy_dev) { + if (ag->phy_dev) phy_stop(ag->phy_dev); - } else { - if (pdata->has_ar7240_switch) + else if (pdata->has_ar7240_switch) ag71xx_ar7240_stop(ag); + + spin_lock_irqsave(&ag->lock, flags); + if (ag->link) { ag->link = 0; ag71xx_link_adjust(ag); } + spin_unlock_irqrestore(&ag->lock, flags); } static int ag71xx_phy_connect_fixed(struct ag71xx *ag) |