Commit 6b80858d authored by Joe Perches's avatar Joe Perches Committed by David S. Miller

forcedeth: convert dprintk(KERN_DEBUG to netdev_dbg

Use the more standard macro to preface netdev->name.
Fix casting on resource use.
Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 344d0dce
...@@ -1139,22 +1139,22 @@ static int mii_rw(struct net_device *dev, int addr, int miireg, int value) ...@@ -1139,22 +1139,22 @@ static int mii_rw(struct net_device *dev, int addr, int miireg, int value)
if (reg_delay(dev, NvRegMIIControl, NVREG_MIICTL_INUSE, 0, if (reg_delay(dev, NvRegMIIControl, NVREG_MIICTL_INUSE, 0,
NV_MIIPHY_DELAY, NV_MIIPHY_DELAYMAX)) { NV_MIIPHY_DELAY, NV_MIIPHY_DELAYMAX)) {
dprintk(KERN_DEBUG "%s: mii_rw of reg %d at PHY %d timed out.\n", netdev_dbg(dev, "mii_rw of reg %d at PHY %d timed out\n",
dev->name, miireg, addr); miireg, addr);
retval = -1; retval = -1;
} else if (value != MII_READ) { } else if (value != MII_READ) {
/* it was a write operation - fewer failures are detectable */ /* it was a write operation - fewer failures are detectable */
dprintk(KERN_DEBUG "%s: mii_rw wrote 0x%x to reg %d at PHY %d\n", netdev_dbg(dev, "mii_rw wrote 0x%x to reg %d at PHY %d\n",
dev->name, value, miireg, addr); value, miireg, addr);
retval = 0; retval = 0;
} else if (readl(base + NvRegMIIStatus) & NVREG_MIISTAT_ERROR) { } else if (readl(base + NvRegMIIStatus) & NVREG_MIISTAT_ERROR) {
dprintk(KERN_DEBUG "%s: mii_rw of reg %d at PHY %d failed.\n", netdev_dbg(dev, "mii_rw of reg %d at PHY %d failed\n",
dev->name, miireg, addr); miireg, addr);
retval = -1; retval = -1;
} else { } else {
retval = readl(base + NvRegMIIData); retval = readl(base + NvRegMIIData);
dprintk(KERN_DEBUG "%s: mii_rw read from reg %d at PHY %d: 0x%x.\n", netdev_dbg(dev, "mii_rw read from reg %d at PHY %d: 0x%x\n",
dev->name, miireg, addr, retval); miireg, addr, retval);
} }
return retval; return retval;
...@@ -1506,7 +1506,7 @@ static void nv_start_rx(struct net_device *dev) ...@@ -1506,7 +1506,7 @@ static void nv_start_rx(struct net_device *dev)
u8 __iomem *base = get_hwbase(dev); u8 __iomem *base = get_hwbase(dev);
u32 rx_ctrl = readl(base + NvRegReceiverControl); u32 rx_ctrl = readl(base + NvRegReceiverControl);
dprintk(KERN_DEBUG "%s: nv_start_rx\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
/* Already running? Stop it. */ /* Already running? Stop it. */
if ((readl(base + NvRegReceiverControl) & NVREG_RCVCTL_START) && !np->mac_in_use) { if ((readl(base + NvRegReceiverControl) & NVREG_RCVCTL_START) && !np->mac_in_use) {
rx_ctrl &= ~NVREG_RCVCTL_START; rx_ctrl &= ~NVREG_RCVCTL_START;
...@@ -1519,8 +1519,8 @@ static void nv_start_rx(struct net_device *dev) ...@@ -1519,8 +1519,8 @@ static void nv_start_rx(struct net_device *dev)
if (np->mac_in_use) if (np->mac_in_use)
rx_ctrl &= ~NVREG_RCVCTL_RX_PATH_EN; rx_ctrl &= ~NVREG_RCVCTL_RX_PATH_EN;
writel(rx_ctrl, base + NvRegReceiverControl); writel(rx_ctrl, base + NvRegReceiverControl);
dprintk(KERN_DEBUG "%s: nv_start_rx to duplex %d, speed 0x%08x.\n", netdev_dbg(dev, "%s: duplex %d, speed 0x%08x\n",
dev->name, np->duplex, np->linkspeed); __func__, np->duplex, np->linkspeed);
pci_push(base); pci_push(base);
} }
...@@ -1530,7 +1530,7 @@ static void nv_stop_rx(struct net_device *dev) ...@@ -1530,7 +1530,7 @@ static void nv_stop_rx(struct net_device *dev)
u8 __iomem *base = get_hwbase(dev); u8 __iomem *base = get_hwbase(dev);
u32 rx_ctrl = readl(base + NvRegReceiverControl); u32 rx_ctrl = readl(base + NvRegReceiverControl);
dprintk(KERN_DEBUG "%s: nv_stop_rx\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
if (!np->mac_in_use) if (!np->mac_in_use)
rx_ctrl &= ~NVREG_RCVCTL_START; rx_ctrl &= ~NVREG_RCVCTL_START;
else else
...@@ -1551,7 +1551,7 @@ static void nv_start_tx(struct net_device *dev) ...@@ -1551,7 +1551,7 @@ static void nv_start_tx(struct net_device *dev)
u8 __iomem *base = get_hwbase(dev); u8 __iomem *base = get_hwbase(dev);
u32 tx_ctrl = readl(base + NvRegTransmitterControl); u32 tx_ctrl = readl(base + NvRegTransmitterControl);
dprintk(KERN_DEBUG "%s: nv_start_tx\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
tx_ctrl |= NVREG_XMITCTL_START; tx_ctrl |= NVREG_XMITCTL_START;
if (np->mac_in_use) if (np->mac_in_use)
tx_ctrl &= ~NVREG_XMITCTL_TX_PATH_EN; tx_ctrl &= ~NVREG_XMITCTL_TX_PATH_EN;
...@@ -1565,7 +1565,7 @@ static void nv_stop_tx(struct net_device *dev) ...@@ -1565,7 +1565,7 @@ static void nv_stop_tx(struct net_device *dev)
u8 __iomem *base = get_hwbase(dev); u8 __iomem *base = get_hwbase(dev);
u32 tx_ctrl = readl(base + NvRegTransmitterControl); u32 tx_ctrl = readl(base + NvRegTransmitterControl);
dprintk(KERN_DEBUG "%s: nv_stop_tx\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
if (!np->mac_in_use) if (!np->mac_in_use)
tx_ctrl &= ~NVREG_XMITCTL_START; tx_ctrl &= ~NVREG_XMITCTL_START;
else else
...@@ -1598,7 +1598,7 @@ static void nv_txrx_reset(struct net_device *dev) ...@@ -1598,7 +1598,7 @@ static void nv_txrx_reset(struct net_device *dev)
struct fe_priv *np = netdev_priv(dev); struct fe_priv *np = netdev_priv(dev);
u8 __iomem *base = get_hwbase(dev); u8 __iomem *base = get_hwbase(dev);
dprintk(KERN_DEBUG "%s: nv_txrx_reset\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
writel(NVREG_TXRXCTL_BIT2 | NVREG_TXRXCTL_RESET | np->txrxctl_bits, base + NvRegTxRxControl); writel(NVREG_TXRXCTL_BIT2 | NVREG_TXRXCTL_RESET | np->txrxctl_bits, base + NvRegTxRxControl);
pci_push(base); pci_push(base);
udelay(NV_TXRX_RESET_DELAY); udelay(NV_TXRX_RESET_DELAY);
...@@ -1612,7 +1612,7 @@ static void nv_mac_reset(struct net_device *dev) ...@@ -1612,7 +1612,7 @@ static void nv_mac_reset(struct net_device *dev)
u8 __iomem *base = get_hwbase(dev); u8 __iomem *base = get_hwbase(dev);
u32 temp1, temp2, temp3; u32 temp1, temp2, temp3;
dprintk(KERN_DEBUG "%s: nv_mac_reset\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
writel(NVREG_TXRXCTL_BIT2 | NVREG_TXRXCTL_RESET | np->txrxctl_bits, base + NvRegTxRxControl); writel(NVREG_TXRXCTL_BIT2 | NVREG_TXRXCTL_RESET | np->txrxctl_bits, base + NvRegTxRxControl);
pci_push(base); pci_push(base);
...@@ -2190,8 +2190,8 @@ static netdev_tx_t nv_start_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -2190,8 +2190,8 @@ static netdev_tx_t nv_start_xmit(struct sk_buff *skb, struct net_device *dev)
spin_unlock_irqrestore(&np->lock, flags); spin_unlock_irqrestore(&np->lock, flags);
dprintk(KERN_DEBUG "%s: nv_start_xmit: entries %d queued for transmission. tx_flags_extra: %x\n", netdev_dbg(dev, "%s: entries %d queued for transmission. tx_flags_extra: %x\n",
dev->name, entries, tx_flags_extra); __func__, entries, tx_flags_extra);
{ {
int j; int j;
for (j = 0; j < 64; j++) { for (j = 0; j < 64; j++) {
...@@ -2341,8 +2341,8 @@ static netdev_tx_t nv_start_xmit_optimized(struct sk_buff *skb, ...@@ -2341,8 +2341,8 @@ static netdev_tx_t nv_start_xmit_optimized(struct sk_buff *skb,
spin_unlock_irqrestore(&np->lock, flags); spin_unlock_irqrestore(&np->lock, flags);
dprintk(KERN_DEBUG "%s: nv_start_xmit_optimized: entries %d queued for transmission. tx_flags_extra: %x\n", netdev_dbg(dev, "%s: entries %d queued for transmission. tx_flags_extra: %x\n",
dev->name, entries, tx_flags_extra); __func__, entries, tx_flags_extra);
{ {
int j; int j;
for (j = 0; j < 64; j++) { for (j = 0; j < 64; j++) {
...@@ -2391,8 +2391,7 @@ static int nv_tx_done(struct net_device *dev, int limit) ...@@ -2391,8 +2391,7 @@ static int nv_tx_done(struct net_device *dev, int limit)
!((flags = le32_to_cpu(np->get_tx.orig->flaglen)) & NV_TX_VALID) && !((flags = le32_to_cpu(np->get_tx.orig->flaglen)) & NV_TX_VALID) &&
(tx_work < limit)) { (tx_work < limit)) {
dprintk(KERN_DEBUG "%s: nv_tx_done: flags 0x%x.\n", netdev_dbg(dev, "%s: flags 0x%x\n", __func__, flags);
dev->name, flags);
nv_unmap_txskb(np, np->get_tx_ctx); nv_unmap_txskb(np, np->get_tx_ctx);
...@@ -2456,8 +2455,7 @@ static int nv_tx_done_optimized(struct net_device *dev, int limit) ...@@ -2456,8 +2455,7 @@ static int nv_tx_done_optimized(struct net_device *dev, int limit)
!((flags = le32_to_cpu(np->get_tx.ex->flaglen)) & NV_TX2_VALID) && !((flags = le32_to_cpu(np->get_tx.ex->flaglen)) & NV_TX2_VALID) &&
(tx_work < limit)) { (tx_work < limit)) {
dprintk(KERN_DEBUG "%s: nv_tx_done_optimized: flags 0x%x.\n", netdev_dbg(dev, "%s: flags 0x%x\n", __func__, flags);
dev->name, flags);
nv_unmap_txskb(np, np->get_tx_ctx); nv_unmap_txskb(np, np->get_tx_ctx);
...@@ -2608,8 +2606,8 @@ static int nv_getlen(struct net_device *dev, void *packet, int datalen) ...@@ -2608,8 +2606,8 @@ static int nv_getlen(struct net_device *dev, void *packet, int datalen)
protolen = ntohs(((struct ethhdr *)packet)->h_proto); protolen = ntohs(((struct ethhdr *)packet)->h_proto);
hdrlen = ETH_HLEN; hdrlen = ETH_HLEN;
} }
dprintk(KERN_DEBUG "%s: nv_getlen: datalen %d, protolen %d, hdrlen %d\n", netdev_dbg(dev, "%s: datalen %d, protolen %d, hdrlen %d\n",
dev->name, datalen, protolen, hdrlen); __func__, datalen, protolen, hdrlen);
if (protolen > ETH_DATA_LEN) if (protolen > ETH_DATA_LEN)
return datalen; /* Value in proto field not a len, no checks possible */ return datalen; /* Value in proto field not a len, no checks possible */
...@@ -2620,26 +2618,25 @@ static int nv_getlen(struct net_device *dev, void *packet, int datalen) ...@@ -2620,26 +2618,25 @@ static int nv_getlen(struct net_device *dev, void *packet, int datalen)
/* more data on wire than in 802 header, trim of /* more data on wire than in 802 header, trim of
* additional data. * additional data.
*/ */
dprintk(KERN_DEBUG "%s: nv_getlen: accepting %d bytes.\n", netdev_dbg(dev, "%s: accepting %d bytes\n",
dev->name, protolen); __func__, protolen);
return protolen; return protolen;
} else { } else {
/* less data on wire than mentioned in header. /* less data on wire than mentioned in header.
* Discard the packet. * Discard the packet.
*/ */
dprintk(KERN_DEBUG "%s: nv_getlen: discarding long packet.\n", netdev_dbg(dev, "%s: discarding long packet\n",
dev->name); __func__);
return -1; return -1;
} }
} else { } else {
/* short packet. Accept only if 802 values are also short */ /* short packet. Accept only if 802 values are also short */
if (protolen > ETH_ZLEN) { if (protolen > ETH_ZLEN) {
dprintk(KERN_DEBUG "%s: nv_getlen: discarding short packet.\n", netdev_dbg(dev, "%s: discarding short packet\n",
dev->name); __func__);
return -1; return -1;
} }
dprintk(KERN_DEBUG "%s: nv_getlen: accepting %d bytes.\n", netdev_dbg(dev, "%s: accepting %d bytes\n", __func__, datalen);
dev->name, datalen);
return datalen; return datalen;
} }
} }
...@@ -2656,8 +2653,7 @@ static int nv_rx_process(struct net_device *dev, int limit) ...@@ -2656,8 +2653,7 @@ static int nv_rx_process(struct net_device *dev, int limit)
!((flags = le32_to_cpu(np->get_rx.orig->flaglen)) & NV_RX_AVAIL) && !((flags = le32_to_cpu(np->get_rx.orig->flaglen)) & NV_RX_AVAIL) &&
(rx_work < limit)) { (rx_work < limit)) {
dprintk(KERN_DEBUG "%s: nv_rx_process: flags 0x%x.\n", netdev_dbg(dev, "%s: flags 0x%x\n", __func__, flags);
dev->name, flags);
/* /*
* the packet is for us - immediately tear down the pci mapping. * the packet is for us - immediately tear down the pci mapping.
...@@ -2672,9 +2668,9 @@ static int nv_rx_process(struct net_device *dev, int limit) ...@@ -2672,9 +2668,9 @@ static int nv_rx_process(struct net_device *dev, int limit)
{ {
int j; int j;
dprintk(KERN_DEBUG "Dumping packet (flags 0x%x).", flags); netdev_dbg(dev, "Dumping packet (flags 0x%x)\n", flags);
for (j = 0; j < 64; j++) { for (j = 0; j < 64; j++) {
if ((j%16) == 0) if ((j%16) == 0 && j)
dprintk("\n%03x:", j); dprintk("\n%03x:", j);
dprintk(" %02x", ((unsigned char *)skb->data)[j]); dprintk(" %02x", ((unsigned char *)skb->data)[j]);
} }
...@@ -2754,8 +2750,8 @@ static int nv_rx_process(struct net_device *dev, int limit) ...@@ -2754,8 +2750,8 @@ static int nv_rx_process(struct net_device *dev, int limit)
/* got a valid packet - forward it to the network core */ /* got a valid packet - forward it to the network core */
skb_put(skb, len); skb_put(skb, len);
skb->protocol = eth_type_trans(skb, dev); skb->protocol = eth_type_trans(skb, dev);
dprintk(KERN_DEBUG "%s: nv_rx_process: %d bytes, proto %d accepted.\n", netdev_dbg(dev, "%s: %d bytes, proto %d accepted\n",
dev->name, len, skb->protocol); __func__, len, skb->protocol);
napi_gro_receive(&np->napi, skb); napi_gro_receive(&np->napi, skb);
dev->stats.rx_packets++; dev->stats.rx_packets++;
dev->stats.rx_bytes += len; dev->stats.rx_bytes += len;
...@@ -2784,8 +2780,7 @@ static int nv_rx_process_optimized(struct net_device *dev, int limit) ...@@ -2784,8 +2780,7 @@ static int nv_rx_process_optimized(struct net_device *dev, int limit)
!((flags = le32_to_cpu(np->get_rx.ex->flaglen)) & NV_RX2_AVAIL) && !((flags = le32_to_cpu(np->get_rx.ex->flaglen)) & NV_RX2_AVAIL) &&
(rx_work < limit)) { (rx_work < limit)) {
dprintk(KERN_DEBUG "%s: nv_rx_process_optimized: flags 0x%x.\n", netdev_dbg(dev, "%s: flags 0x%x\n", __func__, flags);
dev->name, flags);
/* /*
* the packet is for us - immediately tear down the pci mapping. * the packet is for us - immediately tear down the pci mapping.
...@@ -2800,9 +2795,9 @@ static int nv_rx_process_optimized(struct net_device *dev, int limit) ...@@ -2800,9 +2795,9 @@ static int nv_rx_process_optimized(struct net_device *dev, int limit)
{ {
int j; int j;
dprintk(KERN_DEBUG "Dumping packet (flags 0x%x).", flags); netdev_dbg(dev, "Dumping packet (flags 0x%x)\n", flags);
for (j = 0; j < 64; j++) { for (j = 0; j < 64; j++) {
if ((j%16) == 0) if ((j%16) == 0 && j)
dprintk("\n%03x:", j); dprintk("\n%03x:", j);
dprintk(" %02x", ((unsigned char *)skb->data)[j]); dprintk(" %02x", ((unsigned char *)skb->data)[j]);
} }
...@@ -2840,8 +2835,8 @@ static int nv_rx_process_optimized(struct net_device *dev, int limit) ...@@ -2840,8 +2835,8 @@ static int nv_rx_process_optimized(struct net_device *dev, int limit)
skb->protocol = eth_type_trans(skb, dev); skb->protocol = eth_type_trans(skb, dev);
prefetch(skb->data); prefetch(skb->data);
dprintk(KERN_DEBUG "%s: nv_rx_process_optimized: %d bytes, proto %d accepted.\n", netdev_dbg(dev, "%s: %d bytes, proto %d accepted\n",
dev->name, len, skb->protocol); __func__, len, skb->protocol);
if (likely(!np->vlangrp)) { if (likely(!np->vlangrp)) {
napi_gro_receive(&np->napi, skb); napi_gro_receive(&np->napi, skb);
...@@ -3134,8 +3129,8 @@ static int nv_update_linkspeed(struct net_device *dev) ...@@ -3134,8 +3129,8 @@ static int nv_update_linkspeed(struct net_device *dev)
mii_status = mii_rw(dev, np->phyaddr, MII_BMSR, MII_READ); mii_status = mii_rw(dev, np->phyaddr, MII_BMSR, MII_READ);
if (!(mii_status & BMSR_LSTATUS)) { if (!(mii_status & BMSR_LSTATUS)) {
dprintk(KERN_DEBUG "%s: no link detected by phy - falling back to 10HD.\n", netdev_dbg(dev,
dev->name); "no link detected by phy - falling back to 10HD\n");
newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_10; newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_10;
newdup = 0; newdup = 0;
retval = 0; retval = 0;
...@@ -3143,8 +3138,8 @@ static int nv_update_linkspeed(struct net_device *dev) ...@@ -3143,8 +3138,8 @@ static int nv_update_linkspeed(struct net_device *dev)
} }
if (np->autoneg == 0) { if (np->autoneg == 0) {
dprintk(KERN_DEBUG "%s: nv_update_linkspeed: autoneg off, PHY set to 0x%04x.\n", netdev_dbg(dev, "%s: autoneg off, PHY set to 0x%04x\n",
dev->name, np->fixed_mode); __func__, np->fixed_mode);
if (np->fixed_mode & LPA_100FULL) { if (np->fixed_mode & LPA_100FULL) {
newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_100; newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_100;
newdup = 1; newdup = 1;
...@@ -3167,14 +3162,15 @@ static int nv_update_linkspeed(struct net_device *dev) ...@@ -3167,14 +3162,15 @@ static int nv_update_linkspeed(struct net_device *dev)
newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_10; newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_10;
newdup = 0; newdup = 0;
retval = 0; retval = 0;
dprintk(KERN_DEBUG "%s: autoneg not completed - falling back to 10HD.\n", dev->name); netdev_dbg(dev,
"autoneg not completed - falling back to 10HD\n");
goto set_speed; goto set_speed;
} }
adv = mii_rw(dev, np->phyaddr, MII_ADVERTISE, MII_READ); adv = mii_rw(dev, np->phyaddr, MII_ADVERTISE, MII_READ);
lpa = mii_rw(dev, np->phyaddr, MII_LPA, MII_READ); lpa = mii_rw(dev, np->phyaddr, MII_LPA, MII_READ);
dprintk(KERN_DEBUG "%s: nv_update_linkspeed: PHY advertises 0x%04x, lpa 0x%04x.\n", netdev_dbg(dev, "%s: PHY advertises 0x%04x, lpa 0x%04x\n",
dev->name, adv, lpa); __func__, adv, lpa);
retval = 1; retval = 1;
if (np->gigabit == PHY_GIGABIT) { if (np->gigabit == PHY_GIGABIT) {
...@@ -3183,8 +3179,8 @@ static int nv_update_linkspeed(struct net_device *dev) ...@@ -3183,8 +3179,8 @@ static int nv_update_linkspeed(struct net_device *dev)
if ((control_1000 & ADVERTISE_1000FULL) && if ((control_1000 & ADVERTISE_1000FULL) &&
(status_1000 & LPA_1000FULL)) { (status_1000 & LPA_1000FULL)) {
dprintk(KERN_DEBUG "%s: nv_update_linkspeed: GBit ethernet detected.\n", netdev_dbg(dev, "%s: GBit ethernet detected\n",
dev->name); __func__);
newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_1000; newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_1000;
newdup = 1; newdup = 1;
goto set_speed; goto set_speed;
...@@ -3206,7 +3202,8 @@ static int nv_update_linkspeed(struct net_device *dev) ...@@ -3206,7 +3202,8 @@ static int nv_update_linkspeed(struct net_device *dev)
newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_10; newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_10;
newdup = 0; newdup = 0;
} else { } else {
dprintk(KERN_DEBUG "%s: bad ability %04x - falling back to 10HD.\n", dev->name, adv_lpa); netdev_dbg(dev, "bad ability %04x - falling back to 10HD\n",
adv_lpa);
newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_10; newls = NVREG_LINKSPEED_FORCE|NVREG_LINKSPEED_10;
newdup = 0; newdup = 0;
} }
...@@ -3363,7 +3360,7 @@ static void nv_link_irq(struct net_device *dev) ...@@ -3363,7 +3360,7 @@ static void nv_link_irq(struct net_device *dev)
if (miistat & (NVREG_MIISTAT_LINKCHANGE)) if (miistat & (NVREG_MIISTAT_LINKCHANGE))
nv_linkchange(dev); nv_linkchange(dev);
dprintk(KERN_DEBUG "%s: link change notification done.\n", dev->name); netdev_dbg(dev, "link change notification done\n");
} }
static void nv_msi_workaround(struct fe_priv *np) static void nv_msi_workaround(struct fe_priv *np)
...@@ -3414,7 +3411,7 @@ static irqreturn_t nv_nic_irq(int foo, void *data) ...@@ -3414,7 +3411,7 @@ static irqreturn_t nv_nic_irq(int foo, void *data)
struct fe_priv *np = netdev_priv(dev); struct fe_priv *np = netdev_priv(dev);
u8 __iomem *base = get_hwbase(dev); u8 __iomem *base = get_hwbase(dev);
dprintk(KERN_DEBUG "%s: nv_nic_irq\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
if (!(np->msi_flags & NV_MSI_X_ENABLED)) { if (!(np->msi_flags & NV_MSI_X_ENABLED)) {
np->events = readl(base + NvRegIrqStatus); np->events = readl(base + NvRegIrqStatus);
...@@ -3423,7 +3420,7 @@ static irqreturn_t nv_nic_irq(int foo, void *data) ...@@ -3423,7 +3420,7 @@ static irqreturn_t nv_nic_irq(int foo, void *data)
np->events = readl(base + NvRegMSIXIrqStatus); np->events = readl(base + NvRegMSIXIrqStatus);
writel(np->events, base + NvRegMSIXIrqStatus); writel(np->events, base + NvRegMSIXIrqStatus);
} }
dprintk(KERN_DEBUG "%s: irq: %08x\n", dev->name, np->events); netdev_dbg(dev, "irq: %08x\n", np->events);
if (!(np->events & np->irqmask)) if (!(np->events & np->irqmask))
return IRQ_NONE; return IRQ_NONE;
...@@ -3437,7 +3434,7 @@ static irqreturn_t nv_nic_irq(int foo, void *data) ...@@ -3437,7 +3434,7 @@ static irqreturn_t nv_nic_irq(int foo, void *data)
__napi_schedule(&np->napi); __napi_schedule(&np->napi);
} }
dprintk(KERN_DEBUG "%s: nv_nic_irq completed\n", dev->name); netdev_dbg(dev, "%s: completed\n", __func__);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
...@@ -3453,7 +3450,7 @@ static irqreturn_t nv_nic_irq_optimized(int foo, void *data) ...@@ -3453,7 +3450,7 @@ static irqreturn_t nv_nic_irq_optimized(int foo, void *data)
struct fe_priv *np = netdev_priv(dev); struct fe_priv *np = netdev_priv(dev);
u8 __iomem *base = get_hwbase(dev); u8 __iomem *base = get_hwbase(dev);
dprintk(KERN_DEBUG "%s: nv_nic_irq_optimized\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
if (!(np->msi_flags & NV_MSI_X_ENABLED)) { if (!(np->msi_flags & NV_MSI_X_ENABLED)) {
np->events = readl(base + NvRegIrqStatus); np->events = readl(base + NvRegIrqStatus);
...@@ -3462,7 +3459,7 @@ static irqreturn_t nv_nic_irq_optimized(int foo, void *data) ...@@ -3462,7 +3459,7 @@ static irqreturn_t nv_nic_irq_optimized(int foo, void *data)
np->events = readl(base + NvRegMSIXIrqStatus); np->events = readl(base + NvRegMSIXIrqStatus);
writel(np->events, base + NvRegMSIXIrqStatus); writel(np->events, base + NvRegMSIXIrqStatus);
} }
dprintk(KERN_DEBUG "%s: irq: %08x\n", dev->name, np->events); netdev_dbg(dev, "irq: %08x\n", np->events);
if (!(np->events & np->irqmask)) if (!(np->events & np->irqmask))
return IRQ_NONE; return IRQ_NONE;
...@@ -3475,7 +3472,7 @@ static irqreturn_t nv_nic_irq_optimized(int foo, void *data) ...@@ -3475,7 +3472,7 @@ static irqreturn_t nv_nic_irq_optimized(int foo, void *data)
writel(0, base + NvRegIrqMask); writel(0, base + NvRegIrqMask);
__napi_schedule(&np->napi); __napi_schedule(&np->napi);
} }
dprintk(KERN_DEBUG "%s: nv_nic_irq_optimized completed\n", dev->name); netdev_dbg(dev, "%s: completed\n", __func__);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
...@@ -3489,12 +3486,12 @@ static irqreturn_t nv_nic_irq_tx(int foo, void *data) ...@@ -3489,12 +3486,12 @@ static irqreturn_t nv_nic_irq_tx(int foo, void *data)
int i; int i;
unsigned long flags; unsigned long flags;
dprintk(KERN_DEBUG "%s: nv_nic_irq_tx\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
for (i = 0;; i++) { for (i = 0;; i++) {
events = readl(base + NvRegMSIXIrqStatus) & NVREG_IRQ_TX_ALL; events = readl(base + NvRegMSIXIrqStatus) & NVREG_IRQ_TX_ALL;
writel(NVREG_IRQ_TX_ALL, base + NvRegMSIXIrqStatus); writel(NVREG_IRQ_TX_ALL, base + NvRegMSIXIrqStatus);
dprintk(KERN_DEBUG "%s: tx irq: %08x\n", dev->name, events); netdev_dbg(dev, "tx irq: %08x\n", events);
if (!(events & np->irqmask)) if (!(events & np->irqmask))
break; break;
...@@ -3518,7 +3515,7 @@ static irqreturn_t nv_nic_irq_tx(int foo, void *data) ...@@ -3518,7 +3515,7 @@ static irqreturn_t nv_nic_irq_tx(int foo, void *data)
} }
} }
dprintk(KERN_DEBUG "%s: nv_nic_irq_tx completed\n", dev->name); netdev_dbg(dev, "%s: completed\n", __func__);
return IRQ_RETVAL(i); return IRQ_RETVAL(i);
} }
...@@ -3603,12 +3600,12 @@ static irqreturn_t nv_nic_irq_rx(int foo, void *data) ...@@ -3603,12 +3600,12 @@ static irqreturn_t nv_nic_irq_rx(int foo, void *data)
int i; int i;
unsigned long flags; unsigned long flags;
dprintk(KERN_DEBUG "%s: nv_nic_irq_rx\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
for (i = 0;; i++) { for (i = 0;; i++) {
events = readl(base + NvRegMSIXIrqStatus) & NVREG_IRQ_RX_ALL; events = readl(base + NvRegMSIXIrqStatus) & NVREG_IRQ_RX_ALL;
writel(NVREG_IRQ_RX_ALL, base + NvRegMSIXIrqStatus); writel(NVREG_IRQ_RX_ALL, base + NvRegMSIXIrqStatus);
dprintk(KERN_DEBUG "%s: rx irq: %08x\n", dev->name, events); netdev_dbg(dev, "rx irq: %08x\n", events);
if (!(events & np->irqmask)) if (!(events & np->irqmask))
break; break;
...@@ -3636,7 +3633,7 @@ static irqreturn_t nv_nic_irq_rx(int foo, void *data) ...@@ -3636,7 +3633,7 @@ static irqreturn_t nv_nic_irq_rx(int foo, void *data)
break; break;
} }
} }
dprintk(KERN_DEBUG "%s: nv_nic_irq_rx completed\n", dev->name); netdev_dbg(dev, "%s: completed\n", __func__);
return IRQ_RETVAL(i); return IRQ_RETVAL(i);
} }
...@@ -3650,12 +3647,12 @@ static irqreturn_t nv_nic_irq_other(int foo, void *data) ...@@ -3650,12 +3647,12 @@ static irqreturn_t nv_nic_irq_other(int foo, void *data)
int i; int i;
unsigned long flags; unsigned long flags;
dprintk(KERN_DEBUG "%s: nv_nic_irq_other\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
for (i = 0;; i++) { for (i = 0;; i++) {
events = readl(base + NvRegMSIXIrqStatus) & NVREG_IRQ_OTHER; events = readl(base + NvRegMSIXIrqStatus) & NVREG_IRQ_OTHER;
writel(NVREG_IRQ_OTHER, base + NvRegMSIXIrqStatus); writel(NVREG_IRQ_OTHER, base + NvRegMSIXIrqStatus);
dprintk(KERN_DEBUG "%s: irq: %08x\n", dev->name, events); netdev_dbg(dev, "irq: %08x\n", events);
if (!(events & np->irqmask)) if (!(events & np->irqmask))
break; break;
...@@ -3705,7 +3702,7 @@ static irqreturn_t nv_nic_irq_other(int foo, void *data) ...@@ -3705,7 +3702,7 @@ static irqreturn_t nv_nic_irq_other(int foo, void *data)
} }
} }
dprintk(KERN_DEBUG "%s: nv_nic_irq_other completed\n", dev->name); netdev_dbg(dev, "%s: completed\n", __func__);
return IRQ_RETVAL(i); return IRQ_RETVAL(i);
} }
...@@ -3717,7 +3714,7 @@ static irqreturn_t nv_nic_irq_test(int foo, void *data) ...@@ -3717,7 +3714,7 @@ static irqreturn_t nv_nic_irq_test(int foo, void *data)
u8 __iomem *base = get_hwbase(dev); u8 __iomem *base = get_hwbase(dev);
u32 events; u32 events;
dprintk(KERN_DEBUG "%s: nv_nic_irq_test\n", dev->name); netdev_dbg(dev, "%s\n", __func__);
if (!(np->msi_flags & NV_MSI_X_ENABLED)) { if (!(np->msi_flags & NV_MSI_X_ENABLED)) {
events = readl(base + NvRegIrqStatus) & NVREG_IRQSTAT_MASK; events = readl(base + NvRegIrqStatus) & NVREG_IRQSTAT_MASK;
...@@ -3727,7 +3724,7 @@ static irqreturn_t nv_nic_irq_test(int foo, void *data) ...@@ -3727,7 +3724,7 @@ static irqreturn_t nv_nic_irq_test(int foo, void *data)
writel(NVREG_IRQ_TIMER, base + NvRegMSIXIrqStatus); writel(NVREG_IRQ_TIMER, base + NvRegMSIXIrqStatus);
} }
pci_push(base); pci_push(base);
dprintk(KERN_DEBUG "%s: irq: %08x\n", dev->name, events); netdev_dbg(dev, "irq: %08x\n", events);
if (!(events & NVREG_IRQ_TIMER)) if (!(events & NVREG_IRQ_TIMER))
return IRQ_RETVAL(0); return IRQ_RETVAL(0);
...@@ -3737,7 +3734,7 @@ static irqreturn_t nv_nic_irq_test(int foo, void *data) ...@@ -3737,7 +3734,7 @@ static irqreturn_t nv_nic_irq_test(int foo, void *data)
np->intr_test = 1; np->intr_test = 1;
spin_unlock(&np->lock); spin_unlock(&np->lock);
dprintk(KERN_DEBUG "%s: nv_nic_irq_test completed\n", dev->name); netdev_dbg(dev, "%s: completed\n", __func__);
return IRQ_RETVAL(1); return IRQ_RETVAL(1);
} }
...@@ -4874,21 +4871,21 @@ static int nv_loopback_test(struct net_device *dev) ...@@ -4874,21 +4871,21 @@ static int nv_loopback_test(struct net_device *dev)
if (ret) { if (ret) {
if (len != pkt_len) { if (len != pkt_len) {
ret = 0; ret = 0;
dprintk(KERN_DEBUG "%s: loopback len mismatch %d vs %d\n", netdev_dbg(dev, "loopback len mismatch %d vs %d\n",
dev->name, len, pkt_len); len, pkt_len);
} else { } else {
rx_skb = np->rx_skb[0].skb; rx_skb = np->rx_skb[0].skb;
for (i = 0; i < pkt_len; i++) { for (i = 0; i < pkt_len; i++) {
if (rx_skb->data[i] != (u8)(i & 0xff)) { if (rx_skb->data[i] != (u8)(i & 0xff)) {
ret = 0; ret = 0;
dprintk(KERN_DEBUG "%s: loopback pattern check failed on byte %d\n", netdev_dbg(dev, "loopback pattern check failed on byte %d\n",
dev->name, i); i);
break; break;
} }
} }
} }
} else { } else {
dprintk(KERN_DEBUG "%s: loopback - did not receive test packet\n", dev->name); netdev_dbg(dev, "loopback - did not receive test packet\n");
} }
pci_unmap_single(np->pci_dev, test_dma_addr, pci_unmap_single(np->pci_dev, test_dma_addr,
...@@ -5138,7 +5135,7 @@ static int nv_open(struct net_device *dev) ...@@ -5138,7 +5135,7 @@ static int nv_open(struct net_device *dev)
int oom, i; int oom, i;
u32 low; u32 low;
dprintk(KERN_DEBUG "nv_open: begin\n"); netdev_dbg(dev, "%s\n", __func__);
/* power up phy */ /* power up phy */
mii_rw(dev, np->phyaddr, MII_BMCR, mii_rw(dev, np->phyaddr, MII_BMCR,
...@@ -5435,10 +5432,11 @@ static int __devinit nv_probe(struct pci_dev *pci_dev, const struct pci_device_i ...@@ -5435,10 +5432,11 @@ static int __devinit nv_probe(struct pci_dev *pci_dev, const struct pci_device_i
err = -EINVAL; err = -EINVAL;
addr = 0; addr = 0;
for (i = 0; i < DEVICE_COUNT_RESOURCE; i++) { for (i = 0; i < DEVICE_COUNT_RESOURCE; i++) {
dprintk(KERN_DEBUG "%s: resource %d start %p len %ld flags 0x%08lx.\n", netdev_dbg(dev, "%s: resource %d start %p len %lld flags 0x%08lx\n",
pci_name(pci_dev), i, (void *)pci_resource_start(pci_dev, i), pci_name(pci_dev), i,
pci_resource_len(pci_dev, i), (void *)(unsigned long)pci_resource_start(pci_dev, i),
pci_resource_flags(pci_dev, i)); (long long)pci_resource_len(pci_dev, i),
pci_resource_flags(pci_dev, i));
if (pci_resource_flags(pci_dev, i) & IORESOURCE_MEM && if (pci_resource_flags(pci_dev, i) & IORESOURCE_MEM &&
pci_resource_len(pci_dev, i) >= np->register_size) { pci_resource_len(pci_dev, i) >= np->register_size) {
addr = pci_resource_start(pci_dev, i); addr = pci_resource_start(pci_dev, i);
...@@ -5607,8 +5605,8 @@ static int __devinit nv_probe(struct pci_dev *pci_dev, const struct pci_device_i ...@@ -5607,8 +5605,8 @@ static int __devinit nv_probe(struct pci_dev *pci_dev, const struct pci_device_i
random_ether_addr(dev->dev_addr); random_ether_addr(dev->dev_addr);
} }
dprintk(KERN_DEBUG "%s: MAC Address %pM\n", netdev_dbg(dev, "%s: MAC Address %pM\n",
pci_name(pci_dev), dev->dev_addr); pci_name(pci_dev), dev->dev_addr);
/* set mac address */ /* set mac address */
nv_copy_mac_to_hw(dev); nv_copy_mac_to_hw(dev);
...@@ -5741,8 +5739,8 @@ static int __devinit nv_probe(struct pci_dev *pci_dev, const struct pci_device_i ...@@ -5741,8 +5739,8 @@ static int __devinit nv_probe(struct pci_dev *pci_dev, const struct pci_device_i
np->phy_model = id2 & PHYID2_MODEL_MASK; np->phy_model = id2 & PHYID2_MODEL_MASK;
id1 = (id1 & PHYID1_OUI_MASK) << PHYID1_OUI_SHFT; id1 = (id1 & PHYID1_OUI_MASK) << PHYID1_OUI_SHFT;
id2 = (id2 & PHYID2_OUI_MASK) >> PHYID2_OUI_SHFT; id2 = (id2 & PHYID2_OUI_MASK) >> PHYID2_OUI_SHFT;
dprintk(KERN_DEBUG "%s: open: Found PHY %04x:%04x at address %d.\n", netdev_dbg(dev, "%s: %s: Found PHY %04x:%04x at address %d\n",
pci_name(pci_dev), id1, id2, phyaddr); pci_name(pci_dev), __func__, id1, id2, phyaddr);
np->phyaddr = phyaddr; np->phyaddr = phyaddr;
np->phy_oui = id1 | id2; np->phy_oui = id1 | id2;
......
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