Commit c041076a authored by Alexander Duyck's avatar Alexander Duyck Committed by David S. Miller

igb: update hw_debug macro to make use of netdev_dbg call

This change updates the igb driver to make use of the netdev_dbg function
macros now provided in netdevice.h

This is meant to be provided as an alternative to the patch provided by
Joe Perches.

It also removes igb_get_time_str since I found that it is unused code that
is no longer used even in debug.
Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 7e5ab157
...@@ -31,6 +31,7 @@ ...@@ -31,6 +31,7 @@
#include <linux/types.h> #include <linux/types.h>
#include <linux/delay.h> #include <linux/delay.h>
#include <linux/io.h> #include <linux/io.h>
#include <linux/netdevice.h>
#include "e1000_regs.h" #include "e1000_regs.h"
#include "e1000_defines.h" #include "e1000_defines.h"
...@@ -507,14 +508,11 @@ struct e1000_hw { ...@@ -507,14 +508,11 @@ struct e1000_hw {
u8 revision_id; u8 revision_id;
}; };
#ifdef DEBUG extern struct net_device *igb_get_hw_dev(struct e1000_hw *hw);
extern char *igb_get_hw_dev_name(struct e1000_hw *hw);
#define hw_dbg(format, arg...) \ #define hw_dbg(format, arg...) \
printk(KERN_DEBUG "%s: " format, igb_get_hw_dev_name(hw), ##arg) netdev_dbg(igb_get_hw_dev(hw), format, ##arg)
#else
#define hw_dbg(format, arg...)
#endif
#endif
/* These functions must be implemented by drivers */ /* These functions must be implemented by drivers */
s32 igb_read_pcie_cap_reg(struct e1000_hw *hw, u32 reg, u16 *value); s32 igb_read_pcie_cap_reg(struct e1000_hw *hw, u32 reg, u16 *value);
s32 igb_write_pcie_cap_reg(struct e1000_hw *hw, u32 reg, u16 *value); s32 igb_write_pcie_cap_reg(struct e1000_hw *hw, u32 reg, u16 *value);
#endif /* _E1000_HW_H_ */
...@@ -226,42 +226,16 @@ static cycle_t igb_read_clock(const struct cyclecounter *tc) ...@@ -226,42 +226,16 @@ static cycle_t igb_read_clock(const struct cyclecounter *tc)
return stamp; return stamp;
} }
#ifdef DEBUG
/** /**
* igb_get_hw_dev_name - return device name string * igb_get_hw_dev - return device
* used by hardware layer to print debugging information * used by hardware layer to print debugging information
**/ **/
char *igb_get_hw_dev_name(struct e1000_hw *hw) struct net_device *igb_get_hw_dev(struct e1000_hw *hw)
{ {
struct igb_adapter *adapter = hw->back; struct igb_adapter *adapter = hw->back;
return adapter->netdev->name; return adapter->netdev;
} }
/**
* igb_get_time_str - format current NIC and system time as string
*/
static char *igb_get_time_str(struct igb_adapter *adapter,
char buffer[160])
{
cycle_t hw = adapter->cycles.read(&adapter->cycles);
struct timespec nic = ns_to_timespec(timecounter_read(&adapter->clock));
struct timespec sys;
struct timespec delta;
getnstimeofday(&sys);
delta = timespec_sub(nic, sys);
sprintf(buffer,
"HW %llu, NIC %ld.%09lus, SYS %ld.%09lus, NIC-SYS %lds + %09luns",
hw,
(long)nic.tv_sec, nic.tv_nsec,
(long)sys.tv_sec, sys.tv_nsec,
(long)delta.tv_sec, delta.tv_nsec);
return buffer;
}
#endif
/** /**
* igb_init_module - Driver Registration Routine * igb_init_module - Driver Registration Routine
* *
......
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