net: toshiba: fix return type of ndo_start_xmit function
authorYueHaibing <yuehaibing@huawei.com>
Wed, 19 Sep 2018 10:23:39 +0000 (18:23 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 20 Sep 2018 04:17:58 +0000 (21:17 -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/toshiba/ps3_gelic_net.c
drivers/net/ethernet/toshiba/ps3_gelic_net.h
drivers/net/ethernet/toshiba/spider_net.c
drivers/net/ethernet/toshiba/tc35815.c

index 88d74aef218a2b7480a1e84f61ea9f8f63ad3e49..75237c81c63d65dda6951137bcd69ce43b5d65cd 100644 (file)
@@ -845,9 +845,9 @@ static int gelic_card_kick_txdma(struct gelic_card *card,
  * @skb: packet to send out
  * @netdev: interface device structure
  *
- * returns 0 on success, <0 on failure
+ * returns NETDEV_TX_OK on success, NETDEV_TX_BUSY on failure
  */
-int gelic_net_xmit(struct sk_buff *skb, struct net_device *netdev)
+netdev_tx_t gelic_net_xmit(struct sk_buff *skb, struct net_device *netdev)
 {
        struct gelic_card *card = netdev_card(netdev);
        struct gelic_descr *descr;
index 003d0452d9cb157b1b7b2418341d2b9099342ceb..fbbf9b54b173bfc3d14c61253e83e51d3f4088db 100644 (file)
@@ -370,7 +370,7 @@ void gelic_card_up(struct gelic_card *card);
 void gelic_card_down(struct gelic_card *card);
 int gelic_net_open(struct net_device *netdev);
 int gelic_net_stop(struct net_device *netdev);
-int gelic_net_xmit(struct sk_buff *skb, struct net_device *netdev);
+netdev_tx_t gelic_net_xmit(struct sk_buff *skb, struct net_device *netdev);
 void gelic_net_set_multi(struct net_device *netdev);
 void gelic_net_tx_timeout(struct net_device *netdev);
 int gelic_net_setup_netdev(struct net_device *netdev, struct gelic_card *card);
index d925b8203996691f1c380d70ff28de9f9c0b0a49..23417266b7ecc24aca2a0e562cd17985399c9cc3 100644 (file)
@@ -880,9 +880,9 @@ out:
  * @skb: packet to send out
  * @netdev: interface device structure
  *
- * returns 0 on success, !0 on failure
+ * returns NETDEV_TX_OK on success, NETDEV_TX_BUSY on failure
  */
-static int
+static netdev_tx_t
 spider_net_xmit(struct sk_buff *skb, struct net_device *netdev)
 {
        int cnt;
index 7163a8d10dba0bb5bb30299e7de1f1dce76fafc4..6a71c2c0f17d1f9032787bb11c338fed6bf1bbd2 100644 (file)
@@ -474,7 +474,8 @@ static void free_rxbuf_skb(struct pci_dev *hwdev, struct sk_buff *skb, dma_addr_
 /* Index to functions, as function prototypes. */
 
 static int     tc35815_open(struct net_device *dev);
-static int     tc35815_send_packet(struct sk_buff *skb, struct net_device *dev);
+static netdev_tx_t     tc35815_send_packet(struct sk_buff *skb,
+                                           struct net_device *dev);
 static irqreturn_t     tc35815_interrupt(int irq, void *dev_id);
 static int     tc35815_rx(struct net_device *dev, int limit);
 static int     tc35815_poll(struct napi_struct *napi, int budget);
@@ -1248,7 +1249,8 @@ tc35815_open(struct net_device *dev)
  * invariant will hold if you make sure that the netif_*_queue()
  * calls are done at the proper times.
  */
-static int tc35815_send_packet(struct sk_buff *skb, struct net_device *dev)
+static netdev_tx_t
+tc35815_send_packet(struct sk_buff *skb, struct net_device *dev)
 {
        struct tc35815_local *lp = netdev_priv(dev);
        struct TxFD *txfd;