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

netdev: convert ni65 to net_device_ops

Also, use internal net_device_stats.
Signed-off-by: default avatarStephen Hemminger <shemminger@vyatta.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 2c7669e3
...@@ -237,7 +237,7 @@ struct priv ...@@ -237,7 +237,7 @@ struct priv
void *tmdbounce[TMDNUM]; void *tmdbounce[TMDNUM];
int tmdbouncenum; int tmdbouncenum;
int lock,xmit_queued; int lock,xmit_queued;
struct net_device_stats stats;
void *self; void *self;
int cmdr_addr; int cmdr_addr;
int cardno; int cardno;
...@@ -257,7 +257,6 @@ static void ni65_timeout(struct net_device *dev); ...@@ -257,7 +257,6 @@ static void ni65_timeout(struct net_device *dev);
static int ni65_close(struct net_device *dev); static int ni65_close(struct net_device *dev);
static int ni65_alloc_buffer(struct net_device *dev); static int ni65_alloc_buffer(struct net_device *dev);
static void ni65_free_buffer(struct priv *p); static void ni65_free_buffer(struct priv *p);
static struct net_device_stats *ni65_get_stats(struct net_device *);
static void set_multicast_list(struct net_device *dev); static void set_multicast_list(struct net_device *dev);
static int irqtab[] __initdata = { 9,12,15,5 }; /* irq config-translate */ static int irqtab[] __initdata = { 9,12,15,5 }; /* irq config-translate */
...@@ -401,6 +400,17 @@ out: ...@@ -401,6 +400,17 @@ out:
return ERR_PTR(err); return ERR_PTR(err);
} }
static const struct net_device_ops ni65_netdev_ops = {
.ndo_open = ni65_open,
.ndo_stop = ni65_close,
.ndo_start_xmit = ni65_send_packet,
.ndo_tx_timeout = ni65_timeout,
.ndo_set_multicast_list = set_multicast_list,
.ndo_change_mtu = eth_change_mtu,
.ndo_set_mac_address = eth_mac_addr,
.ndo_validate_addr = eth_validate_addr,
};
/* /*
* this is the real card probe .. * this is the real card probe ..
*/ */
...@@ -549,13 +559,9 @@ static int __init ni65_probe1(struct net_device *dev,int ioaddr) ...@@ -549,13 +559,9 @@ static int __init ni65_probe1(struct net_device *dev,int ioaddr)
} }
dev->base_addr = ioaddr; dev->base_addr = ioaddr;
dev->open = ni65_open; dev->netdev_ops = &ni65_netdev_ops;
dev->stop = ni65_close;
dev->hard_start_xmit = ni65_send_packet;
dev->tx_timeout = ni65_timeout;
dev->watchdog_timeo = HZ/2; dev->watchdog_timeo = HZ/2;
dev->get_stats = ni65_get_stats;
dev->set_multicast_list = set_multicast_list;
return 0; /* everything is OK */ return 0; /* everything is OK */
} }
...@@ -901,13 +907,13 @@ static irqreturn_t ni65_interrupt(int irq, void * dev_id) ...@@ -901,13 +907,13 @@ static irqreturn_t ni65_interrupt(int irq, void * dev_id)
if(debuglevel > 1) if(debuglevel > 1)
printk(KERN_ERR "%s: general error: %04x.\n",dev->name,csr0); printk(KERN_ERR "%s: general error: %04x.\n",dev->name,csr0);
if(csr0 & CSR0_BABL) if(csr0 & CSR0_BABL)
p->stats.tx_errors++; dev->stats.tx_errors++;
if(csr0 & CSR0_MISS) { if(csr0 & CSR0_MISS) {
int i; int i;
for(i=0;i<RMDNUM;i++) for(i=0;i<RMDNUM;i++)
printk("%02x ",p->rmdhead[i].u.s.status); printk("%02x ",p->rmdhead[i].u.s.status);
printk("\n"); printk("\n");
p->stats.rx_errors++; dev->stats.rx_errors++;
} }
if(csr0 & CSR0_MERR) { if(csr0 & CSR0_MERR) {
if(debuglevel > 1) if(debuglevel > 1)
...@@ -997,12 +1003,12 @@ static void ni65_xmit_intr(struct net_device *dev,int csr0) ...@@ -997,12 +1003,12 @@ static void ni65_xmit_intr(struct net_device *dev,int csr0)
#endif #endif
/* checking some errors */ /* checking some errors */
if(tmdp->status2 & XMIT_RTRY) if(tmdp->status2 & XMIT_RTRY)
p->stats.tx_aborted_errors++; dev->stats.tx_aborted_errors++;
if(tmdp->status2 & XMIT_LCAR) if(tmdp->status2 & XMIT_LCAR)
p->stats.tx_carrier_errors++; dev->stats.tx_carrier_errors++;
if(tmdp->status2 & (XMIT_BUFF | XMIT_UFLO )) { if(tmdp->status2 & (XMIT_BUFF | XMIT_UFLO )) {
/* this stops the xmitter */ /* this stops the xmitter */
p->stats.tx_fifo_errors++; dev->stats.tx_fifo_errors++;
if(debuglevel > 0) if(debuglevel > 0)
printk(KERN_ERR "%s: Xmit FIFO/BUFF error\n",dev->name); printk(KERN_ERR "%s: Xmit FIFO/BUFF error\n",dev->name);
if(p->features & INIT_RING_BEFORE_START) { if(p->features & INIT_RING_BEFORE_START) {
...@@ -1016,12 +1022,12 @@ static void ni65_xmit_intr(struct net_device *dev,int csr0) ...@@ -1016,12 +1022,12 @@ static void ni65_xmit_intr(struct net_device *dev,int csr0)
if(debuglevel > 2) if(debuglevel > 2)
printk(KERN_ERR "%s: xmit-error: %04x %02x-%04x\n",dev->name,csr0,(int) tmdstat,(int) tmdp->status2); printk(KERN_ERR "%s: xmit-error: %04x %02x-%04x\n",dev->name,csr0,(int) tmdstat,(int) tmdp->status2);
if(!(csr0 & CSR0_BABL)) /* don't count errors twice */ if(!(csr0 & CSR0_BABL)) /* don't count errors twice */
p->stats.tx_errors++; dev->stats.tx_errors++;
tmdp->status2 = 0; tmdp->status2 = 0;
} }
else { else {
p->stats.tx_bytes -= (short)(tmdp->blen); dev->stats.tx_bytes -= (short)(tmdp->blen);
p->stats.tx_packets++; dev->stats.tx_packets++;
} }
#ifdef XMT_VIA_SKB #ifdef XMT_VIA_SKB
...@@ -1057,7 +1063,7 @@ static void ni65_recv_intr(struct net_device *dev,int csr0) ...@@ -1057,7 +1063,7 @@ static void ni65_recv_intr(struct net_device *dev,int csr0)
if(!(rmdstat & RCV_ERR)) { if(!(rmdstat & RCV_ERR)) {
if(rmdstat & RCV_START) if(rmdstat & RCV_START)
{ {
p->stats.rx_length_errors++; dev->stats.rx_length_errors++;
printk(KERN_ERR "%s: recv, packet too long: %d\n",dev->name,rmdp->mlen & 0x0fff); printk(KERN_ERR "%s: recv, packet too long: %d\n",dev->name,rmdp->mlen & 0x0fff);
} }
} }
...@@ -1066,16 +1072,16 @@ static void ni65_recv_intr(struct net_device *dev,int csr0) ...@@ -1066,16 +1072,16 @@ static void ni65_recv_intr(struct net_device *dev,int csr0)
printk(KERN_ERR "%s: receive-error: %04x, lance-status: %04x/%04x\n", printk(KERN_ERR "%s: receive-error: %04x, lance-status: %04x/%04x\n",
dev->name,(int) rmdstat,csr0,(int) inw(PORT+L_DATAREG) ); dev->name,(int) rmdstat,csr0,(int) inw(PORT+L_DATAREG) );
if(rmdstat & RCV_FRAM) if(rmdstat & RCV_FRAM)
p->stats.rx_frame_errors++; dev->stats.rx_frame_errors++;
if(rmdstat & RCV_OFLO) if(rmdstat & RCV_OFLO)
p->stats.rx_over_errors++; dev->stats.rx_over_errors++;
if(rmdstat & RCV_CRC) if(rmdstat & RCV_CRC)
p->stats.rx_crc_errors++; dev->stats.rx_crc_errors++;
if(rmdstat & RCV_BUF_ERR) if(rmdstat & RCV_BUF_ERR)
p->stats.rx_fifo_errors++; dev->stats.rx_fifo_errors++;
} }
if(!(csr0 & CSR0_MISS)) /* don't count errors twice */ if(!(csr0 & CSR0_MISS)) /* don't count errors twice */
p->stats.rx_errors++; dev->stats.rx_errors++;
} }
else if( (len = (rmdp->mlen & 0x0fff) - 4) >= 60) else if( (len = (rmdp->mlen & 0x0fff) - 4) >= 60)
{ {
...@@ -1106,20 +1112,20 @@ static void ni65_recv_intr(struct net_device *dev,int csr0) ...@@ -1106,20 +1112,20 @@ static void ni65_recv_intr(struct net_device *dev,int csr0)
skb_put(skb,len); skb_put(skb,len);
skb_copy_to_linear_data(skb, (unsigned char *) p->recvbounce[p->rmdnum],len); skb_copy_to_linear_data(skb, (unsigned char *) p->recvbounce[p->rmdnum],len);
#endif #endif
p->stats.rx_packets++; dev->stats.rx_packets++;
p->stats.rx_bytes += len; dev->stats.rx_bytes += len;
skb->protocol=eth_type_trans(skb,dev); skb->protocol=eth_type_trans(skb,dev);
netif_rx(skb); netif_rx(skb);
} }
else else
{ {
printk(KERN_ERR "%s: can't alloc new sk_buff\n",dev->name); printk(KERN_ERR "%s: can't alloc new sk_buff\n",dev->name);
p->stats.rx_dropped++; dev->stats.rx_dropped++;
} }
} }
else { else {
printk(KERN_INFO "%s: received runt packet\n",dev->name); printk(KERN_INFO "%s: received runt packet\n",dev->name);
p->stats.rx_errors++; dev->stats.rx_errors++;
} }
rmdp->blen = -(R_BUF_SIZE-8); rmdp->blen = -(R_BUF_SIZE-8);
rmdp->mlen = 0; rmdp->mlen = 0;
...@@ -1213,23 +1219,6 @@ static int ni65_send_packet(struct sk_buff *skb, struct net_device *dev) ...@@ -1213,23 +1219,6 @@ static int ni65_send_packet(struct sk_buff *skb, struct net_device *dev)
return 0; return 0;
} }
static struct net_device_stats *ni65_get_stats(struct net_device *dev)
{
#if 0
int i;
struct priv *p = dev->ml_priv;
for(i=0;i<RMDNUM;i++)
{
struct rmd *rmdp = p->rmdhead + ((p->rmdnum + i) & (RMDNUM-1));
printk("%02x ",rmdp->u.s.status);
}
printk("\n");
#endif
return &((struct priv *)dev->ml_priv)->stats;
}
static void set_multicast_list(struct net_device *dev) static void set_multicast_list(struct net_device *dev)
{ {
if(!ni65_lance_reinit(dev)) if(!ni65_lance_reinit(dev))
......
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