net: stmmac: replace if (netif_msg_type) by their netif_xxx counterpart
authorLABBE Corentin <clabbe.montjoie@gmail.com>
Wed, 16 Nov 2016 19:09:41 +0000 (20:09 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 17 Nov 2016 18:30:30 +0000 (13:30 -0500)
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: Corentin Labbe <clabbe.montjoie@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c

index d160bdb4d29be34933dea13e9a2939d4b2df28ca..fbd1cd79233d8c00b8837de6c194d502d588c9e5 100644 (file)
@@ -755,10 +755,9 @@ static void stmmac_adjust_link(struct net_device *dev)
                                stmmac_hw_fix_mac_speed(priv);
                                break;
                        default:
-                               if (netif_msg_link(priv))
-                                       netdev_warn(priv->dev,
-                                                   "Speed (%d) not 10/100\n",
-                                                   phydev->speed);
+                               netif_warn(priv, link, priv->dev,
+                                          "Speed (%d) not 10/100\n",
+                                          phydev->speed);
                                break;
                        }
 
@@ -1036,14 +1035,14 @@ static int init_dma_desc_rings(struct net_device *dev, gfp_t flags)
 
        priv->dma_buf_sz = bfsize;
 
-       if (netif_msg_probe(priv)) {
-               netdev_dbg(priv->dev, "(%s) dma_rx_phy=0x%08x dma_tx_phy=0x%08x\n",
-                          __func__, (u32)priv->dma_rx_phy,
-                          (u32)priv->dma_tx_phy);
+       netif_dbg(priv, probe, priv->dev,
+                 "(%s) dma_rx_phy=0x%08x dma_tx_phy=0x%08x\n",
+                 __func__, (u32)priv->dma_rx_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++) {
                struct dma_desc *p;
                if (priv->extend_desc)
@@ -1055,11 +1054,9 @@ static int init_dma_desc_rings(struct net_device *dev, gfp_t flags)
                if (ret)
                        goto err_init_rx_buffers;
 
-               if (netif_msg_probe(priv))
-                       netdev_dbg(priv->dev, "[%p]\t[%p]\t[%x]\n",
-                                  priv->rx_skbuff[i],
-                                priv->rx_skbuff[i]->data,
-                                (unsigned int)priv->rx_skbuff_dma[i]);
+               netif_dbg(priv, probe, priv->dev, "[%p]\t[%p]\t[%x]\n",
+                         priv->rx_skbuff[i], priv->rx_skbuff[i]->data,
+                         (unsigned int)priv->rx_skbuff_dma[i]);
        }
        priv->cur_rx = 0;
        priv->dirty_rx = (unsigned int)(i - DMA_RX_SIZE);
@@ -1389,9 +1386,8 @@ static void stmmac_tx_clean(struct stmmac_priv *priv)
                netif_tx_lock(priv->dev);
                if (netif_queue_stopped(priv->dev) &&
                    stmmac_tx_avail(priv) > STMMAC_TX_THRESH) {
-                       if (netif_msg_tx_done(priv))
-                               netdev_dbg(priv->dev, "%s: restart transmit\n",
-                                          __func__);
+                       netif_dbg(priv, tx_done, priv->dev,
+                                 "%s: restart transmit\n", __func__);
                        netif_wake_queue(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)
        priv->cur_tx = STMMAC_GET_ENTRY(priv->cur_tx, DMA_TX_SIZE);
 
        if (unlikely(stmmac_tx_avail(priv) <= (MAX_SKB_FRAGS + 1))) {
-               if (netif_msg_hw(priv))
-                       netdev_dbg(priv->dev, "%s: stop transmitted packets\n",
-                                  __func__);
+               netif_dbg(priv, hw, priv->dev, "%s: stop transmitted packets\n",
+                         __func__);
                netif_stop_queue(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 (netif_msg_hw(priv))
-                       netdev_dbg(priv->dev,
-                                  "%s: stop transmitted packets\n", __func__);
+               netif_dbg(priv, hw, priv->dev, "%s: stop transmitted packets\n",
+                         __func__);
                netif_stop_queue(dev);
        }
 
@@ -2465,9 +2459,8 @@ static inline void stmmac_rx_refill(struct stmmac_priv *priv)
                        if (priv->rx_zeroc_thresh > 0)
                                priv->rx_zeroc_thresh--;
 
-                       if (netif_msg_rx_status(priv))
-                               netdev_dbg(priv->dev,
-                                          "refill entry #%d\n", entry);
+                       netif_dbg(priv, rx_status, priv->dev,
+                                 "refill entry #%d\n", entry);
                }
                wmb();