aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYueHaibing <yuehaibing@huawei.com>2018-09-26 05:15:38 -0400
committerDavid S. Miller <davem@davemloft.net>2018-09-28 13:25:11 -0400
commiteddf11e18dff0e8671e06ce54e64cfc843303ab9 (patch)
tree2d9a65f6b442bc2f69d7193e2daa1afe50066d69
parent31df0cffa974df05a7036d1e5e5dbf0733c7f5ff (diff)
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.c5
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)
43} 43}
44 44
45/* Called with rcu_read_lock_bh. */ 45/* Called with rcu_read_lock_bh. */
46static int internal_dev_xmit(struct sk_buff *skb, struct net_device *netdev) 46static netdev_tx_t
47internal_dev_xmit(struct sk_buff *skb, struct net_device *netdev)
47{ 48{
48 int len, err; 49 int len, err;
49 50
@@ -62,7 +63,7 @@ static int internal_dev_xmit(struct sk_buff *skb, struct net_device *netdev)
62 } else { 63 } else {
63 netdev->stats.tx_errors++; 64 netdev->stats.tx_errors++;
64 } 65 }
65 return 0; 66 return NETDEV_TX_OK;
66} 67}
67 68
68static int internal_dev_open(struct net_device *netdev) 69static int internal_dev_open(struct net_device *netdev)