diff options
author | hayeswang <hayeswang@realtek.com> | 2014-10-29 11:12:16 +0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-10-30 15:49:00 -0400 |
commit | f4c7476b041d200c3b347f019eebf05e6d0b47f9 (patch) | |
tree | 15725bc40b85bfacf6f2f7aca23e06bcc3383d21 /drivers/net/usb | |
parent | 923e1ee3ff0b585cc4f56cf696c8455708537ffb (diff) | |
download | op-kernel-dev-f4c7476b041d200c3b347f019eebf05e6d0b47f9.zip op-kernel-dev-f4c7476b041d200c3b347f019eebf05e6d0b47f9.tar.gz |
r8152: reset tp->speed before autoresuming in open function
If (tp->speed & LINK_STATUS) is not zero, the rtl8152_resume()
would call rtl_start_rx() before enabling the tx/rx. Avoid this
by resetting it to zero.
Signed-off-by: Hayes Wang <hayeswang@realtek.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/usb')
-rw-r--r-- | drivers/net/usb/r8152.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index a17ca58..f224231 100644 --- a/drivers/net/usb/r8152.c +++ b/drivers/net/usb/r8152.c @@ -2891,6 +2891,9 @@ static int rtl8152_open(struct net_device *netdev) if (res) goto out; + /* set speed to 0 to avoid autoresume try to submit rx */ + tp->speed = 0; + res = usb_autopm_get_interface(tp->intf); if (res < 0) { free_all_mem(tp); @@ -2904,6 +2907,8 @@ static int rtl8152_open(struct net_device *netdev) clear_bit(WORK_ENABLE, &tp->flags); usb_kill_urb(tp->intr_urb); cancel_delayed_work_sync(&tp->schedule); + + /* disable the tx/rx, if the workqueue has enabled them. */ if (tp->speed & LINK_STATUS) tp->rtl_ops.disable(tp); } |