Commit b3e51069 authored by LABBE Corentin's avatar LABBE Corentin Committed by David S. Miller

net: stmmac: replace if (netif_msg_type) by their netif_xxx counterpart

As sugested by Joe Perches, we could replace all
if (netif_msg_type(priv)) dev_xxx(priv->devices, ...)
by the simpler macro netif_xxx(priv, hw, priv->dev, ...)
Signed-off-by: default avatarCorentin Labbe <clabbe.montjoie@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent de9a2165
...@@ -755,10 +755,9 @@ static void stmmac_adjust_link(struct net_device *dev) ...@@ -755,10 +755,9 @@ static void stmmac_adjust_link(struct net_device *dev)
stmmac_hw_fix_mac_speed(priv); stmmac_hw_fix_mac_speed(priv);
break; break;
default: default:
if (netif_msg_link(priv)) netif_warn(priv, link, priv->dev,
netdev_warn(priv->dev, "Speed (%d) not 10/100\n",
"Speed (%d) not 10/100\n", phydev->speed);
phydev->speed);
break; break;
} }
...@@ -1036,14 +1035,14 @@ static int init_dma_desc_rings(struct net_device *dev, gfp_t flags) ...@@ -1036,14 +1035,14 @@ static int init_dma_desc_rings(struct net_device *dev, gfp_t flags)
priv->dma_buf_sz = bfsize; priv->dma_buf_sz = bfsize;
if (netif_msg_probe(priv)) { netif_dbg(priv, probe, priv->dev,
netdev_dbg(priv->dev, "(%s) dma_rx_phy=0x%08x dma_tx_phy=0x%08x\n", "(%s) dma_rx_phy=0x%08x dma_tx_phy=0x%08x\n",
__func__, (u32)priv->dma_rx_phy, __func__, (u32)priv->dma_rx_phy, (u32)priv->dma_tx_phy);
(u32)priv->dma_tx_phy);
/* RX INITIALIZATION */
netif_dbg(priv, probe, priv->dev,
"SKB addresses:\nskb\t\tskb data\tdma data\n");
/* RX INITIALIZATION */
netdev_dbg(priv->dev, "SKB addresses:\nskb\t\tskb data\tdma data\n");
}
for (i = 0; i < DMA_RX_SIZE; i++) { for (i = 0; i < DMA_RX_SIZE; i++) {
struct dma_desc *p; struct dma_desc *p;
if (priv->extend_desc) if (priv->extend_desc)
...@@ -1055,11 +1054,9 @@ static int init_dma_desc_rings(struct net_device *dev, gfp_t flags) ...@@ -1055,11 +1054,9 @@ static int init_dma_desc_rings(struct net_device *dev, gfp_t flags)
if (ret) if (ret)
goto err_init_rx_buffers; goto err_init_rx_buffers;
if (netif_msg_probe(priv)) netif_dbg(priv, probe, priv->dev, "[%p]\t[%p]\t[%x]\n",
netdev_dbg(priv->dev, "[%p]\t[%p]\t[%x]\n", priv->rx_skbuff[i], priv->rx_skbuff[i]->data,
priv->rx_skbuff[i], (unsigned int)priv->rx_skbuff_dma[i]);
priv->rx_skbuff[i]->data,
(unsigned int)priv->rx_skbuff_dma[i]);
} }
priv->cur_rx = 0; priv->cur_rx = 0;
priv->dirty_rx = (unsigned int)(i - DMA_RX_SIZE); priv->dirty_rx = (unsigned int)(i - DMA_RX_SIZE);
...@@ -1389,9 +1386,8 @@ static void stmmac_tx_clean(struct stmmac_priv *priv) ...@@ -1389,9 +1386,8 @@ static void stmmac_tx_clean(struct stmmac_priv *priv)
netif_tx_lock(priv->dev); netif_tx_lock(priv->dev);
if (netif_queue_stopped(priv->dev) && if (netif_queue_stopped(priv->dev) &&
stmmac_tx_avail(priv) > STMMAC_TX_THRESH) { stmmac_tx_avail(priv) > STMMAC_TX_THRESH) {
if (netif_msg_tx_done(priv)) netif_dbg(priv, tx_done, priv->dev,
netdev_dbg(priv->dev, "%s: restart transmit\n", "%s: restart transmit\n", __func__);
__func__);
netif_wake_queue(priv->dev); netif_wake_queue(priv->dev);
} }
netif_tx_unlock(priv->dev); netif_tx_unlock(priv->dev);
...@@ -2096,9 +2092,8 @@ static netdev_tx_t stmmac_tso_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -2096,9 +2092,8 @@ static netdev_tx_t stmmac_tso_xmit(struct sk_buff *skb, struct net_device *dev)
priv->cur_tx = STMMAC_GET_ENTRY(priv->cur_tx, DMA_TX_SIZE); priv->cur_tx = STMMAC_GET_ENTRY(priv->cur_tx, DMA_TX_SIZE);
if (unlikely(stmmac_tx_avail(priv) <= (MAX_SKB_FRAGS + 1))) { if (unlikely(stmmac_tx_avail(priv) <= (MAX_SKB_FRAGS + 1))) {
if (netif_msg_hw(priv)) netif_dbg(priv, hw, priv->dev, "%s: stop transmitted packets\n",
netdev_dbg(priv->dev, "%s: stop transmitted packets\n", __func__);
__func__);
netif_stop_queue(dev); netif_stop_queue(dev);
} }
...@@ -2298,9 +2293,8 @@ static netdev_tx_t stmmac_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -2298,9 +2293,8 @@ static netdev_tx_t stmmac_xmit(struct sk_buff *skb, struct net_device *dev)
} }
if (unlikely(stmmac_tx_avail(priv) <= (MAX_SKB_FRAGS + 1))) { if (unlikely(stmmac_tx_avail(priv) <= (MAX_SKB_FRAGS + 1))) {
if (netif_msg_hw(priv)) netif_dbg(priv, hw, priv->dev, "%s: stop transmitted packets\n",
netdev_dbg(priv->dev, __func__);
"%s: stop transmitted packets\n", __func__);
netif_stop_queue(dev); netif_stop_queue(dev);
} }
...@@ -2465,9 +2459,8 @@ static inline void stmmac_rx_refill(struct stmmac_priv *priv) ...@@ -2465,9 +2459,8 @@ static inline void stmmac_rx_refill(struct stmmac_priv *priv)
if (priv->rx_zeroc_thresh > 0) if (priv->rx_zeroc_thresh > 0)
priv->rx_zeroc_thresh--; priv->rx_zeroc_thresh--;
if (netif_msg_rx_status(priv)) netif_dbg(priv, rx_status, priv->dev,
netdev_dbg(priv->dev, "refill entry #%d\n", entry);
"refill entry #%d\n", entry);
} }
wmb(); wmb();
......
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