Commit ce594e98 authored by hayeswang's avatar hayeswang Committed by David S. Miller

r8152: simply the arguments

Replace &tp->napi with napi and tp->netdev with netdev.
Signed-off-by: default avatarHayes Wang <hayeswang@realtek.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a2d3ebab
...@@ -1761,6 +1761,7 @@ static int rx_bottom(struct r8152 *tp, int budget) ...@@ -1761,6 +1761,7 @@ static int rx_bottom(struct r8152 *tp, int budget)
unsigned long flags; unsigned long flags;
struct list_head *cursor, *next, rx_queue; struct list_head *cursor, *next, rx_queue;
int ret = 0, work_done = 0; int ret = 0, work_done = 0;
struct napi_struct *napi = &tp->napi;
if (!skb_queue_empty(&tp->rx_queue)) { if (!skb_queue_empty(&tp->rx_queue)) {
while (work_done < budget) { while (work_done < budget) {
...@@ -1773,7 +1774,7 @@ static int rx_bottom(struct r8152 *tp, int budget) ...@@ -1773,7 +1774,7 @@ static int rx_bottom(struct r8152 *tp, int budget)
break; break;
pkt_len = skb->len; pkt_len = skb->len;
napi_gro_receive(&tp->napi, skb); napi_gro_receive(napi, skb);
work_done++; work_done++;
stats->rx_packets++; stats->rx_packets++;
stats->rx_bytes += pkt_len; stats->rx_bytes += pkt_len;
...@@ -1823,7 +1824,7 @@ static int rx_bottom(struct r8152 *tp, int budget) ...@@ -1823,7 +1824,7 @@ static int rx_bottom(struct r8152 *tp, int budget)
pkt_len -= CRC_SIZE; pkt_len -= CRC_SIZE;
rx_data += sizeof(struct rx_desc); rx_data += sizeof(struct rx_desc);
skb = napi_alloc_skb(&tp->napi, pkt_len); skb = napi_alloc_skb(napi, pkt_len);
if (!skb) { if (!skb) {
stats->rx_dropped++; stats->rx_dropped++;
goto find_next_rx; goto find_next_rx;
...@@ -1835,7 +1836,7 @@ static int rx_bottom(struct r8152 *tp, int budget) ...@@ -1835,7 +1836,7 @@ static int rx_bottom(struct r8152 *tp, int budget)
skb->protocol = eth_type_trans(skb, netdev); skb->protocol = eth_type_trans(skb, netdev);
rtl_rx_vlan_tag(rx_desc, skb); rtl_rx_vlan_tag(rx_desc, skb);
if (work_done < budget) { if (work_done < budget) {
napi_gro_receive(&tp->napi, skb); napi_gro_receive(napi, skb);
work_done++; work_done++;
stats->rx_packets++; stats->rx_packets++;
stats->rx_bytes += pkt_len; stats->rx_bytes += pkt_len;
...@@ -3150,6 +3151,7 @@ static bool rtl8153_in_nway(struct r8152 *tp) ...@@ -3150,6 +3151,7 @@ static bool rtl8153_in_nway(struct r8152 *tp)
static void set_carrier(struct r8152 *tp) static void set_carrier(struct r8152 *tp)
{ {
struct net_device *netdev = tp->netdev; struct net_device *netdev = tp->netdev;
struct napi_struct *napi = &tp->napi;
u8 speed; u8 speed;
speed = rtl8152_get_speed(tp); speed = rtl8152_get_speed(tp);
...@@ -3159,7 +3161,7 @@ static void set_carrier(struct r8152 *tp) ...@@ -3159,7 +3161,7 @@ static void set_carrier(struct r8152 *tp)
tp->rtl_ops.enable(tp); tp->rtl_ops.enable(tp);
set_bit(RTL8152_SET_RX_MODE, &tp->flags); set_bit(RTL8152_SET_RX_MODE, &tp->flags);
netif_stop_queue(netdev); netif_stop_queue(netdev);
napi_disable(&tp->napi); napi_disable(napi);
netif_carrier_on(netdev); netif_carrier_on(netdev);
rtl_start_rx(tp); rtl_start_rx(tp);
napi_enable(&tp->napi); napi_enable(&tp->napi);
...@@ -3169,9 +3171,9 @@ static void set_carrier(struct r8152 *tp) ...@@ -3169,9 +3171,9 @@ static void set_carrier(struct r8152 *tp)
} else { } else {
if (netif_carrier_ok(netdev)) { if (netif_carrier_ok(netdev)) {
netif_carrier_off(netdev); netif_carrier_off(netdev);
napi_disable(&tp->napi); napi_disable(napi);
tp->rtl_ops.disable(tp); tp->rtl_ops.disable(tp);
napi_enable(&tp->napi); napi_enable(napi);
netif_info(tp, link, netdev, "carrier off\n"); netif_info(tp, link, netdev, "carrier off\n");
} }
} }
...@@ -3633,11 +3635,13 @@ static int rtl8152_runtime_suspend(struct r8152 *tp) ...@@ -3633,11 +3635,13 @@ static int rtl8152_runtime_suspend(struct r8152 *tp)
tp->rtl_ops.autosuspend_en(tp, true); tp->rtl_ops.autosuspend_en(tp, true);
if (netif_carrier_ok(netdev)) { if (netif_carrier_ok(netdev)) {
napi_disable(&tp->napi); struct napi_struct *napi = &tp->napi;
napi_disable(napi);
rtl_stop_rx(tp); rtl_stop_rx(tp);
rxdy_gated_en(tp, false); rxdy_gated_en(tp, false);
ocp_write_dword(tp, MCU_TYPE_PLA, PLA_RCR, rcr); ocp_write_dword(tp, MCU_TYPE_PLA, PLA_RCR, rcr);
napi_enable(&tp->napi); napi_enable(napi);
} }
} }
...@@ -3653,12 +3657,14 @@ static int rtl8152_system_suspend(struct r8152 *tp) ...@@ -3653,12 +3657,14 @@ static int rtl8152_system_suspend(struct r8152 *tp)
netif_device_detach(netdev); netif_device_detach(netdev);
if (netif_running(netdev) && test_bit(WORK_ENABLE, &tp->flags)) { if (netif_running(netdev) && test_bit(WORK_ENABLE, &tp->flags)) {
struct napi_struct *napi = &tp->napi;
clear_bit(WORK_ENABLE, &tp->flags); clear_bit(WORK_ENABLE, &tp->flags);
usb_kill_urb(tp->intr_urb); usb_kill_urb(tp->intr_urb);
napi_disable(&tp->napi); napi_disable(napi);
cancel_delayed_work_sync(&tp->schedule); cancel_delayed_work_sync(&tp->schedule);
tp->rtl_ops.down(tp); tp->rtl_ops.down(tp);
napi_enable(&tp->napi); napi_enable(napi);
} }
return ret; return ret;
...@@ -3684,35 +3690,38 @@ static int rtl8152_suspend(struct usb_interface *intf, pm_message_t message) ...@@ -3684,35 +3690,38 @@ static int rtl8152_suspend(struct usb_interface *intf, pm_message_t message)
static int rtl8152_resume(struct usb_interface *intf) static int rtl8152_resume(struct usb_interface *intf)
{ {
struct r8152 *tp = usb_get_intfdata(intf); struct r8152 *tp = usb_get_intfdata(intf);
struct net_device *netdev = tp->netdev;
mutex_lock(&tp->control); mutex_lock(&tp->control);
if (!test_bit(SELECTIVE_SUSPEND, &tp->flags)) { if (!test_bit(SELECTIVE_SUSPEND, &tp->flags)) {
tp->rtl_ops.init(tp); tp->rtl_ops.init(tp);
queue_delayed_work(system_long_wq, &tp->hw_phy_work, 0); queue_delayed_work(system_long_wq, &tp->hw_phy_work, 0);
netif_device_attach(tp->netdev); netif_device_attach(netdev);
} }
if (netif_running(tp->netdev) && tp->netdev->flags & IFF_UP) { if (netif_running(netdev) && netdev->flags & IFF_UP) {
if (test_bit(SELECTIVE_SUSPEND, &tp->flags)) { if (test_bit(SELECTIVE_SUSPEND, &tp->flags)) {
struct napi_struct *napi = &tp->napi;
tp->rtl_ops.autosuspend_en(tp, false); tp->rtl_ops.autosuspend_en(tp, false);
napi_disable(&tp->napi); napi_disable(napi);
set_bit(WORK_ENABLE, &tp->flags); set_bit(WORK_ENABLE, &tp->flags);
if (netif_carrier_ok(tp->netdev)) if (netif_carrier_ok(netdev))
rtl_start_rx(tp); rtl_start_rx(tp);
napi_enable(&tp->napi); napi_enable(napi);
clear_bit(SELECTIVE_SUSPEND, &tp->flags); clear_bit(SELECTIVE_SUSPEND, &tp->flags);
smp_mb__after_atomic(); smp_mb__after_atomic();
if (!list_empty(&tp->rx_done)) if (!list_empty(&tp->rx_done))
napi_schedule(&tp->napi); napi_schedule(&tp->napi);
} else { } else {
tp->rtl_ops.up(tp); tp->rtl_ops.up(tp);
netif_carrier_off(tp->netdev); netif_carrier_off(netdev);
set_bit(WORK_ENABLE, &tp->flags); set_bit(WORK_ENABLE, &tp->flags);
} }
usb_submit_urb(tp->intr_urb, GFP_KERNEL); usb_submit_urb(tp->intr_urb, GFP_KERNEL);
} else if (test_bit(SELECTIVE_SUSPEND, &tp->flags)) { } else if (test_bit(SELECTIVE_SUSPEND, &tp->flags)) {
if (tp->netdev->flags & IFF_UP) if (netdev->flags & IFF_UP)
tp->rtl_ops.autosuspend_en(tp, false); tp->rtl_ops.autosuspend_en(tp, false);
clear_bit(SELECTIVE_SUSPEND, &tp->flags); clear_bit(SELECTIVE_SUSPEND, &tp->flags);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment