Commit 6394d7c9 authored by Stephen Hemminger's avatar Stephen Hemminger Committed by David S. Miller

xir2cps: convert to internal net_device stats

Signed-off-by: default avatarStephen Hemminger <shemminger@vyatta.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 23169a40
...@@ -335,7 +335,7 @@ typedef struct local_info_t { ...@@ -335,7 +335,7 @@ typedef struct local_info_t {
struct net_device *dev; struct net_device *dev;
struct pcmcia_device *p_dev; struct pcmcia_device *p_dev;
dev_node_t node; dev_node_t node;
struct net_device_stats stats;
int card_type; int card_type;
int probe_port; int probe_port;
int silicon; /* silicon revision. 0=old CE2, 1=Scipper, 4=Mohawk */ int silicon; /* silicon revision. 0=old CE2, 1=Scipper, 4=Mohawk */
...@@ -355,7 +355,6 @@ typedef struct local_info_t { ...@@ -355,7 +355,6 @@ typedef struct local_info_t {
static int do_start_xmit(struct sk_buff *skb, struct net_device *dev); static int do_start_xmit(struct sk_buff *skb, struct net_device *dev);
static void xirc_tx_timeout(struct net_device *dev); static void xirc_tx_timeout(struct net_device *dev);
static void xirc2ps_tx_timeout_task(struct work_struct *work); static void xirc2ps_tx_timeout_task(struct work_struct *work);
static struct net_device_stats *do_get_stats(struct net_device *dev);
static void set_addresses(struct net_device *dev); static void set_addresses(struct net_device *dev);
static void set_multicast_list(struct net_device *dev); static void set_multicast_list(struct net_device *dev);
static int set_card_type(struct pcmcia_device *link, const void *s); static int set_card_type(struct pcmcia_device *link, const void *s);
...@@ -583,7 +582,6 @@ xirc2ps_probe(struct pcmcia_device *link) ...@@ -583,7 +582,6 @@ xirc2ps_probe(struct pcmcia_device *link)
/* Fill in card specific entries */ /* Fill in card specific entries */
dev->hard_start_xmit = &do_start_xmit; dev->hard_start_xmit = &do_start_xmit;
dev->set_config = &do_config; dev->set_config = &do_config;
dev->get_stats = &do_get_stats;
dev->do_ioctl = &do_ioctl; dev->do_ioctl = &do_ioctl;
SET_ETHTOOL_OPS(dev, &netdev_ethtool_ops); SET_ETHTOOL_OPS(dev, &netdev_ethtool_ops);
dev->set_multicast_list = &set_multicast_list; dev->set_multicast_list = &set_multicast_list;
...@@ -1172,7 +1170,7 @@ xirc2ps_interrupt(int irq, void *dev_id) ...@@ -1172,7 +1170,7 @@ xirc2ps_interrupt(int irq, void *dev_id)
if (bytes_rcvd > maxrx_bytes && (rsr & PktRxOk)) { if (bytes_rcvd > maxrx_bytes && (rsr & PktRxOk)) {
/* too many bytes received during this int, drop the rest of the /* too many bytes received during this int, drop the rest of the
* packets */ * packets */
lp->stats.rx_dropped++; dev->stats.rx_dropped++;
DEBUG(2, "%s: RX drop, too much done\n", dev->name); DEBUG(2, "%s: RX drop, too much done\n", dev->name);
} else if (rsr & PktRxOk) { } else if (rsr & PktRxOk) {
struct sk_buff *skb; struct sk_buff *skb;
...@@ -1186,7 +1184,7 @@ xirc2ps_interrupt(int irq, void *dev_id) ...@@ -1186,7 +1184,7 @@ xirc2ps_interrupt(int irq, void *dev_id)
if (!skb) { if (!skb) {
printk(KNOT_XIRC "low memory, packet dropped (size=%u)\n", printk(KNOT_XIRC "low memory, packet dropped (size=%u)\n",
pktlen); pktlen);
lp->stats.rx_dropped++; dev->stats.rx_dropped++;
} else { /* okay get the packet */ } else { /* okay get the packet */
skb_reserve(skb, 2); skb_reserve(skb, 2);
if (lp->silicon == 0 ) { /* work around a hardware bug */ if (lp->silicon == 0 ) { /* work around a hardware bug */
...@@ -1242,24 +1240,24 @@ xirc2ps_interrupt(int irq, void *dev_id) ...@@ -1242,24 +1240,24 @@ xirc2ps_interrupt(int irq, void *dev_id)
} }
skb->protocol = eth_type_trans(skb, dev); skb->protocol = eth_type_trans(skb, dev);
netif_rx(skb); netif_rx(skb);
lp->stats.rx_packets++; dev->stats.rx_packets++;
lp->stats.rx_bytes += pktlen; dev->stats.rx_bytes += pktlen;
if (!(rsr & PhyPkt)) if (!(rsr & PhyPkt))
lp->stats.multicast++; dev->stats.multicast++;
} }
} else { /* bad packet */ } else { /* bad packet */
DEBUG(5, "rsr=%#02x\n", rsr); DEBUG(5, "rsr=%#02x\n", rsr);
} }
if (rsr & PktTooLong) { if (rsr & PktTooLong) {
lp->stats.rx_frame_errors++; dev->stats.rx_frame_errors++;
DEBUG(3, "%s: Packet too long\n", dev->name); DEBUG(3, "%s: Packet too long\n", dev->name);
} }
if (rsr & CRCErr) { if (rsr & CRCErr) {
lp->stats.rx_crc_errors++; dev->stats.rx_crc_errors++;
DEBUG(3, "%s: CRC error\n", dev->name); DEBUG(3, "%s: CRC error\n", dev->name);
} }
if (rsr & AlignErr) { if (rsr & AlignErr) {
lp->stats.rx_fifo_errors++; /* okay ? */ dev->stats.rx_fifo_errors++; /* okay ? */
DEBUG(3, "%s: Alignment error\n", dev->name); DEBUG(3, "%s: Alignment error\n", dev->name);
} }
...@@ -1270,7 +1268,7 @@ xirc2ps_interrupt(int irq, void *dev_id) ...@@ -1270,7 +1268,7 @@ xirc2ps_interrupt(int irq, void *dev_id)
eth_status = GetByte(XIRCREG_ESR); eth_status = GetByte(XIRCREG_ESR);
} }
if (rx_status & 0x10) { /* Receive overrun */ if (rx_status & 0x10) { /* Receive overrun */
lp->stats.rx_over_errors++; dev->stats.rx_over_errors++;
PutByte(XIRCREG_CR, ClearRxOvrun); PutByte(XIRCREG_CR, ClearRxOvrun);
DEBUG(3, "receive overrun cleared\n"); DEBUG(3, "receive overrun cleared\n");
} }
...@@ -1283,11 +1281,11 @@ xirc2ps_interrupt(int irq, void *dev_id) ...@@ -1283,11 +1281,11 @@ xirc2ps_interrupt(int irq, void *dev_id)
nn = GetByte(XIRCREG0_PTR); nn = GetByte(XIRCREG0_PTR);
lp->last_ptr_value = nn; lp->last_ptr_value = nn;
if (nn < n) /* rollover */ if (nn < n) /* rollover */
lp->stats.tx_packets += 256 - n; dev->stats.tx_packets += 256 - n;
else if (n == nn) { /* happens sometimes - don't know why */ else if (n == nn) { /* happens sometimes - don't know why */
DEBUG(0, "PTR not changed?\n"); DEBUG(0, "PTR not changed?\n");
} else } else
lp->stats.tx_packets += lp->last_ptr_value - n; dev->stats.tx_packets += lp->last_ptr_value - n;
netif_wake_queue(dev); netif_wake_queue(dev);
} }
if (tx_status & 0x0002) { /* Execessive collissions */ if (tx_status & 0x0002) { /* Execessive collissions */
...@@ -1295,7 +1293,7 @@ xirc2ps_interrupt(int irq, void *dev_id) ...@@ -1295,7 +1293,7 @@ xirc2ps_interrupt(int irq, void *dev_id)
PutByte(XIRCREG_CR, RestartTx); /* restart transmitter process */ PutByte(XIRCREG_CR, RestartTx); /* restart transmitter process */
} }
if (tx_status & 0x0040) if (tx_status & 0x0040)
lp->stats.tx_aborted_errors++; dev->stats.tx_aborted_errors++;
/* recalculate our work chunk so that we limit the duration of this /* recalculate our work chunk so that we limit the duration of this
* ISR to about 1/10 of a second. * ISR to about 1/10 of a second.
...@@ -1353,7 +1351,7 @@ static void ...@@ -1353,7 +1351,7 @@ static void
xirc_tx_timeout(struct net_device *dev) xirc_tx_timeout(struct net_device *dev)
{ {
local_info_t *lp = netdev_priv(dev); local_info_t *lp = netdev_priv(dev);
lp->stats.tx_errors++; dev->stats.tx_errors++;
printk(KERN_NOTICE "%s: transmit timed out\n", dev->name); printk(KERN_NOTICE "%s: transmit timed out\n", dev->name);
schedule_work(&lp->tx_timeout_task); schedule_work(&lp->tx_timeout_task);
} }
...@@ -1409,20 +1407,11 @@ do_start_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -1409,20 +1407,11 @@ do_start_xmit(struct sk_buff *skb, struct net_device *dev)
dev_kfree_skb (skb); dev_kfree_skb (skb);
dev->trans_start = jiffies; dev->trans_start = jiffies;
lp->stats.tx_bytes += pktlen; dev->stats.tx_bytes += pktlen;
netif_start_queue(dev); netif_start_queue(dev);
return 0; return 0;
} }
static struct net_device_stats *
do_get_stats(struct net_device *dev)
{
local_info_t *lp = netdev_priv(dev);
/* lp->stats.rx_missed_errors = GetByte(?) */
return &lp->stats;
}
/**************** /****************
* Set all addresses: This first one is the individual address, * Set all addresses: This first one is the individual address,
* the next 9 addresses are taken from the multicast list and * the next 9 addresses are taken from the multicast list and
......
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