ethernet: amd: use pr_info_once()
authorVarka Bhadram <varkabhadram@gmail.com>
Thu, 11 Sep 2014 07:20:50 +0000 (12:50 +0530)
committerDavid S. Miller <davem@davemloft.net>
Fri, 12 Sep 2014 22:02:41 +0000 (18:02 -0400)
It will use pr_info_one() to print the version info of the
driver in probe function only once. No need to use the static
variable here.

Signed-off-by: Varka Bhadram <varkab@cdac.in>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/amd/au1000_eth.c

index 31c48a7..6c323f4 100644 (file)
@@ -1140,7 +1140,6 @@ static const struct net_device_ops au1000_netdev_ops = {
 
 static int au1000_probe(struct platform_device *pdev)
 {
-       static unsigned version_printed;
        struct au1000_private *aup = NULL;
        struct au1000_eth_platform_data *pd;
        struct net_device *dev = NULL;
@@ -1371,9 +1370,8 @@ static int au1000_probe(struct platform_device *pdev)
 
        netdev_info(dev, "Au1xx0 Ethernet found at 0x%lx, irq %d\n",
                        (unsigned long)base->start, irq);
-       if (version_printed++ == 0)
-               pr_info("%s version %s %s\n",
-                                       DRV_NAME, DRV_VERSION, DRV_AUTHOR);
+
+       pr_info_once("%s version %s %s\n", DRV_NAME, DRV_VERSION, DRV_AUTHOR);
 
        return 0;