Commit d8a6472f authored by David S. Miller's avatar David S. Miller

Merge branch 'netdev_tx_t'

Luc Van Oostenryck says:

====================
net: always use netdev_tx_t for xmit()'s return type

The ndo_start_xmit() methods should return a 'netdev_tx_t', not
an int, and so should return NETDEV_TX_OK, not 0.
The patches in the series fix most of the remaning drivers and
subsystems (those included in allyesconfig on x86).
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents fe80536a 2a784784
...@@ -1006,7 +1006,7 @@ static void cfhsi_aggregation_tout(struct timer_list *t) ...@@ -1006,7 +1006,7 @@ static void cfhsi_aggregation_tout(struct timer_list *t)
cfhsi_start_tx(cfhsi); cfhsi_start_tx(cfhsi);
} }
static int cfhsi_xmit(struct sk_buff *skb, struct net_device *dev) static netdev_tx_t cfhsi_xmit(struct sk_buff *skb, struct net_device *dev)
{ {
struct cfhsi *cfhsi = NULL; struct cfhsi *cfhsi = NULL;
int start_xfer = 0; int start_xfer = 0;
...@@ -1072,7 +1072,7 @@ static int cfhsi_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -1072,7 +1072,7 @@ static int cfhsi_xmit(struct sk_buff *skb, struct net_device *dev)
spin_unlock_bh(&cfhsi->lock); spin_unlock_bh(&cfhsi->lock);
if (aggregate_ready) if (aggregate_ready)
cfhsi_start_tx(cfhsi); cfhsi_start_tx(cfhsi);
return 0; return NETDEV_TX_OK;
} }
/* Delete inactivity timer if started. */ /* Delete inactivity timer if started. */
...@@ -1102,7 +1102,7 @@ static int cfhsi_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -1102,7 +1102,7 @@ static int cfhsi_xmit(struct sk_buff *skb, struct net_device *dev)
queue_work(cfhsi->wq, &cfhsi->wake_up_work); queue_work(cfhsi->wq, &cfhsi->wake_up_work);
} }
return 0; return NETDEV_TX_OK;
} }
static const struct net_device_ops cfhsi_netdevops; static const struct net_device_ops cfhsi_netdevops;
......
...@@ -266,7 +266,7 @@ static int handle_tx(struct ser_device *ser) ...@@ -266,7 +266,7 @@ static int handle_tx(struct ser_device *ser)
return tty_wr; return tty_wr;
} }
static int caif_xmit(struct sk_buff *skb, struct net_device *dev) static netdev_tx_t caif_xmit(struct sk_buff *skb, struct net_device *dev)
{ {
struct ser_device *ser; struct ser_device *ser;
......
...@@ -488,7 +488,7 @@ static void cfspi_xfer_done_cb(struct cfspi_ifc *ifc) ...@@ -488,7 +488,7 @@ static void cfspi_xfer_done_cb(struct cfspi_ifc *ifc)
complete(&cfspi->comp); complete(&cfspi->comp);
} }
static int cfspi_xmit(struct sk_buff *skb, struct net_device *dev) static netdev_tx_t cfspi_xmit(struct sk_buff *skb, struct net_device *dev)
{ {
struct cfspi *cfspi = NULL; struct cfspi *cfspi = NULL;
unsigned long flags; unsigned long flags;
...@@ -514,7 +514,7 @@ static int cfspi_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -514,7 +514,7 @@ static int cfspi_xmit(struct sk_buff *skb, struct net_device *dev)
cfspi->cfdev.flowctrl(cfspi->ndev, 0); cfspi->cfdev.flowctrl(cfspi->ndev, 0);
} }
return 0; return NETDEV_TX_OK;
} }
int cfspi_rxfrm(struct cfspi *cfspi, u8 *buf, size_t len) int cfspi_rxfrm(struct cfspi *cfspi, u8 *buf, size_t len)
......
...@@ -519,7 +519,7 @@ static struct buf_info *cfv_alloc_and_copy_to_shm(struct cfv_info *cfv, ...@@ -519,7 +519,7 @@ static struct buf_info *cfv_alloc_and_copy_to_shm(struct cfv_info *cfv,
} }
/* Put the CAIF packet on the virtio ring and kick the receiver */ /* Put the CAIF packet on the virtio ring and kick the receiver */
static int cfv_netdev_tx(struct sk_buff *skb, struct net_device *netdev) static netdev_tx_t cfv_netdev_tx(struct sk_buff *skb, struct net_device *netdev)
{ {
struct cfv_info *cfv = netdev_priv(netdev); struct cfv_info *cfv = netdev_priv(netdev);
struct buf_info *buf_info; struct buf_info *buf_info;
......
...@@ -95,7 +95,7 @@ static int aq_ndev_close(struct net_device *ndev) ...@@ -95,7 +95,7 @@ static int aq_ndev_close(struct net_device *ndev)
return err; return err;
} }
static int aq_ndev_start_xmit(struct sk_buff *skb, struct net_device *ndev) static netdev_tx_t aq_ndev_start_xmit(struct sk_buff *skb, struct net_device *ndev)
{ {
struct aq_nic_s *aq_nic = netdev_priv(ndev); struct aq_nic_s *aq_nic = netdev_priv(ndev);
......
...@@ -673,7 +673,7 @@ static struct net_device_stats *arc_emac_stats(struct net_device *ndev) ...@@ -673,7 +673,7 @@ static struct net_device_stats *arc_emac_stats(struct net_device *ndev)
* *
* This function is invoked from upper layers to initiate transmission. * This function is invoked from upper layers to initiate transmission.
*/ */
static int arc_emac_tx(struct sk_buff *skb, struct net_device *ndev) static netdev_tx_t arc_emac_tx(struct sk_buff *skb, struct net_device *ndev)
{ {
struct arc_emac_priv *priv = netdev_priv(ndev); struct arc_emac_priv *priv = netdev_priv(ndev);
unsigned int len, *txbd_curr = &priv->txbd_curr; unsigned int len, *txbd_curr = &priv->txbd_curr;
......
...@@ -384,7 +384,7 @@ static void nb8800_tx_dma_start_irq(struct net_device *dev) ...@@ -384,7 +384,7 @@ static void nb8800_tx_dma_start_irq(struct net_device *dev)
spin_unlock(&priv->tx_lock); spin_unlock(&priv->tx_lock);
} }
static int nb8800_xmit(struct sk_buff *skb, struct net_device *dev) static netdev_tx_t nb8800_xmit(struct sk_buff *skb, struct net_device *dev)
{ {
struct nb8800_priv *priv = netdev_priv(dev); struct nb8800_priv *priv = netdev_priv(dev);
struct nb8800_tx_desc *txd; struct nb8800_tx_desc *txd;
......
...@@ -562,7 +562,7 @@ int t4vf_sge_alloc_eth_txq(struct adapter *, struct sge_eth_txq *, ...@@ -562,7 +562,7 @@ int t4vf_sge_alloc_eth_txq(struct adapter *, struct sge_eth_txq *,
unsigned int); unsigned int);
void t4vf_free_sge_resources(struct adapter *); void t4vf_free_sge_resources(struct adapter *);
int t4vf_eth_xmit(struct sk_buff *, struct net_device *); netdev_tx_t t4vf_eth_xmit(struct sk_buff *, struct net_device *);
int t4vf_ethrx_handler(struct sge_rspq *, const __be64 *, int t4vf_ethrx_handler(struct sge_rspq *, const __be64 *,
const struct pkt_gl *); const struct pkt_gl *);
......
...@@ -1154,7 +1154,7 @@ static inline void txq_advance(struct sge_txq *tq, unsigned int n) ...@@ -1154,7 +1154,7 @@ static inline void txq_advance(struct sge_txq *tq, unsigned int n)
* *
* Add a packet to an SGE Ethernet TX queue. Runs with softirqs disabled. * Add a packet to an SGE Ethernet TX queue. Runs with softirqs disabled.
*/ */
int t4vf_eth_xmit(struct sk_buff *skb, struct net_device *dev) netdev_tx_t t4vf_eth_xmit(struct sk_buff *skb, struct net_device *dev)
{ {
u32 wr_mid; u32 wr_mid;
u64 cntrl, *end; u64 cntrl, *end;
......
...@@ -974,7 +974,7 @@ static int nfp_net_prep_tx_meta(struct sk_buff *skb, u64 tls_handle) ...@@ -974,7 +974,7 @@ static int nfp_net_prep_tx_meta(struct sk_buff *skb, u64 tls_handle)
* *
* Return: NETDEV_TX_OK on success. * Return: NETDEV_TX_OK on success.
*/ */
static int nfp_net_tx(struct sk_buff *skb, struct net_device *netdev) static netdev_tx_t nfp_net_tx(struct sk_buff *skb, struct net_device *netdev)
{ {
struct nfp_net *nn = netdev_priv(netdev); struct nfp_net *nn = netdev_priv(netdev);
const skb_frag_t *frag; const skb_frag_t *frag;
......
...@@ -2064,7 +2064,7 @@ static int pch_gbe_stop(struct net_device *netdev) ...@@ -2064,7 +2064,7 @@ static int pch_gbe_stop(struct net_device *netdev)
* - NETDEV_TX_OK: Normal end * - NETDEV_TX_OK: Normal end
* - NETDEV_TX_BUSY: Error end * - NETDEV_TX_BUSY: Error end
*/ */
static int pch_gbe_xmit_frame(struct sk_buff *skb, struct net_device *netdev) static netdev_tx_t pch_gbe_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
{ {
struct pch_gbe_adapter *adapter = netdev_priv(netdev); struct pch_gbe_adapter *adapter = netdev_priv(netdev);
struct pch_gbe_tx_ring *tx_ring = adapter->tx_ring; struct pch_gbe_tx_ring *tx_ring = adapter->tx_ring;
......
...@@ -697,7 +697,7 @@ static void xlgmac_tx_timeout(struct net_device *netdev, unsigned int txqueue) ...@@ -697,7 +697,7 @@ static void xlgmac_tx_timeout(struct net_device *netdev, unsigned int txqueue)
schedule_work(&pdata->restart_work); schedule_work(&pdata->restart_work);
} }
static int xlgmac_xmit(struct sk_buff *skb, struct net_device *netdev) static netdev_tx_t xlgmac_xmit(struct sk_buff *skb, struct net_device *netdev)
{ {
struct xlgmac_pdata *pdata = netdev_priv(netdev); struct xlgmac_pdata *pdata = netdev_priv(netdev);
struct xlgmac_pkt_info *tx_pkt_info; struct xlgmac_pkt_info *tx_pkt_info;
......
...@@ -142,7 +142,7 @@ static void plip_timer_bh(struct work_struct *work); ...@@ -142,7 +142,7 @@ static void plip_timer_bh(struct work_struct *work);
static void plip_interrupt(void *dev_id); static void plip_interrupt(void *dev_id);
/* Functions for DEV methods */ /* Functions for DEV methods */
static int plip_tx_packet(struct sk_buff *skb, struct net_device *dev); static netdev_tx_t plip_tx_packet(struct sk_buff *skb, struct net_device *dev);
static int plip_hard_header(struct sk_buff *skb, struct net_device *dev, static int plip_hard_header(struct sk_buff *skb, struct net_device *dev,
unsigned short type, const void *daddr, unsigned short type, const void *daddr,
const void *saddr, unsigned len); const void *saddr, unsigned len);
...@@ -958,7 +958,7 @@ plip_interrupt(void *dev_id) ...@@ -958,7 +958,7 @@ plip_interrupt(void *dev_id)
spin_unlock_irqrestore(&nl->lock, flags); spin_unlock_irqrestore(&nl->lock, flags);
} }
static int static netdev_tx_t
plip_tx_packet(struct sk_buff *skb, struct net_device *dev) plip_tx_packet(struct sk_buff *skb, struct net_device *dev)
{ {
struct net_local *nl = netdev_priv(dev); struct net_local *nl = netdev_priv(dev);
......
...@@ -358,7 +358,7 @@ static int ipheth_close(struct net_device *net) ...@@ -358,7 +358,7 @@ static int ipheth_close(struct net_device *net)
return 0; return 0;
} }
static int ipheth_tx(struct sk_buff *skb, struct net_device *net) static netdev_tx_t ipheth_tx(struct sk_buff *skb, struct net_device *net)
{ {
struct ipheth_device *dev = netdev_priv(net); struct ipheth_device *dev = netdev_priv(net);
struct usb_device *udev = dev->udev; struct usb_device *udev = dev->udev;
......
...@@ -210,7 +210,7 @@ static netdev_features_t hsr_fix_features(struct net_device *dev, ...@@ -210,7 +210,7 @@ static netdev_features_t hsr_fix_features(struct net_device *dev,
return hsr_features_recompute(hsr, features); return hsr_features_recompute(hsr, features);
} }
static int hsr_dev_xmit(struct sk_buff *skb, struct net_device *dev) static netdev_tx_t hsr_dev_xmit(struct sk_buff *skb, struct net_device *dev)
{ {
struct hsr_priv *hsr = netdev_priv(dev); struct hsr_priv *hsr = netdev_priv(dev);
struct hsr_port *master; struct hsr_port *master;
......
...@@ -73,7 +73,7 @@ static void l2tp_eth_dev_uninit(struct net_device *dev) ...@@ -73,7 +73,7 @@ static void l2tp_eth_dev_uninit(struct net_device *dev)
*/ */
} }
static int l2tp_eth_dev_xmit(struct sk_buff *skb, struct net_device *dev) static netdev_tx_t l2tp_eth_dev_xmit(struct sk_buff *skb, struct net_device *dev)
{ {
struct l2tp_eth *priv = netdev_priv(dev); struct l2tp_eth *priv = netdev_priv(dev);
struct l2tp_session *session = priv->session; struct l2tp_session *session = priv->session;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment