diff options
author | Bruce Allan <bruce.w.allan@intel.com> | 2010-03-24 08:55:30 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-03-27 00:05:23 -0400 |
commit | 8544b9f7371ec6a7a5c0f8701ddde9e98f52a37e (patch) | |
tree | 36c2e615c3cb879602600f3a3e0f6beb37fdb15a /drivers/net | |
parent | 66aa4a55fe0548c8b13a195c61774db65c5896cd (diff) |
e1000e: Use pr_<level> and netdev_<level>
As an alternative to a quite large patch previously submitted by Joe
Perches to make use of kernel logging API, this patch is much less
intrusive.
Convert e_<level> to netdev_<level>
Use #define pr_fmt
Convert a few printks to pr_<level>
Cc: Joe Perches <joe@perches.com>
Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/e1000e/e1000.h | 19 | ||||
-rw-r--r-- | drivers/net/e1000e/netdev.c | 9 |
2 files changed, 10 insertions, 18 deletions
diff --git a/drivers/net/e1000e/e1000.h b/drivers/net/e1000e/e1000.h index 8da190b930a2..11e02e1f187c 100644 --- a/drivers/net/e1000e/e1000.h +++ b/drivers/net/e1000e/e1000.h | |||
@@ -42,25 +42,16 @@ | |||
42 | 42 | ||
43 | struct e1000_info; | 43 | struct e1000_info; |
44 | 44 | ||
45 | #define e_printk(level, adapter, format, arg...) \ | ||
46 | printk(level "%s: %s: " format, pci_name(adapter->pdev), \ | ||
47 | adapter->netdev->name, ## arg) | ||
48 | |||
49 | #ifdef DEBUG | ||
50 | #define e_dbg(format, arg...) \ | 45 | #define e_dbg(format, arg...) \ |
51 | e_printk(KERN_DEBUG , hw->adapter, format, ## arg) | 46 | netdev_dbg(hw->adapter->netdev, format, ## arg) |
52 | #else | ||
53 | #define e_dbg(format, arg...) do { (void)(hw); } while (0) | ||
54 | #endif | ||
55 | |||
56 | #define e_err(format, arg...) \ | 47 | #define e_err(format, arg...) \ |
57 | e_printk(KERN_ERR, adapter, format, ## arg) | 48 | netdev_err(adapter->netdev, format, ## arg) |
58 | #define e_info(format, arg...) \ | 49 | #define e_info(format, arg...) \ |
59 | e_printk(KERN_INFO, adapter, format, ## arg) | 50 | netdev_info(adapter->netdev, format, ## arg) |
60 | #define e_warn(format, arg...) \ | 51 | #define e_warn(format, arg...) \ |
61 | e_printk(KERN_WARNING, adapter, format, ## arg) | 52 | netdev_warn(adapter->netdev, format, ## arg) |
62 | #define e_notice(format, arg...) \ | 53 | #define e_notice(format, arg...) \ |
63 | e_printk(KERN_NOTICE, adapter, format, ## arg) | 54 | netdev_notice(adapter->netdev, format, ## arg) |
64 | 55 | ||
65 | 56 | ||
66 | /* Interrupt modes, as used by the IntMode parameter */ | 57 | /* Interrupt modes, as used by the IntMode parameter */ |
diff --git a/drivers/net/e1000e/netdev.c b/drivers/net/e1000e/netdev.c index 06ba46ae2983..5304959ae1f7 100644 --- a/drivers/net/e1000e/netdev.c +++ b/drivers/net/e1000e/netdev.c | |||
@@ -26,6 +26,8 @@ | |||
26 | 26 | ||
27 | *******************************************************************************/ | 27 | *******************************************************************************/ |
28 | 28 | ||
29 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | ||
30 | |||
29 | #include <linux/module.h> | 31 | #include <linux/module.h> |
30 | #include <linux/types.h> | 32 | #include <linux/types.h> |
31 | #include <linux/init.h> | 33 | #include <linux/init.h> |
@@ -5509,10 +5511,9 @@ static struct pci_driver e1000_driver = { | |||
5509 | static int __init e1000_init_module(void) | 5511 | static int __init e1000_init_module(void) |
5510 | { | 5512 | { |
5511 | int ret; | 5513 | int ret; |
5512 | printk(KERN_INFO "%s: Intel(R) PRO/1000 Network Driver - %s\n", | 5514 | pr_info("Intel(R) PRO/1000 Network Driver - %s\n", |
5513 | e1000e_driver_name, e1000e_driver_version); | 5515 | e1000e_driver_version); |
5514 | printk(KERN_INFO "%s: Copyright (c) 1999 - 2009 Intel Corporation.\n", | 5516 | pr_info("Copyright (c) 1999 - 2009 Intel Corporation.\n"); |
5515 | e1000e_driver_name); | ||
5516 | ret = pci_register_driver(&e1000_driver); | 5517 | ret = pci_register_driver(&e1000_driver); |
5517 | 5518 | ||
5518 | return ret; | 5519 | return ret; |