gianfar: Use netdev_<level> when possible
authorJoe Perches <joe@perches.com>
Sat, 13 Apr 2013 19:03:18 +0000 (19:03 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 14 Apr 2013 19:41:49 +0000 (15:41 -0400)
Use a more current logging style.

Convert pr_<level> to netdev_<level> when a struct net_device is
available.  Add pr_fmt and neaten other formats too.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/gianfar_ethtool.c
drivers/net/ethernet/freescale/gianfar_ptp.c
drivers/net/ethernet/freescale/gianfar_sysfs.c

index 4e7118f9f0750cedf7630db54e726388f07be1c2..083603f6bec002596ddf4072798a8df51db4e21b 100644 (file)
@@ -389,14 +389,14 @@ static int gfar_scoalesce(struct net_device *dev,
 
        /* Check the bounds of the values */
        if (cvals->rx_coalesce_usecs > GFAR_MAX_COAL_USECS) {
-               pr_info("Coalescing is limited to %d microseconds\n",
-                       GFAR_MAX_COAL_USECS);
+               netdev_info(dev, "Coalescing is limited to %d microseconds\n",
+                           GFAR_MAX_COAL_USECS);
                return -EINVAL;
        }
 
        if (cvals->rx_max_coalesced_frames > GFAR_MAX_COAL_FRAMES) {
-               pr_info("Coalescing is limited to %d frames\n",
-                       GFAR_MAX_COAL_FRAMES);
+               netdev_info(dev, "Coalescing is limited to %d frames\n",
+                           GFAR_MAX_COAL_FRAMES);
                return -EINVAL;
        }
 
@@ -418,14 +418,14 @@ static int gfar_scoalesce(struct net_device *dev,
 
        /* Check the bounds of the values */
        if (cvals->tx_coalesce_usecs > GFAR_MAX_COAL_USECS) {
-               pr_info("Coalescing is limited to %d microseconds\n",
-                       GFAR_MAX_COAL_USECS);
+               netdev_info(dev, "Coalescing is limited to %d microseconds\n",
+                           GFAR_MAX_COAL_USECS);
                return -EINVAL;
        }
 
        if (cvals->tx_max_coalesced_frames > GFAR_MAX_COAL_FRAMES) {
-               pr_info("Coalescing is limited to %d frames\n",
-                       GFAR_MAX_COAL_FRAMES);
+               netdev_info(dev, "Coalescing is limited to %d frames\n",
+                           GFAR_MAX_COAL_FRAMES);
                return -EINVAL;
        }
 
@@ -735,7 +735,8 @@ static int gfar_ethflow_to_filer_table(struct gfar_private *priv, u64 ethflow,
                cmp_rqfpr = RQFPR_IPV6 |RQFPR_UDP;
                break;
        default:
-               pr_err("Right now this class is not supported\n");
+               netdev_err(priv->ndev,
+                          "Right now this class is not supported\n");
                ret = 0;
                goto err;
        }
@@ -751,7 +752,8 @@ static int gfar_ethflow_to_filer_table(struct gfar_private *priv, u64 ethflow,
        }
 
        if (i == MAX_FILER_IDX + 1) {
-               pr_err("No parse rule found, can't create hash rules\n");
+               netdev_err(priv->ndev,
+                          "No parse rule found, can't create hash rules\n");
                ret = 0;
                goto err;
        }
@@ -1568,7 +1570,7 @@ static int gfar_process_filer_changes(struct gfar_private *priv)
        gfar_cluster_filer(tab);
        gfar_optimize_filer_masks(tab);
 
-       pr_debug("\n\tSummary:\n"
+       pr_debug("\tSummary:\n"
                 "\tData on hardware: %d\n"
                 "\tCompression rate: %d%%\n",
                 tab->index, 100 - (100 * tab->index) / i);
index 2e5daee0438a55891072595400d643bd8ad1340b..fe8e9e5cfb2b720728bf40faf39d56978e01c85b 100644 (file)
@@ -17,6 +17,9 @@
  *  along with this program; if not, write to the Free Software
  *  Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  */
+
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+
 #include <linux/device.h>
 #include <linux/hrtimer.h>
 #include <linux/init.h>
index cd14a4d449c2b6cfee53e24f63e8499582a70a33..acb55af7e3f36cbb8a4f3ce7f1220efc90f5ac01 100644 (file)
@@ -337,5 +337,5 @@ void gfar_init_sysfs(struct net_device *dev)
        rc |= device_create_file(&dev->dev, &dev_attr_fifo_starve);
        rc |= device_create_file(&dev->dev, &dev_attr_fifo_starve_off);
        if (rc)
-               dev_err(&dev->dev, "Error creating gianfar sysfs files.\n");
+               dev_err(&dev->dev, "Error creating gianfar sysfs files\n");
 }