Skip to content

L
linux
Commit c9ea53d0 authored by Joe Perches's avatar Joe Perches Committed by David S. Miller
Browse files
Options

ipg: Use current logging styles 

Add and use pr_fmt, pr_<level> and netdev_<level>.
Convert printks with %[n].[n]x to %nx to be shorter
and clearer.
Consolidate printks to use a single printk rather
than continued printks without KERN_CONT.
Removed unnecessary trailing periods.
Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent c76ecb7a
Hide whitespace changes
Inline Side-by-side
Showing with 95 additions and 96 deletions
drivers/net/ipg.c
View file @ c9ea53d0
...@@ -20,6 +20,9 @@ ...@@ -20,6 +20,9 @@
* http://www.icplus.com.tw * http://www.icplus.com.tw
* jesse@icplus.com.tw * jesse@icplus.com.tw
*/ */
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
#include <linux/crc32.h> #include <linux/crc32.h>
#include <linux/ethtool.h> #include <linux/ethtool.h>
#include <linux/interrupt.h> #include <linux/interrupt.h>
...@@ -118,23 +121,23 @@ static void ipg_dump_rfdlist(struct net_device *dev) ...@@ -118,23 +121,23 @@ static void ipg_dump_rfdlist(struct net_device *dev)
IPG_DEBUG_MSG("_dump_rfdlist\n"); IPG_DEBUG_MSG("_dump_rfdlist\n");
printk(KERN_INFO "rx_current = %2.2x\n", sp->rx_current); netdev_info(dev, "rx_current = %02x\n", sp->rx_current);
printk(KERN_INFO "rx_dirty = %2.2x\n", sp->rx_dirty); netdev_info(dev, "rx_dirty = %02x\n", sp->rx_dirty);
printk(KERN_INFO "RFDList start address = %16.16lx\n", netdev_info(dev, "RFDList start address = %016lx\n",
(unsigned long) sp->rxd_map); (unsigned long)sp->rxd_map);
printk(KERN_INFO "RFDListPtr register = %8.8x%8.8x\n", netdev_info(dev, "RFDListPtr register = %08x%08x\n",
ipg_r32(IPG_RFDLISTPTR1), ipg_r32(IPG_RFDLISTPTR0)); ipg_r32(IPG_RFDLISTPTR1), ipg_r32(IPG_RFDLISTPTR0));
for (i = 0; i < IPG_RFDLIST_LENGTH; i++) { for (i = 0; i < IPG_RFDLIST_LENGTH; i++) {
offset = (u32) &sp->rxd[i].next_desc - (u32) sp->rxd; offset = (u32) &sp->rxd[i].next_desc - (u32) sp->rxd;
printk(KERN_INFO "%2.2x %4.4x RFDNextPtr = %16.16lx\n", i, netdev_info(dev, "%02x %04x RFDNextPtr = %016lx\n",
offset, (unsigned long) sp->rxd[i].next_desc); i, offset, (unsigned long)sp->rxd[i].next_desc);
offset = (u32) &sp->rxd[i].rfs - (u32) sp->rxd; offset = (u32) &sp->rxd[i].rfs - (u32) sp->rxd;
printk(KERN_INFO "%2.2x %4.4x RFS = %16.16lx\n", i, netdev_info(dev, "%02x %04x RFS = %016lx\n",
offset, (unsigned long) sp->rxd[i].rfs); i, offset, (unsigned long)sp->rxd[i].rfs);
offset = (u32) &sp->rxd[i].frag_info - (u32) sp->rxd; offset = (u32) &sp->rxd[i].frag_info - (u32) sp->rxd;
printk(KERN_INFO "%2.2x %4.4x frag_info = %16.16lx\n", i, netdev_info(dev, "%02x %04x frag_info = %016lx\n",
offset, (unsigned long) sp->rxd[i].frag_info); i, offset, (unsigned long)sp->rxd[i].frag_info);
} }
} }
...@@ -147,24 +150,24 @@ static void ipg_dump_tfdlist(struct net_device *dev) ...@@ -147,24 +150,24 @@ static void ipg_dump_tfdlist(struct net_device *dev)
IPG_DEBUG_MSG("_dump_tfdlist\n"); IPG_DEBUG_MSG("_dump_tfdlist\n");
printk(KERN_INFO "tx_current = %2.2x\n", sp->tx_current); netdev_info(dev, "tx_current = %02x\n", sp->tx_current);
printk(KERN_INFO "tx_dirty = %2.2x\n", sp->tx_dirty); netdev_info(dev, "tx_dirty = %02x\n", sp->tx_dirty);
printk(KERN_INFO "TFDList start address = %16.16lx\n", netdev_info(dev, "TFDList start address = %016lx\n",
(unsigned long) sp->txd_map); (unsigned long) sp->txd_map);
printk(KERN_INFO "TFDListPtr register = %8.8x%8.8x\n", netdev_info(dev, "TFDListPtr register = %08x%08x\n",
ipg_r32(IPG_TFDLISTPTR1), ipg_r32(IPG_TFDLISTPTR0)); ipg_r32(IPG_TFDLISTPTR1), ipg_r32(IPG_TFDLISTPTR0));
for (i = 0; i < IPG_TFDLIST_LENGTH; i++) { for (i = 0; i < IPG_TFDLIST_LENGTH; i++) {
offset = (u32) &sp->txd[i].next_desc - (u32) sp->txd; offset = (u32) &sp->txd[i].next_desc - (u32) sp->txd;
printk(KERN_INFO "%2.2x %4.4x TFDNextPtr = %16.16lx\n", i, netdev_info(dev, "%02x %04x TFDNextPtr = %016lx\n",
offset, (unsigned long) sp->txd[i].next_desc); i, offset, (unsigned long)sp->txd[i].next_desc);
offset = (u32) &sp->txd[i].tfc - (u32) sp->txd; offset = (u32) &sp->txd[i].tfc - (u32) sp->txd;
printk(KERN_INFO "%2.2x %4.4x TFC = %16.16lx\n", i, netdev_info(dev, "%02x %04x TFC = %016lx\n",
offset, (unsigned long) sp->txd[i].tfc); i, offset, (unsigned long) sp->txd[i].tfc);
offset = (u32) &sp->txd[i].frag_info - (u32) sp->txd; offset = (u32) &sp->txd[i].frag_info - (u32) sp->txd;
printk(KERN_INFO "%2.2x %4.4x frag_info = %16.16lx\n", i, netdev_info(dev, "%02x %04x frag_info = %016lx\n",
offset, (unsigned long) sp->txd[i].frag_info); i, offset, (unsigned long) sp->txd[i].frag_info);
} }
} }
#endif #endif
...@@ -480,6 +483,10 @@ static int ipg_config_autoneg(struct net_device *dev) ...@@ -480,6 +483,10 @@ static int ipg_config_autoneg(struct net_device *dev)
u32 mac_ctrl_val; u32 mac_ctrl_val;
u32 asicctrl; u32 asicctrl;
u8 phyctrl; u8 phyctrl;
const char *speed;
const char *duplex;
const char *tx_desc;
const char *rx_desc;
IPG_DEBUG_MSG("_config_autoneg\n"); IPG_DEBUG_MSG("_config_autoneg\n");
...@@ -499,27 +506,27 @@ static int ipg_config_autoneg(struct net_device *dev) ...@@ -499,27 +506,27 @@ static int ipg_config_autoneg(struct net_device *dev)
*/ */
sp->tenmbpsmode = 0; sp->tenmbpsmode = 0;
printk(KERN_INFO "%s: Link speed = ", dev->name);
/* Determine actual speed of operation. */ /* Determine actual speed of operation. */
switch (phyctrl & IPG_PC_LINK_SPEED) { switch (phyctrl & IPG_PC_LINK_SPEED) {
case IPG_PC_LINK_SPEED_10MBPS: case IPG_PC_LINK_SPEED_10MBPS:
printk("10Mbps.\n"); speed = "10Mbps";
printk(KERN_INFO "%s: 10Mbps operational mode enabled.\n",
dev->name);
sp->tenmbpsmode = 1; sp->tenmbpsmode = 1;
break; break;
case IPG_PC_LINK_SPEED_100MBPS: case IPG_PC_LINK_SPEED_100MBPS:
printk("100Mbps.\n"); speed = "100Mbps";
break; break;
case IPG_PC_LINK_SPEED_1000MBPS: case IPG_PC_LINK_SPEED_1000MBPS:
printk("1000Mbps.\n"); speed = "1000Mbps";
break; break;
default: default:
printk("undefined!\n"); speed = "undefined!";
return 0; return 0;
} }
netdev_info(dev, "Link speed = %s\n", speed);
if (sp->tenmbpsmode == 1)
netdev_info(dev, "10Mbps operational mode enabled\n");
if (phyctrl & IPG_PC_DUPLEX_STATUS) { if (phyctrl & IPG_PC_DUPLEX_STATUS) {
fullduplex = 1; fullduplex = 1;
txflowcontrol = 1; txflowcontrol = 1;
...@@ -528,38 +535,41 @@ static int ipg_config_autoneg(struct net_device *dev) ...@@ -528,38 +535,41 @@ static int ipg_config_autoneg(struct net_device *dev)
/* Configure full duplex, and flow control. */ /* Configure full duplex, and flow control. */
if (fullduplex == 1) { if (fullduplex == 1) {
/* Configure IPG for full duplex operation. */ /* Configure IPG for full duplex operation. */
printk(KERN_INFO "%s: setting full duplex, ", dev->name);
duplex = "full";
mac_ctrl_val |= IPG_MC_DUPLEX_SELECT_FD; mac_ctrl_val |= IPG_MC_DUPLEX_SELECT_FD;
if (txflowcontrol == 1) { if (txflowcontrol == 1) {
printk("TX flow control"); tx_desc = "";
mac_ctrl_val |= IPG_MC_TX_FLOW_CONTROL_ENABLE; mac_ctrl_val |= IPG_MC_TX_FLOW_CONTROL_ENABLE;
} else { } else {
printk("no TX flow control"); tx_desc = "no ";
mac_ctrl_val &= ~IPG_MC_TX_FLOW_CONTROL_ENABLE; mac_ctrl_val &= ~IPG_MC_TX_FLOW_CONTROL_ENABLE;
} }
if (rxflowcontrol == 1) { if (rxflowcontrol == 1) {
printk(", RX flow control."); rx_desc = "";
mac_ctrl_val |= IPG_MC_RX_FLOW_CONTROL_ENABLE; mac_ctrl_val |= IPG_MC_RX_FLOW_CONTROL_ENABLE;
} else { } else {
printk(", no RX flow control."); rx_desc = "no ";
mac_ctrl_val &= ~IPG_MC_RX_FLOW_CONTROL_ENABLE; mac_ctrl_val &= ~IPG_MC_RX_FLOW_CONTROL_ENABLE;
} }
printk("\n");
} else { } else {
/* Configure IPG for half duplex operation. */ duplex = "half";
printk(KERN_INFO "%s: setting half duplex, " tx_desc = "no ";
"no TX flow control, no RX flow control.\n", dev->name); rx_desc = "no ";
mac_ctrl_val &= (~IPG_MC_DUPLEX_SELECT_FD &
mac_ctrl_val &= ~IPG_MC_DUPLEX_SELECT_FD & ~IPG_MC_TX_FLOW_CONTROL_ENABLE &
~IPG_MC_TX_FLOW_CONTROL_ENABLE & ~IPG_MC_RX_FLOW_CONTROL_ENABLE);
~IPG_MC_RX_FLOW_CONTROL_ENABLE;
} }
netdev_info(dev, "setting %s duplex, %sTX, %sRX flow control\n",
duplex, tx_desc, rx_desc);
ipg_w32(mac_ctrl_val, MAC_CTRL); ipg_w32(mac_ctrl_val, MAC_CTRL);
return 0; return 0;
} }
...@@ -784,14 +794,13 @@ static int init_rfdlist(struct net_device *dev) ...@@ -784,14 +794,13 @@ static int init_rfdlist(struct net_device *dev)
* A receive buffer was not ready, break the * A receive buffer was not ready, break the
* RFD list here. * RFD list here.
*/ */
IPG_DEBUG_MSG("Cannot allocate Rx buffer.\n"); IPG_DEBUG_MSG("Cannot allocate Rx buffer\n");
/* Just in case we cannot allocate a single RFD. /* Just in case we cannot allocate a single RFD.
* Should not occur. * Should not occur.
*/ */
if (i == 0) { if (i == 0) {
printk(KERN_ERR "%s: No memory available" netdev_err(dev, "No memory available for RFD list\n");
" for RFD list.\n", dev->name);
return -ENOMEM; return -ENOMEM;
} }
} }
...@@ -838,7 +847,7 @@ static void init_tfdlist(struct net_device *dev) ...@@ -838,7 +847,7 @@ static void init_tfdlist(struct net_device *dev)
sp->tx_dirty = 0; sp->tx_dirty = 0;
/* Write the location of the TFDList to the IPG. */ /* Write the location of the TFDList to the IPG. */
IPG_DDEBUG_MSG("Starting TFDListPtr = %8.8x\n", IPG_DDEBUG_MSG("Starting TFDListPtr = %08x\n",
(u32) sp->txd_map); (u32) sp->txd_map);
ipg_w32((u32) sp->txd_map, TFD_LIST_PTR_0); ipg_w32((u32) sp->txd_map, TFD_LIST_PTR_0);
ipg_w32(0x00000000, TFD_LIST_PTR_1); ipg_w32(0x00000000, TFD_LIST_PTR_1);
...@@ -864,7 +873,7 @@ static void ipg_nic_txfree(struct net_device *dev) ...@@ -864,7 +873,7 @@ static void ipg_nic_txfree(struct net_device *dev)
struct sk_buff *skb = sp->tx_buff[dirty]; struct sk_buff *skb = sp->tx_buff[dirty];
struct ipg_tx *txfd = sp->txd + dirty; struct ipg_tx *txfd = sp->txd + dirty;
IPG_DEBUG_MSG("TFC = %16.16lx\n", (unsigned long) txfd->tfc); IPG_DEBUG_MSG("TFC = %016lx\n", (unsigned long) txfd->tfc);
/* Look at each TFD's TFC field beginning /* Look at each TFD's TFC field beginning
* at the last freed TFD up to the current TFD. * at the last freed TFD up to the current TFD.
...@@ -906,10 +915,8 @@ static void ipg_tx_timeout(struct net_device *dev) ...@@ -906,10 +915,8 @@ static void ipg_tx_timeout(struct net_device *dev)
spin_lock_irq(&sp->lock); spin_lock_irq(&sp->lock);
/* Re-configure after DMA reset. */ /* Re-configure after DMA reset. */
if (ipg_io_config(dev) < 0) { if (ipg_io_config(dev) < 0)
printk(KERN_INFO "%s: Error during re-configuration.\n", netdev_info(dev, "Error during re-configuration\n");
dev->name);
}
init_tfdlist(dev); init_tfdlist(dev);
...@@ -938,7 +945,7 @@ static void ipg_nic_txcleanup(struct net_device *dev) ...@@ -938,7 +945,7 @@ static void ipg_nic_txcleanup(struct net_device *dev)
*/ */
u32 txstatusdword = ipg_r32(TX_STATUS); u32 txstatusdword = ipg_r32(TX_STATUS);
IPG_DEBUG_MSG("TxStatus = %8.8x\n", txstatusdword); IPG_DEBUG_MSG("TxStatus = %08x\n", txstatusdword);
/* Check for Transmit errors. Error bits only valid if /* Check for Transmit errors. Error bits only valid if
* TX_COMPLETE bit in the TXSTATUS register is a 1. * TX_COMPLETE bit in the TXSTATUS register is a 1.
...@@ -953,20 +960,20 @@ static void ipg_nic_txcleanup(struct net_device *dev) ...@@ -953,20 +960,20 @@ static void ipg_nic_txcleanup(struct net_device *dev)
/* Transmit error, increment stat counters. */ /* Transmit error, increment stat counters. */
if (txstatusdword & IPG_TS_TX_ERROR) { if (txstatusdword & IPG_TS_TX_ERROR) {
IPG_DEBUG_MSG("Transmit error.\n"); IPG_DEBUG_MSG("Transmit error\n");
sp->stats.tx_errors++; sp->stats.tx_errors++;
} }
/* Late collision, re-enable transmitter. */ /* Late collision, re-enable transmitter. */
if (txstatusdword & IPG_TS_LATE_COLLISION) { if (txstatusdword & IPG_TS_LATE_COLLISION) {
IPG_DEBUG_MSG("Late collision on transmit.\n"); IPG_DEBUG_MSG("Late collision on transmit\n");
ipg_w32((ipg_r32(MAC_CTRL) | IPG_MC_TX_ENABLE) & ipg_w32((ipg_r32(MAC_CTRL) | IPG_MC_TX_ENABLE) &
IPG_MC_RSVD_MASK, MAC_CTRL); IPG_MC_RSVD_MASK, MAC_CTRL);
} }
/* Maximum collisions, re-enable transmitter. */ /* Maximum collisions, re-enable transmitter. */
if (txstatusdword & IPG_TS_TX_MAX_COLL) { if (txstatusdword & IPG_TS_TX_MAX_COLL) {
IPG_DEBUG_MSG("Maximum collisions on transmit.\n"); IPG_DEBUG_MSG("Maximum collisions on transmit\n");
ipg_w32((ipg_r32(MAC_CTRL) | IPG_MC_TX_ENABLE) & ipg_w32((ipg_r32(MAC_CTRL) | IPG_MC_TX_ENABLE) &
IPG_MC_RSVD_MASK, MAC_CTRL); IPG_MC_RSVD_MASK, MAC_CTRL);
} }
...@@ -975,16 +982,14 @@ static void ipg_nic_txcleanup(struct net_device *dev) ...@@ -975,16 +982,14 @@ static void ipg_nic_txcleanup(struct net_device *dev)
* transmitter. * transmitter.
*/ */
if (txstatusdword & IPG_TS_TX_UNDERRUN) { if (txstatusdword & IPG_TS_TX_UNDERRUN) {
IPG_DEBUG_MSG("Transmitter underrun.\n"); IPG_DEBUG_MSG("Transmitter underrun\n");
sp->stats.tx_fifo_errors++; sp->stats.tx_fifo_errors++;
ipg_reset(dev, IPG_AC_TX_RESET | IPG_AC_DMA | ipg_reset(dev, IPG_AC_TX_RESET | IPG_AC_DMA |
IPG_AC_NETWORK | IPG_AC_FIFO); IPG_AC_NETWORK | IPG_AC_FIFO);
/* Re-configure after DMA reset. */ /* Re-configure after DMA reset. */
if (ipg_io_config(dev) < 0) { if (ipg_io_config(dev) < 0) {
printk(KERN_INFO netdev_info(dev, "Error during re-configuration\n");
"%s: Error during re-configuration.\n",
dev->name);
} }
init_tfdlist(dev); init_tfdlist(dev);
...@@ -1063,7 +1068,7 @@ static int ipg_nic_rxrestore(struct net_device *dev) ...@@ -1063,7 +1068,7 @@ static int ipg_nic_rxrestore(struct net_device *dev)
* Linux system). * Linux system).
*/ */
if (ipg_get_rxbuff(dev, entry) < 0) { if (ipg_get_rxbuff(dev, entry) < 0) {
IPG_DEBUG_MSG("Cannot allocate new Rx buffer.\n"); IPG_DEBUG_MSG("Cannot allocate new Rx buffer\n");
break; break;
} }
...@@ -1134,7 +1139,7 @@ static int ipg_nic_rx_check_error(struct net_device *dev) ...@@ -1134,7 +1139,7 @@ static int ipg_nic_rx_check_error(struct net_device *dev)
(IPG_RFS_RXFIFOOVERRUN | IPG_RFS_RXRUNTFRAME | (IPG_RFS_RXFIFOOVERRUN | IPG_RFS_RXRUNTFRAME |
IPG_RFS_RXALIGNMENTERROR | IPG_RFS_RXFCSERROR | IPG_RFS_RXALIGNMENTERROR | IPG_RFS_RXFCSERROR |
IPG_RFS_RXOVERSIZEDFRAME | IPG_RFS_RXLENGTHERROR))) { IPG_RFS_RXOVERSIZEDFRAME | IPG_RFS_RXLENGTHERROR))) {
IPG_DEBUG_MSG("Rx error, RFS = %16.16lx\n", IPG_DEBUG_MSG("Rx error, RFS = %016lx\n",
(unsigned long) rxfd->rfs); (unsigned long) rxfd->rfs);
/* Increment general receive error statistic. */ /* Increment general receive error statistic. */
...@@ -1142,13 +1147,13 @@ static int ipg_nic_rx_check_error(struct net_device *dev) ...@@ -1142,13 +1147,13 @@ static int ipg_nic_rx_check_error(struct net_device *dev)
/* Increment detailed receive error statistics. */ /* Increment detailed receive error statistics. */
if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXFIFOOVERRUN) { if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXFIFOOVERRUN) {
IPG_DEBUG_MSG("RX FIFO overrun occurred.\n"); IPG_DEBUG_MSG("RX FIFO overrun occurred\n");
sp->stats.rx_fifo_errors++; sp->stats.rx_fifo_errors++;
} }
if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXRUNTFRAME) { if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXRUNTFRAME) {
IPG_DEBUG_MSG("RX runt occurred.\n"); IPG_DEBUG_MSG("RX runt occurred\n");
sp->stats.rx_length_errors++; sp->stats.rx_length_errors++;
} }
...@@ -1157,7 +1162,7 @@ static int ipg_nic_rx_check_error(struct net_device *dev) ...@@ -1157,7 +1162,7 @@ static int ipg_nic_rx_check_error(struct net_device *dev)
*/ */
if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXALIGNMENTERROR) { if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXALIGNMENTERROR) {
IPG_DEBUG_MSG("RX alignment error occurred.\n"); IPG_DEBUG_MSG("RX alignment error occurred\n");
sp->stats.rx_frame_errors++; sp->stats.rx_frame_errors++;
} }
...@@ -1404,7 +1409,7 @@ static int ipg_nic_rx(struct net_device *dev) ...@@ -1404,7 +1409,7 @@ static int ipg_nic_rx(struct net_device *dev)
*/ */
if (framelen > sp->rxfrag_size) { if (framelen > sp->rxfrag_size) {
IPG_DEBUG_MSG IPG_DEBUG_MSG
("RFS FrameLen > allocated fragment size.\n"); ("RFS FrameLen > allocated fragment size\n");
framelen = sp->rxfrag_size; framelen = sp->rxfrag_size;
} }
...@@ -1414,7 +1419,7 @@ static int ipg_nic_rx(struct net_device *dev) ...@@ -1414,7 +1419,7 @@ static int ipg_nic_rx(struct net_device *dev)
IPG_RFS_RXALIGNMENTERROR | IPG_RFS_RXFCSERROR | IPG_RFS_RXALIGNMENTERROR | IPG_RFS_RXFCSERROR |
IPG_RFS_RXOVERSIZEDFRAME | IPG_RFS_RXLENGTHERROR)))) { IPG_RFS_RXOVERSIZEDFRAME | IPG_RFS_RXLENGTHERROR)))) {
IPG_DEBUG_MSG("Rx error, RFS = %16.16lx\n", IPG_DEBUG_MSG("Rx error, RFS = %016lx\n",
(unsigned long int) rxfd->rfs); (unsigned long int) rxfd->rfs);
/* Increment general receive error statistic. */ /* Increment general receive error statistic. */
...@@ -1422,12 +1427,12 @@ static int ipg_nic_rx(struct net_device *dev) ...@@ -1422,12 +1427,12 @@ static int ipg_nic_rx(struct net_device *dev)
/* Increment detailed receive error statistics. */ /* Increment detailed receive error statistics. */
if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXFIFOOVERRUN) { if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXFIFOOVERRUN) {
IPG_DEBUG_MSG("RX FIFO overrun occurred.\n"); IPG_DEBUG_MSG("RX FIFO overrun occurred\n");
sp->stats.rx_fifo_errors++; sp->stats.rx_fifo_errors++;
} }
if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXRUNTFRAME) { if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXRUNTFRAME) {
IPG_DEBUG_MSG("RX runt occurred.\n"); IPG_DEBUG_MSG("RX runt occurred\n");
sp->stats.rx_length_errors++; sp->stats.rx_length_errors++;
} }
...@@ -1437,7 +1442,7 @@ static int ipg_nic_rx(struct net_device *dev) ...@@ -1437,7 +1442,7 @@ static int ipg_nic_rx(struct net_device *dev)
*/ */
if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXALIGNMENTERROR) { if (le64_to_cpu(rxfd->rfs) & IPG_RFS_RXALIGNMENTERROR) {
IPG_DEBUG_MSG("RX alignment error occurred.\n"); IPG_DEBUG_MSG("RX alignment error occurred\n");
sp->stats.rx_frame_errors++; sp->stats.rx_frame_errors++;
} }
...@@ -1508,7 +1513,7 @@ static int ipg_nic_rx(struct net_device *dev) ...@@ -1508,7 +1513,7 @@ static int ipg_nic_rx(struct net_device *dev)
rxfd = sp->rxd + entry; rxfd = sp->rxd + entry;
IPG_DEBUG_MSG("Frame requires multiple RFDs.\n"); IPG_DEBUG_MSG("Frame requires multiple RFDs\n");
/* An unexpected event, additional code needed to handle /* An unexpected event, additional code needed to handle
* properly. So for the time being, just disregard the * properly. So for the time being, just disregard the
...@@ -1557,8 +1562,7 @@ static void ipg_reset_after_host_error(struct work_struct *work) ...@@ -1557,8 +1562,7 @@ static void ipg_reset_after_host_error(struct work_struct *work)
init_tfdlist(dev); init_tfdlist(dev);
if (ipg_io_config(dev) < 0) { if (ipg_io_config(dev) < 0) {
printk(KERN_INFO "%s: Cannot recover from PCI error.\n", netdev_info(dev, "Cannot recover from PCI error\n");
dev->name);
schedule_delayed_work(&sp->task, HZ); schedule_delayed_work(&sp->task, HZ);
} }
} }
...@@ -1586,7 +1590,7 @@ static irqreturn_t ipg_interrupt_handler(int irq, void *dev_inst) ...@@ -1586,7 +1590,7 @@ static irqreturn_t ipg_interrupt_handler(int irq, void *dev_inst)
*/ */
status = ipg_r16(INT_STATUS_ACK); status = ipg_r16(INT_STATUS_ACK);
IPG_DEBUG_MSG("IntStatusAck = %4.4x\n", status); IPG_DEBUG_MSG("IntStatusAck = %04x\n", status);
/* Shared IRQ of remove event. */ /* Shared IRQ of remove event. */
if (!(status & IPG_IS_RSVD_MASK)) if (!(status & IPG_IS_RSVD_MASK))
...@@ -1599,7 +1603,7 @@ static irqreturn_t ipg_interrupt_handler(int irq, void *dev_inst) ...@@ -1599,7 +1603,7 @@ static irqreturn_t ipg_interrupt_handler(int irq, void *dev_inst)
/* If RFDListEnd interrupt, restore all used RFDs. */ /* If RFDListEnd interrupt, restore all used RFDs. */
if (status & IPG_IS_RFD_LIST_END) { if (status & IPG_IS_RFD_LIST_END) {
IPG_DEBUG_MSG("RFDListEnd Interrupt.\n"); IPG_DEBUG_MSG("RFDListEnd Interrupt\n");
/* The RFD list end indicates an RFD was encountered /* The RFD list end indicates an RFD was encountered
* with a 0 NextPtr, or with an RFDDone bit set to 1 * with a 0 NextPtr, or with an RFDDone bit set to 1
...@@ -1661,21 +1665,20 @@ static irqreturn_t ipg_interrupt_handler(int irq, void *dev_inst) ...@@ -1661,21 +1665,20 @@ static irqreturn_t ipg_interrupt_handler(int irq, void *dev_inst)
/* If LinkEvent interrupt, resolve autonegotiation. */ /* If LinkEvent interrupt, resolve autonegotiation. */
if (status & IPG_IS_LINK_EVENT) { if (status & IPG_IS_LINK_EVENT) {
if (ipg_config_autoneg(dev) < 0) if (ipg_config_autoneg(dev) < 0)
printk(KERN_INFO "%s: Auto-negotiation error.\n", netdev_info(dev, "Auto-negotiation error\n");
dev->name);
} }
/* If MACCtrlFrame interrupt, do nothing. */ /* If MACCtrlFrame interrupt, do nothing. */
if (status & IPG_IS_MAC_CTRL_FRAME) if (status & IPG_IS_MAC_CTRL_FRAME)
IPG_DEBUG_MSG("MACCtrlFrame interrupt.\n"); IPG_DEBUG_MSG("MACCtrlFrame interrupt\n");
/* If RxComplete interrupt, do nothing. */ /* If RxComplete interrupt, do nothing. */
if (status & IPG_IS_RX_COMPLETE) if (status & IPG_IS_RX_COMPLETE)
IPG_DEBUG_MSG("RxComplete interrupt.\n"); IPG_DEBUG_MSG("RxComplete interrupt\n");
/* If RxEarly interrupt, do nothing. */ /* If RxEarly interrupt, do nothing. */
if (status & IPG_IS_RX_EARLY) if (status & IPG_IS_RX_EARLY)
IPG_DEBUG_MSG("RxEarly interrupt.\n"); IPG_DEBUG_MSG("RxEarly interrupt\n");
out_enable: out_enable:
/* Re-enable IPG interrupts. */ /* Re-enable IPG interrupts. */
...@@ -1749,8 +1752,7 @@ static int ipg_nic_open(struct net_device *dev) ...@@ -1749,8 +1752,7 @@ static int ipg_nic_open(struct net_device *dev)
rc = request_irq(pdev->irq, ipg_interrupt_handler, IRQF_SHARED, rc = request_irq(pdev->irq, ipg_interrupt_handler, IRQF_SHARED,
dev->name, dev); dev->name, dev);
if (rc < 0) { if (rc < 0) {
printk(KERN_INFO "%s: Error when requesting interrupt.\n", netdev_info(dev, "Error when requesting interrupt\n");
dev->name);
goto out; goto out;
} }
...@@ -1770,8 +1772,7 @@ static int ipg_nic_open(struct net_device *dev) ...@@ -1770,8 +1772,7 @@ static int ipg_nic_open(struct net_device *dev)
rc = init_rfdlist(dev); rc = init_rfdlist(dev);
if (rc < 0) { if (rc < 0) {
printk(KERN_INFO "%s: Error during configuration.\n", netdev_info(dev, "Error during configuration\n");
dev->name);
goto err_free_tx_2; goto err_free_tx_2;
} }
...@@ -1779,14 +1780,13 @@ static int ipg_nic_open(struct net_device *dev) ...@@ -1779,14 +1780,13 @@ static int ipg_nic_open(struct net_device *dev)
rc = ipg_io_config(dev); rc = ipg_io_config(dev);
if (rc < 0) { if (rc < 0) {
printk(KERN_INFO "%s: Error during configuration.\n", netdev_info(dev, "Error during configuration\n");
dev->name);
goto err_release_tfdlist_3; goto err_release_tfdlist_3;
} }
/* Resolve autonegotiation. */ /* Resolve autonegotiation. */
if (ipg_config_autoneg(dev) < 0) if (ipg_config_autoneg(dev) < 0)
printk(KERN_INFO "%s: Auto-negotiation error.\n", dev->name); netdev_info(dev, "Auto-negotiation error\n");
/* initialize JUMBO Frame control variable */ /* initialize JUMBO Frame control variable */
sp->jumbo.found_start = 0; sp->jumbo.found_start = 0;
...@@ -2222,7 +2222,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev, ...@@ -2222,7 +2222,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev,
if (rc < 0) if (rc < 0)
goto out; goto out;
printk(KERN_INFO "%s: %s\n", pci_name(pdev), ipg_brand_name[i]); pr_info("%s: %s\n", pci_name(pdev), ipg_brand_name[i]);
pci_set_master(pdev); pci_set_master(pdev);
...@@ -2230,8 +2230,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev, ...@@ -2230,8 +2230,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev,
if (rc < 0) { if (rc < 0) {
rc = pci_set_dma_mask(pdev, DMA_BIT_MASK(32)); rc = pci_set_dma_mask(pdev, DMA_BIT_MASK(32));
if (rc < 0) { if (rc < 0) {
printk(KERN_ERR "%s: DMA config failed.\n", pr_err("%s: DMA config failed\n", pci_name(pdev));
pci_name(pdev));
goto err_disable_0; goto err_disable_0;
} }
} }
...@@ -2241,7 +2240,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev, ...@@ -2241,7 +2240,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev,
*/ */
dev = alloc_etherdev(sizeof(struct ipg_nic_private)); dev = alloc_etherdev(sizeof(struct ipg_nic_private));
if (!dev) { if (!dev) {
printk(KERN_ERR "%s: alloc_etherdev failed\n", pci_name(pdev)); pr_err("%s: alloc_etherdev failed\n", pci_name(pdev));
rc = -ENOMEM; rc = -ENOMEM;
goto err_disable_0; goto err_disable_0;
} }
...@@ -2267,7 +2266,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev, ...@@ -2267,7 +2266,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev,
ioaddr = pci_iomap(pdev, 1, pci_resource_len(pdev, 1)); ioaddr = pci_iomap(pdev, 1, pci_resource_len(pdev, 1));
if (!ioaddr) { if (!ioaddr) {
printk(KERN_ERR "%s cannot map MMIO\n", pci_name(pdev)); pr_err("%s: cannot map MMIO\n", pci_name(pdev));
rc = -EIO; rc = -EIO;
goto err_release_regions_2; goto err_release_regions_2;
} }
...@@ -2289,7 +2288,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev, ...@@ -2289,7 +2288,7 @@ static int __devinit ipg_probe(struct pci_dev *pdev,
if (rc < 0) if (rc < 0)
goto err_unmap_3; goto err_unmap_3;
printk(KERN_INFO "Ethernet device registered as: %s\n", dev->name); netdev_info(dev, "Ethernet device registered\n");
out: out:
return rc; return rc;
......
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
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7