diff options
author | Patrick McHardy <kaber@trash.net> | 2009-07-05 22:23:38 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-07-05 22:23:38 -0400 |
commit | ec634fe328182a1a098585bfc7b69e5042bdb08d (patch) | |
tree | 02883aaee743af10d78d6d8fe9181944b4dd4529 /drivers/staging | |
parent | a73e76e23c20c05cb8c0b7ba8bc9daa04b05c80f (diff) |
net: convert remaining non-symbolic return values in ndo_start_xmit() functions
This patch converts the remaining occurences of raw return values to their
symbolic counterparts in ndo_start_xmit() functions that were missed by the
previous automatic conversion.
Additionally code that assumed the symbolic value of NETDEV_TX_OK to be zero
is changed to explicitly use NETDEV_TX_OK.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/staging')
-rw-r--r-- | drivers/staging/rt2860/rt_main_dev.c | 6 | ||||
-rw-r--r-- | drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c | 4 | ||||
-rw-r--r-- | drivers/staging/rtl8187se/r8180_core.c | 4 | ||||
-rw-r--r-- | drivers/staging/wlan-ng/p80211netdev.c | 2 |
4 files changed, 8 insertions, 8 deletions
diff --git a/drivers/staging/rt2860/rt_main_dev.c b/drivers/staging/rt2860/rt_main_dev.c index f298b9bcec3..35c59d5aa99 100644 --- a/drivers/staging/rt2860/rt_main_dev.c +++ b/drivers/staging/rt2860/rt_main_dev.c | |||
@@ -862,7 +862,7 @@ int rt28xx_packet_xmit(struct sk_buff *skb) | |||
862 | { | 862 | { |
863 | struct net_device *net_dev = skb->dev; | 863 | struct net_device *net_dev = skb->dev; |
864 | PRTMP_ADAPTER pAd = net_dev->ml_priv; | 864 | PRTMP_ADAPTER pAd = net_dev->ml_priv; |
865 | int status = 0; | 865 | int status = NETDEV_TX_OK; |
866 | PNDIS_PACKET pPacket = (PNDIS_PACKET) skb; | 866 | PNDIS_PACKET pPacket = (PNDIS_PACKET) skb; |
867 | 867 | ||
868 | { | 868 | { |
@@ -892,7 +892,7 @@ int rt28xx_packet_xmit(struct sk_buff *skb) | |||
892 | 892 | ||
893 | STASendPackets((NDIS_HANDLE)pAd, (PPNDIS_PACKET) &pPacket, 1); | 893 | STASendPackets((NDIS_HANDLE)pAd, (PPNDIS_PACKET) &pPacket, 1); |
894 | 894 | ||
895 | status = 0; | 895 | status = NETDEV_TX_OK; |
896 | done: | 896 | done: |
897 | 897 | ||
898 | return status; | 898 | return status; |
@@ -923,7 +923,7 @@ INT rt28xx_send_packets( | |||
923 | if (!(net_dev->flags & IFF_UP)) | 923 | if (!(net_dev->flags & IFF_UP)) |
924 | { | 924 | { |
925 | RELEASE_NDIS_PACKET(pAd, (PNDIS_PACKET)skb_p, NDIS_STATUS_FAILURE); | 925 | RELEASE_NDIS_PACKET(pAd, (PNDIS_PACKET)skb_p, NDIS_STATUS_FAILURE); |
926 | return 0; | 926 | return NETDEV_TX_OK; |
927 | } | 927 | } |
928 | 928 | ||
929 | NdisZeroMemory((PUCHAR)&skb_p->cb[CB_OFF], 15); | 929 | NdisZeroMemory((PUCHAR)&skb_p->cb[CB_OFF], 15); |
diff --git a/drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c b/drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c index 1294e05fcf1..1b774601b4a 100644 --- a/drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c +++ b/drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c | |||
@@ -802,13 +802,13 @@ int ieee80211_xmit(struct sk_buff *skb, | |||
802 | if ((*ieee->hard_start_xmit)(txb, dev) == 0) { | 802 | if ((*ieee->hard_start_xmit)(txb, dev) == 0) { |
803 | stats->tx_packets++; | 803 | stats->tx_packets++; |
804 | stats->tx_bytes += txb->payload_size; | 804 | stats->tx_bytes += txb->payload_size; |
805 | return 0; | 805 | return NETDEV_TX_OK; |
806 | } | 806 | } |
807 | ieee80211_txb_free(txb); | 807 | ieee80211_txb_free(txb); |
808 | } | 808 | } |
809 | } | 809 | } |
810 | 810 | ||
811 | return 0; | 811 | return NETDEV_TX_OK; |
812 | 812 | ||
813 | failed: | 813 | failed: |
814 | spin_unlock_irqrestore(&ieee->lock, flags); | 814 | spin_unlock_irqrestore(&ieee->lock, flags); |
diff --git a/drivers/staging/rtl8187se/r8180_core.c b/drivers/staging/rtl8187se/r8180_core.c index 7e2fecae813..26a59118d34 100644 --- a/drivers/staging/rtl8187se/r8180_core.c +++ b/drivers/staging/rtl8187se/r8180_core.c | |||
@@ -3040,7 +3040,7 @@ int rtl8180_hard_start_xmit(struct sk_buff *skb,struct net_device *dev) | |||
3040 | spin_unlock_irqrestore(&priv->tx_lock,flags); | 3040 | spin_unlock_irqrestore(&priv->tx_lock,flags); |
3041 | 3041 | ||
3042 | dev_kfree_skb_any(skb); | 3042 | dev_kfree_skb_any(skb); |
3043 | return 0; | 3043 | return NETDEV_TX_OK; |
3044 | } | 3044 | } |
3045 | 3045 | ||
3046 | rtl8180_tx(dev, skb->data, skb->len, priority, | 3046 | rtl8180_tx(dev, skb->data, skb->len, priority, |
@@ -3051,7 +3051,7 @@ int rtl8180_hard_start_xmit(struct sk_buff *skb,struct net_device *dev) | |||
3051 | spin_unlock_irqrestore(&priv->tx_lock,flags); | 3051 | spin_unlock_irqrestore(&priv->tx_lock,flags); |
3052 | 3052 | ||
3053 | dev_kfree_skb_any(skb); | 3053 | dev_kfree_skb_any(skb); |
3054 | return 0; | 3054 | return NETDEV_TX_OK; |
3055 | } | 3055 | } |
3056 | 3056 | ||
3057 | // longpre 144+48 shortpre 72+24 | 3057 | // longpre 144+48 shortpre 72+24 |
diff --git a/drivers/staging/wlan-ng/p80211netdev.c b/drivers/staging/wlan-ng/p80211netdev.c index 90f499e00dc..c273c034a83 100644 --- a/drivers/staging/wlan-ng/p80211netdev.c +++ b/drivers/staging/wlan-ng/p80211netdev.c | |||
@@ -354,7 +354,7 @@ static int p80211knetdev_hard_start_xmit(struct sk_buff *skb, | |||
354 | p80211_metawep_t p80211_wep; | 354 | p80211_metawep_t p80211_wep; |
355 | 355 | ||
356 | if (skb == NULL) | 356 | if (skb == NULL) |
357 | return 0; | 357 | return NETDEV_TX_OK; |
358 | 358 | ||
359 | if (wlandev->state != WLAN_DEVICE_OPEN) { | 359 | if (wlandev->state != WLAN_DEVICE_OPEN) { |
360 | result = 1; | 360 | result = 1; |