diff options
author | YueHaibing <yuehaibing@huawei.com> | 2018-09-26 11:15:38 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-09-28 19:25:11 +0200 |
commit | eddf11e18dff0e8671e06ce54e64cfc843303ab9 (patch) | |
tree | 2d9a65f6b442bc2f69d7193e2daa1afe50066d69 | |
parent | Merge branch 'netlink-nested-policy-validation' (diff) | |
download | linux-eddf11e18dff0e8671e06ce54e64cfc843303ab9.tar.xz linux-eddf11e18dff0e8671e06ce54e64cfc843303ab9.zip |
net: ovs: fix return type of ndo_start_xmit function
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t',
which is a typedef for an enum type, so make sure the implementation in
this driver has returns 'netdev_tx_t' value, and change the function
return type to netdev_tx_t.
Found by coccinelle.
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/openvswitch/vport-internal_dev.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/net/openvswitch/vport-internal_dev.c b/net/openvswitch/vport-internal_dev.c index bb95c43aae76..26f71cbf7527 100644 --- a/net/openvswitch/vport-internal_dev.c +++ b/net/openvswitch/vport-internal_dev.c @@ -43,7 +43,8 @@ static struct internal_dev *internal_dev_priv(struct net_device *netdev) } /* Called with rcu_read_lock_bh. */ -static int internal_dev_xmit(struct sk_buff *skb, struct net_device *netdev) +static netdev_tx_t +internal_dev_xmit(struct sk_buff *skb, struct net_device *netdev) { int len, err; @@ -62,7 +63,7 @@ static int internal_dev_xmit(struct sk_buff *skb, struct net_device *netdev) } else { netdev->stats.tx_errors++; } - return 0; + return NETDEV_TX_OK; } static int internal_dev_open(struct net_device *netdev) |