diff options
| author | Patrick McHardy <kaber@trash.net> | 2009-07-06 06:23:38 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-07-06 06:23:38 +0400 |
| commit | ec634fe328182a1a098585bfc7b69e5042bdb08d (patch) | |
| tree | 02883aaee743af10d78d6d8fe9181944b4dd4529 /drivers/net/wan/dlci.c | |
| parent | a73e76e23c20c05cb8c0b7ba8bc9daa04b05c80f (diff) | |
| download | linux-ec634fe328182a1a098585bfc7b69e5042bdb08d.tar.xz | |
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/net/wan/dlci.c')
| -rw-r--r-- | drivers/net/wan/dlci.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/wan/dlci.c b/drivers/net/wan/dlci.c index 2fa275a58f9d..8526b6d1ee4d 100644 --- a/drivers/net/wan/dlci.c +++ b/drivers/net/wan/dlci.c @@ -194,7 +194,7 @@ static int dlci_transmit(struct sk_buff *skb, struct net_device *dev) ret = 0; if (!skb || !dev) - return(0); + return NETDEV_TX_OK; dlp = netdev_priv(dev); @@ -219,7 +219,7 @@ static int dlci_transmit(struct sk_buff *skb, struct net_device *dev) /* Alan Cox recommends always returning 0, and always freeing the packet */ /* experience suggest a slightly more conservative approach */ - if (!ret) + if (ret == NETDEV_TX_OK) { dev_kfree_skb(skb); netif_wake_queue(dev); |