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/s390/net/lcs.c | |
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/s390/net/lcs.c')
-rw-r--r-- | drivers/s390/net/lcs.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/s390/net/lcs.c b/drivers/s390/net/lcs.c index 8c675905448b..a70de9b4bf29 100644 --- a/drivers/s390/net/lcs.c +++ b/drivers/s390/net/lcs.c | |||
@@ -1553,24 +1553,24 @@ __lcs_start_xmit(struct lcs_card *card, struct sk_buff *skb, | |||
1553 | struct net_device *dev) | 1553 | struct net_device *dev) |
1554 | { | 1554 | { |
1555 | struct lcs_header *header; | 1555 | struct lcs_header *header; |
1556 | int rc = 0; | 1556 | int rc = NETDEV_TX_OK; |
1557 | 1557 | ||
1558 | LCS_DBF_TEXT(5, trace, "hardxmit"); | 1558 | LCS_DBF_TEXT(5, trace, "hardxmit"); |
1559 | if (skb == NULL) { | 1559 | if (skb == NULL) { |
1560 | card->stats.tx_dropped++; | 1560 | card->stats.tx_dropped++; |
1561 | card->stats.tx_errors++; | 1561 | card->stats.tx_errors++; |
1562 | return 0; | 1562 | return NETDEV_TX_OK; |
1563 | } | 1563 | } |
1564 | if (card->state != DEV_STATE_UP) { | 1564 | if (card->state != DEV_STATE_UP) { |
1565 | dev_kfree_skb(skb); | 1565 | dev_kfree_skb(skb); |
1566 | card->stats.tx_dropped++; | 1566 | card->stats.tx_dropped++; |
1567 | card->stats.tx_errors++; | 1567 | card->stats.tx_errors++; |
1568 | card->stats.tx_carrier_errors++; | 1568 | card->stats.tx_carrier_errors++; |
1569 | return 0; | 1569 | return NETDEV_TX_OK; |
1570 | } | 1570 | } |
1571 | if (skb->protocol == htons(ETH_P_IPV6)) { | 1571 | if (skb->protocol == htons(ETH_P_IPV6)) { |
1572 | dev_kfree_skb(skb); | 1572 | dev_kfree_skb(skb); |
1573 | return 0; | 1573 | return NETDEV_TX_OK; |
1574 | } | 1574 | } |
1575 | netif_stop_queue(card->dev); | 1575 | netif_stop_queue(card->dev); |
1576 | spin_lock(&card->lock); | 1576 | spin_lock(&card->lock); |