]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
net: ibm: fix return type of ndo_start_xmit function
authorYueHaibing <yuehaibing@huawei.com>
Tue, 18 Sep 2018 06:35:47 +0000 (14:35 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 19 Sep 2018 03:03:37 +0000 (20:03 -0700)
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>
drivers/net/ethernet/ibm/ehea/ehea_main.c
drivers/net/ethernet/ibm/emac/core.c
drivers/net/ethernet/ibm/ibmvnic.c

index ba580bfae512326d346e96e67718bec10ed716cc..88128d3b642961de83d26764c73516ca5f7f600d 100644 (file)
@@ -2038,7 +2038,7 @@ static void ehea_xmit3(struct sk_buff *skb, struct net_device *dev,
        dev_consume_skb_any(skb);
 }
 
-static int ehea_start_xmit(struct sk_buff *skb, struct net_device *dev)
+static netdev_tx_t ehea_start_xmit(struct sk_buff *skb, struct net_device *dev)
 {
        struct ehea_port *port = netdev_priv(dev);
        struct ehea_swqe *swqe;
index 7410a1de8f1d45c2e7b98851c21efdb109cad6c8..5107c9450a19e4380a5449642d562a01543fd46a 100644 (file)
@@ -1409,7 +1409,7 @@ static inline u16 emac_tx_csum(struct emac_instance *dev,
        return 0;
 }
 
-static inline int emac_xmit_finish(struct emac_instance *dev, int len)
+static inline netdev_tx_t emac_xmit_finish(struct emac_instance *dev, int len)
 {
        struct emac_regs __iomem *p = dev->emacp;
        struct net_device *ndev = dev->ndev;
@@ -1436,7 +1436,7 @@ static inline int emac_xmit_finish(struct emac_instance *dev, int len)
 }
 
 /* Tx lock BH */
-static int emac_start_xmit(struct sk_buff *skb, struct net_device *ndev)
+static netdev_tx_t emac_start_xmit(struct sk_buff *skb, struct net_device *ndev)
 {
        struct emac_instance *dev = netdev_priv(ndev);
        unsigned int len = skb->len;
@@ -1494,7 +1494,8 @@ static inline int emac_xmit_split(struct emac_instance *dev, int slot,
 }
 
 /* Tx lock BH disabled (SG version for TAH equipped EMACs) */
-static int emac_start_xmit_sg(struct sk_buff *skb, struct net_device *ndev)
+static netdev_tx_t
+emac_start_xmit_sg(struct sk_buff *skb, struct net_device *ndev)
 {
        struct emac_instance *dev = netdev_priv(ndev);
        int nr_frags = skb_shinfo(skb)->nr_frags;
index 4f0daf67b18df2dcf11d7a406ebc1982e0fee466..a8369addfe688412212d8eb7716f6cbbcbb41d28 100644 (file)
@@ -1428,7 +1428,7 @@ static int ibmvnic_xmit_workarounds(struct sk_buff *skb,
        return 0;
 }
 
-static int ibmvnic_xmit(struct sk_buff *skb, struct net_device *netdev)
+static netdev_tx_t ibmvnic_xmit(struct sk_buff *skb, struct net_device *netdev)
 {
        struct ibmvnic_adapter *adapter = netdev_priv(netdev);
        int queue_num = skb_get_queue_mapping(skb);
@@ -1452,7 +1452,7 @@ static int ibmvnic_xmit(struct sk_buff *skb, struct net_device *netdev)
        u64 *handle_array;
        int index = 0;
        u8 proto = 0;
-       int ret = 0;
+       netdev_tx_t ret = NETDEV_TX_OK;
 
        if (adapter->resetting) {
                if (!netif_subqueue_stopped(netdev, skb))