hak5
/
openwrt
mirror of https://github.com/hak5/openwrt.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
openwrt/target/linux/coldfire/patches/047-m547x_8x_fec_cleanup.patch

1224 lines
40 KiB
Diff
Raw Normal View History

use broken-out patches for the coldfire to make it easier to follow differences against the bsp SVN-Revision: 16547
2009-06-23 21:04:37 +00:00
From 5bac5f471fb1124c51a5811757c54174f6224e7e Mon Sep 17 00:00:00 2001
From: Kurt Mahan <kmahan@freescale.com>
Date: Wed, 12 Mar 2008 17:13:29 -0600
Subject: [PATCH] Clean up FEC DMA driver.
LTIBName: m547x-8x-fec-cleanup
Signed-off-by: Kurt Mahan <kmahan@freescale.com>
---
drivers/net/fec/fec.c | 611 +++++++++++++++++++++---------------------------
1 files changed, 267 insertions(+), 344 deletions(-)
--- a/drivers/net/fec/fec.c
+++ b/drivers/net/fec/fec.c
@@ -4,7 +4,6 @@
*
* Code crunched to get it to work on 2.6.24 -- FEC cleanup coming
* soon -- Kurt Mahan
- *
*/
#include <linux/module.h>
#include <linux/kernel.h>
@@ -44,42 +43,40 @@
#undef FEC_2
#endif
-#define VERSION "0.13"
+#define VERSION "0.20"
MODULE_DESCRIPTION( "DMA Fast Ethernet Controller driver ver " VERSION);
-// Private structure
+/* fec private */
struct fec_priv {
struct net_device *netdev; /* owning net device */
- void* fecpriv_txbuf[FEC_TX_BUF_NUMBER]; //Array of transmission buffers
- MCD_bufDescFec *fecpriv_txdesc; // Array of transmission descriptors
- volatile unsigned int fecpriv_current_tx; // Inex of the transmission descriptor that is used by DMA
- volatile unsigned int fecpriv_next_tx; // Inex of the transmission descriptor that can be used for new data
- unsigned int fecpriv_current_rx; // Index of the reception descriptor that is used by DMA
- MCD_bufDescFec *fecpriv_rxdesc; // Array of reception descriptors
-// unsigned char *fecpriv_rxbuf; // Address of reception buffers
- struct sk_buff *askb_rx[FEC_RX_BUF_NUMBER]; // Array of reception skb structure pointers
- unsigned int fecpriv_initiator_rx; // Reception DMA initiator
- unsigned int fecpriv_initiator_tx; // Transmission DMA initiator
- int fecpriv_fec_rx_channel; // DMA reception channel
- int fecpriv_fec_tx_channel; // DMA transmission channel
- int fecpriv_rx_requestor; // DMA reception requestor
- int fecpriv_tx_requestor; // DMA transmission requestor
- void *fecpriv_interrupt_fec_rx_handler; // DMA reception handler
- void *fecpriv_interrupt_fec_tx_handler; // DMA transmission handler
- unsigned char *fecpriv_mac_addr; // Private copy of FEC address
- struct net_device_stats fecpriv_stat; // Pointer to the statistical information
+ void* fecpriv_txbuf[FEC_TX_BUF_NUMBER]; /* tx buffer ptrs */
+ MCD_bufDescFec *fecpriv_txdesc; /* tx descriptor ptrs */
+ volatile unsigned int fecpriv_current_tx; /* current tx desc index */
+ volatile unsigned int fecpriv_next_tx; /* next tx desc index */
+ unsigned int fecpriv_current_rx; /* current rx desc index */
+ MCD_bufDescFec *fecpriv_rxdesc; /* rx descriptor ptrs */
+ struct sk_buff *askb_rx[FEC_RX_BUF_NUMBER]; /* rx SKB ptrs */
+ unsigned int fecpriv_initiator_rx; /* rx dma initiator */
+ unsigned int fecpriv_initiator_tx; /* tx dma initiator */
+ int fecpriv_fec_rx_channel; /* rx dma channel */
+ int fecpriv_fec_tx_channel; /* tx dma channel */
+ int fecpriv_rx_requestor; /* rx dma requestor */
+ int fecpriv_tx_requestor; /* tx dma requestor */
+ void *fecpriv_interrupt_fec_rx_handler; /* dma rx handler */
+ void *fecpriv_interrupt_fec_tx_handler; /* dma tx handler */
+ unsigned char *fecpriv_mac_addr; /* private fec mac addr */
+ struct net_device_stats fecpriv_stat; /* stats ptr */
spinlock_t fecpriv_lock;
int fecpriv_rxflag;
struct tasklet_struct fecpriv_tasklet_reinit;
- int index;
+ int index; /* fec hw number */
};
struct net_device *fec_dev[FEC_MAX_PORTS];
-// FEC functions
+/* FEC functions */
int __init fec_init(void);
struct net_device_stats *fec_get_stat(struct net_device *dev);
-
int fec_open(struct net_device *dev);
int fec_close(struct net_device *nd);
int fec_tx(struct sk_buff *skb, struct net_device *dev);
@@ -93,10 +90,12 @@ void fec_interrupt_fec_tx_handler_fec0(v
void fec_interrupt_fec_rx_handler_fec0(void);
void fec_interrupt_fec_reinit(unsigned long data);
-unsigned char fec_mac_addr_fec0[6] = { 0x00, 0x11, 0x22, 0x33, 0x44, 0x50 }; // Default address of FEC0
+/* default fec0 address */
+unsigned char fec_mac_addr_fec0[6] = { 0x00, 0x11, 0x22, 0x33, 0x44, 0x50 };
#ifdef FEC_2
-unsigned char fec_mac_addr_fec1[6] = { 0x00, 0x11, 0x22, 0x33, 0x44, 0x51 }; // Default address of FEC1
+/* default fec1 address */
+unsigned char fec_mac_addr_fec1[6] = { 0x00, 0x11, 0x22, 0x33, 0x44, 0x51 };
#endif
extern unsigned char uboot_enet0[];
@@ -111,17 +110,18 @@ int __init fec_mac_setup0 (char *s);
#ifdef FEC_2
void fec_interrupt_fec_tx_handler_fec1(void);
void fec_interrupt_fec_rx_handler_fec1(void);
+#endif
#ifndef MODULE
int __init fec_mac_setup1 (char *s);
#endif
-#endif
int fec_read_mii(unsigned int base_addr, unsigned int pa, unsigned int ra, unsigned int *data);
int fec_write_mii(unsigned int base_addr, unsigned int pa, unsigned int ra, unsigned int data);
module_init(fec_init);
/* module_exit(fec_cleanup); */
+
__setup("mac0=", fec_mac_setup0);
#ifdef FEC_2
@@ -140,7 +140,6 @@ int fec_enet_init(struct net_device *dev
fp->index = index;
fp->netdev = dev;
fec_dev[ index ] = dev;
-printk(KERN_INFO "FEI: index=%d\n", index);
if (index == 0) {
/* disable fec0 */
@@ -175,8 +174,6 @@ printk(KERN_INFO "FEI: index=%d\n", inde
/* rx descriptors */
fp->fecpriv_rxdesc = (void*)FEC_RX_DESC_FEC0;
-printk(KERN_INFO "FEI: txdesc=0x%p rxdesc=0x%p\n", fp->fecpriv_txdesc, fp->fecpriv_rxdesc);
-
/* mac addr */
if (uboot_enet0[0] || uboot_enet0[1] || uboot_enet0[2] ||
uboot_enet0[3] || uboot_enet0[4] || uboot_enet0[5]) {
@@ -229,8 +226,6 @@ printk(KERN_INFO "FEI: txdesc=0x%p rxde
#endif
}
-printk(KERN_INFO "FEI: index=%d base_addr=0x%lx\n", index, dev->base_addr);
-
/* clear MIB */
memset((void *) (dev->base_addr + 0x200), 0, FEC_MIB_LEN);
@@ -288,8 +283,7 @@ int __init fec_init(void)
int err;
DECLARE_MAC_BUF(mac);
- printk(KERN_INFO "FEC ENET (DMA) Version .00\n");
-
+ printk(KERN_INFO "FEC ENET (DMA) Version %s\n", VERSION);
for (i = 0; i < FEC_MAX_PORTS; i++) {
dev = alloc_etherdev(sizeof(struct fec_priv));
@@ -333,7 +327,6 @@ void fec_stop(struct net_device *dev)
*
* RETURNS: If no error occurs, this function returns zero.
*************************************************************************/
-
int fec_open(struct net_device *dev)
{
struct fec_priv *fp = netdev_priv(dev);
@@ -343,13 +336,10 @@ int fec_open(struct net_device *dev)
int channel;
int error_code = -EBUSY;
-printk(KERN_INFO "FECOPEN: index=%d\n", fp->index);
-
- //Receive the DMA channels
+ /* Receive the DMA channels */
channel = dma_set_channel_fec(fp->fecpriv_rx_requestor);
- if (channel == -1)
- {
+ if (channel == -1) {
printk("Dma channel cannot be reserved\n");
goto ERRORS;
}
@@ -360,60 +350,54 @@ printk(KERN_INFO "FECOPEN: index=%d\n",
channel = dma_set_channel_fec(fp->fecpriv_tx_requestor);
- if (channel == -1)
- {
+ if (channel == -1) {
printk("Dma channel cannot be reserved\n");
goto ERRORS;
}
-printk(KERN_INFO "FECOPEN2\n");
fp->fecpriv_fec_tx_channel = channel;
dma_connect(channel, (int) fp->fecpriv_interrupt_fec_tx_handler);
- // init tasklet for controller reinitialization
+ /* init tasklet for controller reinitialization */
tasklet_init(&fp->fecpriv_tasklet_reinit, fec_interrupt_fec_reinit, (unsigned long) dev);
-printk(KERN_INFO "FECOPEN3\n");
- // Reset FIFOs
+ /* Reset FIFOs */
FEC_FECFRST(base_addr) |= FEC_SW_RST | FEC_RST_CTL;
FEC_FECFRST(base_addr) &= ~FEC_SW_RST;
- // Reset and disable FEC
+ /* Reset and disable FEC */
FEC_ECR(base_addr) = FEC_ECR_RESET;
- // Wait
udelay(10);
- // Clear all events
+ /* Clear all events */
FEC_EIR(base_addr) = FEC_EIR_CLEAR;
- // Reset FIFO status
+ /* Reset FIFO status */
FEC_FECTFSR(base_addr) = FEC_FECTFSR_MSK;
FEC_FECRFSR(base_addr) = FEC_FECRFSR_MSK;
-#if 0
-/* JKM -- move into HW init */
- // Copy the default address to the device structure
- memcpy(dev->dev_addr, fp->fecpriv_mac_addr, ETH_ALEN);
-#endif
+ /* Set the default address */
+ FEC_PALR(base_addr) = (fp->fecpriv_mac_addr[0] << 24) |
+ (fp->fecpriv_mac_addr[1] << 16) |
+ (fp->fecpriv_mac_addr[2] << 8) |
+ fp->fecpriv_mac_addr[3];
+ FEC_PAUR(base_addr) = (fp->fecpriv_mac_addr[4] << 24) |
+ (fp->fecpriv_mac_addr[5] << 16) | 0x8808;
- // Set the default address
- FEC_PALR(base_addr) = (fp->fecpriv_mac_addr[0] << 24) | (fp->fecpriv_mac_addr[1] << 16) | (fp->fecpriv_mac_addr[2] << 8) | fp->fecpriv_mac_addr[3];
- FEC_PAUR(base_addr) = (fp->fecpriv_mac_addr[4] << 24) | (fp->fecpriv_mac_addr[5] << 16) | 0x8808;
-
- // Reset the group address descriptor
+ /* Reset the group address descriptor */
FEC_GALR(base_addr) = 0x00000000;
FEC_GAUR(base_addr) = 0x00000000;
- // Reset the individual address descriptor
+ /* Reset the individual address descriptor */
FEC_IALR(base_addr) = 0x00000000;
FEC_IAUR(base_addr) = 0x00000000;
- // Set the receive control register
+ /* Set the receive control register */
FEC_RCR(base_addr) = FEC_RCR_MAX_FRM_SIZE | FEC_RCR_MII;
- // Set the receive FIFO control register
+ /* Set the receive FIFO control register */
// FEC_FECRFCR(base_addr) = FEC_FECRFCR_FRM | FEC_FECRFCR_GR | FEC_FECRFCR_MSK;
FEC_FECRFCR(base_addr) = FEC_FECRFCR_FRM | FEC_FECRFCR_GR
| (FEC_FECRFCR_MSK // disable all but ...
@@ -422,10 +406,10 @@ printk(KERN_INFO "FECOPEN3\n");
// & ~FEC_FECRFCR_UF // enable FIFO underflow
);
- //Set the receive FIFO alarm register
+ /* Set the receive FIFO alarm register */
FEC_FECRFAR(base_addr) = FEC_FECRFAR_ALARM;
- // Set the transmit FIFO control register
+ /* Set the transmit FIFO control register */
// FEC_FECTFCR(base_addr) = FEC_FECTFCR_FRM | FEC_FECTFCR_GR | FEC_FECTFCR_MSK;
FEC_FECTFCR(base_addr) = FEC_FECTFCR_FRM | FEC_FECTFCR_GR
| (FEC_FECTFCR_MSK // disable all but ...
@@ -434,16 +418,16 @@ printk(KERN_INFO "FECOPEN3\n");
// & ~FEC_FECTFCR_UF // enable FIFO underflow
& ~FEC_FECTFCR_OF); // enable FIFO overflow
- //Set the transmit FIFO alarm register
+ /* Set the transmit FIFO alarm register */
FEC_FECTFAR(base_addr) = FEC_FECTFAR_ALARM;
- // Set the Tx FIFO watermark
+ /* Set the Tx FIFO watermark */
FEC_FECTFWR(base_addr) = FEC_FECTFWR_XWMRK;
- // Enable the transmitter to append the CRC
+ /* Enable the transmitter to append the CRC */
FEC_CTCWR(base_addr) = FEC_CTCWR_TFCW_CRC;
- // Enable the ethernet interrupts
+ /* Enable the ethernet interrupts */
// FEC_EIMR(base_addr) = FEC_EIMR_MASK;
FEC_EIMR(base_addr) = FEC_EIMR_DISABLE
| FEC_EIR_LC
@@ -453,9 +437,16 @@ printk(KERN_INFO "FECOPEN3\n");
| FEC_EIR_XFERR
| FEC_EIR_RFERR
;
-printk(KERN_INFO "FECOPEN4\n");
-#ifdef CONFIG_FEC_548x_AUTO_NEGOTIATION
+/*
+ * JKM --
+ *
+ * There's a problem with the PHY initialization code --
+ * for now assume uboot left it in an initialized state.
+ */
+// printk(KERN_INFO "FECOPEN: starting auto-negotiation\n");
+// #ifdef CONFIG_FEC_548x_AUTO_NEGOTIATION
+#if 0
if ((error_code = init_transceiver(base_addr, &fduplex)) != 0)
{
printk("Initialization of the transceiver is failed\n");
@@ -464,25 +455,24 @@ printk(KERN_INFO "FECOPEN4\n");
#else
fduplex = 1;
#endif
-printk(KERN_INFO "FECOPEN5\n");
+// printk(KERN_INFO "FECOPEN: done with auto-negotiation\n");
if (fduplex)
- // Enable the full duplex mode
+ /* Enable the full duplex mode */
FEC_TCR(base_addr) = FEC_TCR_FDEN | FEC_TCR_HBC;
else
- // Disable reception of frames while transmitting
+ /* Disable reception of frames while transmitting */
FEC_RCR(base_addr) |= FEC_RCR_DRT;
- // Enable MIB
+ /* Enable MIB */
FEC_MIBC(base_addr) = FEC_MIBC_ENABLE;
- // Enable FEC
+ /* Enable FEC */
FEC_ECR(base_addr) |= FEC_ECR_ETHEREN;
- // Initialize transmission descriptors and start DMA for the transmission
+ /* Initialize tx descriptors and start DMA for the transmission */
for (i = 0; i < FEC_TX_BUF_NUMBER; i++)
fp->fecpriv_txdesc[i].statCtrl = MCD_FEC_INTERRUPT;
-printk(KERN_INFO "FECOPEN6\n");
fp->fecpriv_txdesc[i - 1].statCtrl |= MCD_FEC_WRAP;
@@ -494,31 +484,27 @@ printk(KERN_INFO "FECOPEN6\n");
FEC_TX_DMA_PRI, MCD_FECTX_DMA | MCD_INTERRUPT,
MCD_NO_CSUM | MCD_NO_BYTE_SWAP);
- // Initialize reception descriptors and start DMA for the reception
- for (i = 0; i < FEC_RX_BUF_NUMBER; i++)
- {
+ /* Initialize rx descriptors and start DMA for the reception */
+ for (i = 0; i < FEC_RX_BUF_NUMBER; i++) {
fp->askb_rx[i] = alloc_skb(FEC_MAXBUF_SIZE + 16, GFP_DMA);
- if (!fp->askb_rx[i])
- {
- fp->fecpriv_rxdesc[i].dataPointer = 0;
- fp->fecpriv_rxdesc[i].statCtrl = 0;
- fp->fecpriv_rxdesc[i].length = 0;
+ if (!fp->askb_rx[i]) {
+ fp->fecpriv_rxdesc[i].dataPointer = 0;
+ fp->fecpriv_rxdesc[i].statCtrl = 0;
+ fp->fecpriv_rxdesc[i].length = 0;
}
- else
- {
- skb_reserve(fp->askb_rx[i], 16);
+ else {
+ skb_reserve(fp->askb_rx[i], 16);
fp->askb_rx[i]->dev = dev;
- fp->fecpriv_rxdesc[i].dataPointer = (unsigned int) virt_to_phys(fp->askb_rx[i]->tail);
- fp->fecpriv_rxdesc[i].statCtrl = MCD_FEC_BUF_READY | MCD_FEC_INTERRUPT;
- fp->fecpriv_rxdesc[i].length = FEC_MAXBUF_SIZE;
- }
+ fp->fecpriv_rxdesc[i].dataPointer = (unsigned int) virt_to_phys(fp->askb_rx[i]->tail);
+ fp->fecpriv_rxdesc[i].statCtrl = MCD_FEC_BUF_READY | MCD_FEC_INTERRUPT;
+ fp->fecpriv_rxdesc[i].length = FEC_MAXBUF_SIZE;
+ }
}
-printk(KERN_INFO "FECOPEN7\n");
fp->fecpriv_rxdesc[i - 1].statCtrl |= MCD_FEC_WRAP;
fp->fecpriv_current_rx = 0;
- // flush entire data cache before restarting the DMA
+ /* flush entire data cache before restarting the DMA */
#if 0
/* JKM -- currently running with cache turned off */
DcacheFlushInvalidate();
@@ -531,31 +517,24 @@ printk(KERN_INFO "FECOPEN7\n");
MCD_NO_CSUM | MCD_NO_BYTE_SWAP);
netif_start_queue(dev);
-
-// MOD_INC_USE_COUNT;
-printk(KERN_INFO "FECOPEN: finished\n");
-
return 0;
ERRORS:
- // Remove the channels and return with the error code
- if (fp->fecpriv_fec_rx_channel != -1)
- {
+ /* Remove the channels and return with the error code */
+ if (fp->fecpriv_fec_rx_channel != -1) {
dma_disconnect(fp->fecpriv_fec_rx_channel);
dma_remove_channel_by_number(fp->fecpriv_fec_rx_channel);
fp->fecpriv_fec_rx_channel = -1;
}
- if (fp->fecpriv_fec_tx_channel != -1)
- {
+ if (fp->fecpriv_fec_tx_channel != -1) {
dma_disconnect(fp->fecpriv_fec_tx_channel);
dma_remove_channel_by_number(fp->fecpriv_fec_tx_channel);
fp->fecpriv_fec_tx_channel = -1;
}
return error_code;
-
}
/************************************************************************
@@ -568,31 +547,26 @@ ERRORS:
*************************************************************************/
int fec_close(struct net_device *dev)
{
- //Receive the pointer to the private structure
struct fec_priv *fp = netdev_priv(dev);
-
- // Receive the base address
unsigned long base_addr = (unsigned long) dev->base_addr;
-
unsigned long time;
-
int i;
netif_stop_queue(dev);
- // Perform the graceful stop
+ /* Perform the graceful stop */
FEC_TCR(base_addr) |= FEC_TCR_GTS;
time = jiffies;
- // Wait for the graceful stop
+ /* Wait for the graceful stop */
while (!(FEC_EIR(base_addr) & FEC_EIR_GRA) && jiffies - time < FEC_GR_TIMEOUT * HZ)
schedule();
- // Disable FEC
+ /* Disable FEC */
FEC_ECR(base_addr) = FEC_ECR_DISABLE;
- // Reset the DMA channels
+ /* Reset the DMA channels */
spin_lock_irq(&fp->fecpriv_lock);
MCD_killDma(fp->fecpriv_fec_tx_channel);
spin_unlock_irq(&fp->fecpriv_lock);
@@ -600,12 +574,12 @@ int fec_close(struct net_device *dev)
dma_disconnect(fp->fecpriv_fec_tx_channel);
fp->fecpriv_fec_tx_channel = -1;
- for (i = 0; i < FEC_TX_BUF_NUMBER; i++)
- if (fp->fecpriv_txbuf[i])
- {
+ for (i = 0; i < FEC_TX_BUF_NUMBER; i++) {
+ if (fp->fecpriv_txbuf[i]) {
kfree(fp->fecpriv_txbuf[i]);
fp->fecpriv_txbuf[i] = NULL;
}
+ }
spin_lock_irq(&fp->fecpriv_lock);
MCD_killDma(fp->fecpriv_fec_rx_channel);
@@ -615,15 +589,12 @@ int fec_close(struct net_device *dev)
dma_disconnect(fp->fecpriv_fec_rx_channel);
fp->fecpriv_fec_rx_channel = -1;
- for (i = 0; i < FEC_RX_BUF_NUMBER; i++)
- {
- if (fp->askb_rx[i])
- {
- kfree_skb(fp->askb_rx[i]);
- fp->askb_rx[i] = NULL;
- }
+ for (i = 0; i < FEC_RX_BUF_NUMBER; i++) {
+ if (fp->askb_rx[i]) {
+ kfree_skb(fp->askb_rx[i]);
+ fp->askb_rx[i] = NULL;
+ }
}
-// MOD_DEC_USE_COUNT;
return 0;
}
@@ -635,14 +606,10 @@ int fec_close(struct net_device *dev)
*************************************************************************/
struct net_device_stats * fec_get_stat(struct net_device *dev)
{
-
- //Receive the pointer to the private structure
struct fec_priv *fp = netdev_priv(dev);
-
- // Receive the base address
unsigned long base_addr = dev->base_addr;
- // Receive the statistical information
+ /* Receive the statistical information */
fp->fecpriv_stat.rx_packets = FECSTAT_RMON_R_PACKETS(base_addr);
fp->fecpriv_stat.tx_packets = FECSTAT_RMON_T_PACKETS(base_addr);
fp->fecpriv_stat.rx_bytes = FECSTAT_RMON_R_OCTETS(base_addr);
@@ -651,7 +618,10 @@ struct net_device_stats * fec_get_stat(s
fp->fecpriv_stat.multicast = FECSTAT_RMON_R_MC_PKT(base_addr);
fp->fecpriv_stat.collisions = FECSTAT_RMON_T_COL(base_addr);
- fp->fecpriv_stat.rx_length_errors = FECSTAT_RMON_R_UNDERSIZE(base_addr) + FECSTAT_RMON_R_OVERSIZE(base_addr) + FECSTAT_RMON_R_FRAG(base_addr) + FECSTAT_RMON_R_JAB(base_addr);
+ fp->fecpriv_stat.rx_length_errors = FECSTAT_RMON_R_UNDERSIZE(base_addr) +
+ FECSTAT_RMON_R_OVERSIZE(base_addr) +
+ FECSTAT_RMON_R_FRAG(base_addr) +
+ FECSTAT_RMON_R_JAB(base_addr);
fp->fecpriv_stat.rx_crc_errors = FECSTAT_IEEE_R_CRC(base_addr);
fp->fecpriv_stat.rx_frame_errors = FECSTAT_IEEE_R_ALIGN(base_addr);
fp->fecpriv_stat.rx_over_errors = FECSTAT_IEEE_R_MACERR(base_addr);
@@ -660,9 +630,18 @@ struct net_device_stats * fec_get_stat(s
fp->fecpriv_stat.tx_fifo_errors = FECSTAT_IEEE_T_MACERR(base_addr);
fp->fecpriv_stat.tx_window_errors = FECSTAT_IEEE_T_LCOL(base_addr);
- // I hope that one frame doesn't have more than one error
- fp->fecpriv_stat.rx_errors = fp->fecpriv_stat.rx_length_errors + fp->fecpriv_stat.rx_crc_errors + fp->fecpriv_stat.rx_frame_errors + fp->fecpriv_stat.rx_over_errors + fp->fecpriv_stat.rx_dropped;
- fp->fecpriv_stat.tx_errors = fp->fecpriv_stat.tx_carrier_errors + fp->fecpriv_stat.tx_fifo_errors + fp->fecpriv_stat.tx_window_errors + fp->fecpriv_stat.tx_aborted_errors + fp->fecpriv_stat.tx_heartbeat_errors + fp->fecpriv_stat.tx_dropped;
+ /* I hope that one frame doesn't have more than one error */
+ fp->fecpriv_stat.rx_errors = fp->fecpriv_stat.rx_length_errors +
+ fp->fecpriv_stat.rx_crc_errors +
+ fp->fecpriv_stat.rx_frame_errors +
+ fp->fecpriv_stat.rx_over_errors +
+ fp->fecpriv_stat.rx_dropped;
+ fp->fecpriv_stat.tx_errors = fp->fecpriv_stat.tx_carrier_errors +
+ fp->fecpriv_stat.tx_fifo_errors +
+ fp->fecpriv_stat.tx_window_errors +
+ fp->fecpriv_stat.tx_aborted_errors +
+ fp->fecpriv_stat.tx_heartbeat_errors +
+ fp->fecpriv_stat.tx_dropped;
return &fp->fecpriv_stat;
}
@@ -674,59 +653,47 @@ struct net_device_stats * fec_get_stat(s
*************************************************************************/
void fec_set_multicast_list(struct net_device *dev)
{
- // Pointer to the address list
struct dev_mc_list *dmi;
-
unsigned int crc, data;
int i, j, k;
-
- // Receive the base address
unsigned long base_addr = (unsigned long) dev->base_addr;
- if (dev->flags & IFF_PROMISC || dev->flags & IFF_ALLMULTI)
- {
- // Allow all incoming frames
+ if (dev->flags & IFF_PROMISC || dev->flags & IFF_ALLMULTI) {
+ /* Allow all incoming frames */
FEC_GALR(base_addr) = 0xFFFFFFFF;
FEC_GAUR(base_addr) = 0xFFFFFFFF;
return;
}
- // Reset the group address register
+
+ /* Reset the group address register */
FEC_GALR(base_addr) = 0x00000000;
FEC_GAUR(base_addr) = 0x00000000;
- // Process all addresses
- for (i = 0, dmi = dev->mc_list; i < dev->mc_count; i++, dmi = dmi->next)
- {
- // Processing must be only for the group addresses
+ /* Process all addresses */
+ for (i = 0, dmi = dev->mc_list; i < dev->mc_count; i++, dmi = dmi->next) {
+ /* Processing must be only for the group addresses */
if (!(dmi->dmi_addr[0] & 1))
continue;
- // Calculate crc value for the current address
+ /* Calculate crc value for the current address */
crc = 0xFFFFFFFF;
- for (j = 0; j < dmi->dmi_addrlen; j++)
- {
-
- for (k = 0, data = dmi->dmi_addr[j]; k < 8; k++, data >>= 1)
- {
+ for (j = 0; j < dmi->dmi_addrlen; j++) {
+ for (k = 0, data = dmi->dmi_addr[j]; k < 8; k++, data >>= 1) {
if ((crc ^ data) & 1)
crc = (crc >> 1) ^ FEC_CRCPOL;
else
crc >>= 1;
-
}
-
}
- // Add this value
+ /* Add this value */
crc >>= 26;
crc &= 0x3F;
if (crc > 31)
FEC_GAUR(base_addr) |= 0x1 << (crc - 32);
else
FEC_GALR(base_addr) |= 0x1 << crc;
-
}
-
}
/************************************************************************
@@ -736,26 +703,27 @@ void fec_set_multicast_list(struct net_d
*************************************************************************/
int fec_set_mac_address(struct net_device *dev, void *p)
{
- //Receive the pointer to the private structure
struct fec_priv *fp = netdev_priv(dev);
-
- // Receive the base address
unsigned long base_addr = (unsigned long) dev->base_addr;
-
struct sockaddr *addr = p;
if (netif_running(dev))
return -EBUSY;
- // Copy a new address to the device structure
+ /* Copy a new address to the device structure */
memcpy(dev->dev_addr, addr->sa_data, dev->addr_len);
- // Copy a new address to the private structure
+ /* Copy a new address to the private structure */
memcpy(fp->fecpriv_mac_addr, addr->sa_data, 6);
- // Set the address to the registers
- FEC_PALR(base_addr) = (fp->fecpriv_mac_addr[0] << 24) | (fp->fecpriv_mac_addr[1] << 16) | (fp->fecpriv_mac_addr[2] << 8) | fp->fecpriv_mac_addr[3];
- FEC_PAUR(base_addr) = (fp->fecpriv_mac_addr[4] << 24) | (fp->fecpriv_mac_addr[5] << 16) | 0x8808;
+ /* Set the address to the registers */
+ FEC_PALR(base_addr) = (fp->fecpriv_mac_addr[0] << 24) |
+ (fp->fecpriv_mac_addr[1] << 16) |
+ (fp->fecpriv_mac_addr[2] << 8) |
+ fp->fecpriv_mac_addr[3];
+ FEC_PAUR(base_addr) = (fp->fecpriv_mac_addr[4] << 24) |
+ (fp->fecpriv_mac_addr[5] << 16) |
+ 0x8808;
return 0;
}
@@ -769,27 +737,24 @@ int fec_set_mac_address(struct net_devic
*************************************************************************/
int fec_tx(struct sk_buff *skb, struct net_device *dev)
{
-
- //Receive the pointer to the private structure
struct fec_priv *fp = netdev_priv(dev);
-
void *data, *data_aligned;
int offset;
data = kmalloc(skb->len + 15, GFP_DMA | GFP_ATOMIC);
- if (!data)
- {
+ if (!data) {
fp->fecpriv_stat.tx_dropped++;
dev_kfree_skb(skb);
return 0;
}
- offset = (((unsigned long)virt_to_phys(data) + 15) & 0xFFFFFFF0) - (unsigned long)virt_to_phys(data);
+ offset = (((unsigned long)virt_to_phys(data) + 15) & 0xFFFFFFF0) -
+ (unsigned long)virt_to_phys(data);
data_aligned = (void*)((unsigned long)data + offset);
memcpy(data_aligned, skb->data, skb->len);
- // flush data cache before initializing the descriptor and starting DMA
+ /* flush data cache before initializing the descriptor and starting DMA */
#if 0
/* JKM -- currently running with cache turned off */
DcacheFlushInvalidateCacheBlock((void*)virt_to_phys(data_aligned), skb->len);
@@ -797,7 +762,7 @@ int fec_tx(struct sk_buff *skb, struct n
spin_lock_irq(&fp->fecpriv_lock);
- // Initialize the descriptor
+ /* Initialize the descriptor */
fp->fecpriv_txbuf[fp->fecpriv_next_tx] = data;
fp->fecpriv_txdesc[fp->fecpriv_next_tx].dataPointer = (unsigned int) virt_to_phys(data_aligned);
fp->fecpriv_txdesc[fp->fecpriv_next_tx].length = skb->len;
@@ -809,7 +774,7 @@ int fec_tx(struct sk_buff *skb, struct n
spin_unlock_irq(&fp->fecpriv_lock);
- // Tell the DMA to continue the transmission
+ /* Tell the DMA to continue the transmission */
MCD_continDma(fp->fecpriv_fec_tx_channel);
dev_kfree_skb(skb);
@@ -835,14 +800,10 @@ void fec_tx_timeout(struct net_device *d
spin_lock_irq(&fp->fecpriv_lock);
MCD_killDma(fp->fecpriv_fec_tx_channel);
- for (i = 0; i < FEC_TX_BUF_NUMBER; i++)
- {
- if (fp->fecpriv_txbuf[i])
- {
-
+ for (i = 0; i < FEC_TX_BUF_NUMBER; i++) {
+ if (fp->fecpriv_txbuf[i]) {
kfree(fp->fecpriv_txbuf[i]);
fp->fecpriv_txbuf[i] = NULL;
-
}
fp->fecpriv_txdesc[i].statCtrl = MCD_FEC_INTERRUPT;
}
@@ -850,14 +811,14 @@ void fec_tx_timeout(struct net_device *d
fp->fecpriv_current_tx = fp->fecpriv_next_tx = 0;
- // Reset FIFOs
+ /* Reset FIFOs */
FEC_FECFRST(base_addr) |= FEC_SW_RST;
FEC_FECFRST(base_addr) &= ~FEC_SW_RST;
- // Reset and disable FEC
+ /* Reset and disable FEC */
// FEC_ECR(base_addr) = FEC_ECR_RESET;
- // Enable FEC
+ /* Enable FEC */
FEC_ECR(base_addr) |= FEC_ECR_ETHEREN;
MCD_startDma(fp->fecpriv_fec_tx_channel, (char *) fp->fecpriv_txdesc, 0,
@@ -883,23 +844,22 @@ int fec_read_mii(unsigned int base_addr,
{
unsigned long time;
- // Clear the MII interrupt bit
+ /* Clear the MII interrupt bit */
FEC_EIR(base_addr) = FEC_EIR_MII;
- // Write to the MII management frame register
+ /* Write to the MII management frame register */
FEC_MMFR(base_addr) = FEC_MMFR_READ | (pa << 23) | (ra << 18);
time = jiffies;
- // Wait for the reading
- while (!(FEC_EIR(base_addr) & FEC_EIR_MII))
- {
+ /* Wait for the reading */
+ while (!(FEC_EIR(base_addr) & FEC_EIR_MII)) {
if (jiffies - time > FEC_MII_TIMEOUT * HZ)
return -ETIME;
schedule();
}
- // Clear the MII interrupt bit
+ /* Clear the MII interrupt bit */
FEC_EIR(base_addr) = FEC_EIR_MII;
*data = FEC_MMFR(base_addr) & 0x0000FFFF;
@@ -918,25 +878,22 @@ int fec_write_mii(unsigned int base_addr
{
unsigned long time;
- // Clear the MII interrupt bit
+ /* Clear the MII interrupt bit */
FEC_EIR(base_addr) = FEC_EIR_MII;
- // Write to the MII management frame register
+ /* Write to the MII management frame register */
FEC_MMFR(base_addr) = FEC_MMFR_WRITE | (pa << 23) | (ra << 18) | data;
time = jiffies;
- // Wait for the writing
-
- while (!(FEC_EIR(base_addr) & FEC_EIR_MII))
- {
+ /* Wait for the writing */
+ while (!(FEC_EIR(base_addr) & FEC_EIR_MII)) {
if (jiffies - time > FEC_MII_TIMEOUT * HZ)
return -ETIME;
-
schedule();
}
- // Clear the MII interrupt bit
+ /* Clear the MII interrupt bit */
FEC_EIR(base_addr) = FEC_EIR_MII;
return 0;
@@ -953,27 +910,24 @@ void fec_interrupt_fec_tx_handler(struct
{
struct fec_priv *fp = netdev_priv(dev);
- //Release the socket buffer
- if(fp->fecpriv_txbuf[fp->fecpriv_current_tx])
- {
+ /* Release the socket buffer */
+ if(fp->fecpriv_txbuf[fp->fecpriv_current_tx]) {
kfree(fp->fecpriv_txbuf[fp->fecpriv_current_tx]);
fp->fecpriv_txbuf[fp->fecpriv_current_tx] = NULL;
}
fp->fecpriv_current_tx = (fp->fecpriv_current_tx + 1) & FEC_TX_INDEX_MASK;
- if (MCD_dmaStatus(fp->fecpriv_fec_tx_channel) == MCD_DONE)
- for (; fp->fecpriv_current_tx != fp->fecpriv_next_tx; fp->fecpriv_current_tx = (fp->fecpriv_current_tx + 1) & FEC_TX_INDEX_MASK)
- {
- if(fp->fecpriv_txbuf[fp->fecpriv_current_tx])
- {
+ if (MCD_dmaStatus(fp->fecpriv_fec_tx_channel) == MCD_DONE) {
+ for (; fp->fecpriv_current_tx != fp->fecpriv_next_tx; fp->fecpriv_current_tx = (fp->fecpriv_current_tx + 1) & FEC_TX_INDEX_MASK) {
+ if(fp->fecpriv_txbuf[fp->fecpriv_current_tx]) {
kfree(fp->fecpriv_txbuf[fp->fecpriv_current_tx]);
fp->fecpriv_txbuf[fp->fecpriv_current_tx] = NULL;
}
}
+ }
if (netif_queue_stopped(dev))
netif_wake_queue(dev);
-
}
/************************************************************************
@@ -1008,49 +962,41 @@ void fec_interrupt_fec_rx_handler(struct
}
}
*/
- for (; fp->fecpriv_rxdesc[fp->fecpriv_current_rx].statCtrl & MCD_FEC_END_FRAME; fp->fecpriv_current_rx = (fp->fecpriv_current_rx + 1) & FEC_RX_INDEX_MASK)
- {
+ for (; fp->fecpriv_rxdesc[fp->fecpriv_current_rx].statCtrl & MCD_FEC_END_FRAME; fp->fecpriv_current_rx = (fp->fecpriv_current_rx + 1) & FEC_RX_INDEX_MASK) {
if( (fp->fecpriv_rxdesc[fp->fecpriv_current_rx].length <= FEC_MAXBUF_SIZE) &&
- (fp->fecpriv_rxdesc[fp->fecpriv_current_rx].length > 4)) // --tym--
- {
+ (fp->fecpriv_rxdesc[fp->fecpriv_current_rx].length > 4)) { /* --tym-- */
skb = fp->askb_rx[fp->fecpriv_current_rx];
if (!skb)
- {
- fp->fecpriv_stat.rx_dropped++;
- }
- else
- {
- // flush data cache before initializing the descriptor and starting DMA
+ fp->fecpriv_stat.rx_dropped++;
+ else {
+ /* flush data cache before initializing the descriptor and starting DMA */
// DcacheFlushInvalidateCacheBlock((void*)virt_to_phys(fp->askb_rx[fp->fecpriv_current_rx]->tail), fp->askb_rx[fp->fecpriv_current_rx]->len);
skb_put(skb, fp->fecpriv_rxdesc[fp->fecpriv_current_rx].length - 4);
-// skb->dev = dev;
skb->protocol = eth_type_trans(skb, dev);
netif_rx(skb);
- }
- fp->fecpriv_rxdesc[fp->fecpriv_current_rx].statCtrl &= ~MCD_FEC_END_FRAME;
- // allocate new skbuff
+ }
+ fp->fecpriv_rxdesc[fp->fecpriv_current_rx].statCtrl &= ~MCD_FEC_END_FRAME;
+ /* allocate new skbuff */
fp->askb_rx[fp->fecpriv_current_rx] = alloc_skb(FEC_MAXBUF_SIZE + 16, /*GFP_ATOMIC |*/ GFP_DMA);
- if (!fp->askb_rx[fp->fecpriv_current_rx])
- {
- fp->fecpriv_rxdesc[fp->fecpriv_current_rx].dataPointer = 0;
- fp->fecpriv_rxdesc[fp->fecpriv_current_rx].length = 0;
- fp->fecpriv_stat.rx_dropped++;
- }
- else
- {
- skb_reserve(fp->askb_rx[fp->fecpriv_current_rx], 16);
- fp->askb_rx[fp->fecpriv_current_rx]->dev = dev;
+ if (!fp->askb_rx[fp->fecpriv_current_rx]) {
+ fp->fecpriv_rxdesc[fp->fecpriv_current_rx].dataPointer = 0;
+ fp->fecpriv_rxdesc[fp->fecpriv_current_rx].length = 0;
+ fp->fecpriv_stat.rx_dropped++;
+ }
+ else {
+ skb_reserve(fp->askb_rx[fp->fecpriv_current_rx], 16);
+ fp->askb_rx[fp->fecpriv_current_rx]->dev = dev;
- // flush data cache before initializing the descriptor and starting DMA
+ /* flush data cache before initializing the descriptor and starting DMA */
#if 0
/* JKM -- currently running with cache turned off */
- DcacheFlushInvalidateCacheBlock((void*)virt_to_phys(fp->askb_rx[fp->fecpriv_current_rx]->tail), FEC_MAXBUF_SIZE);
+ DcacheFlushInvalidateCacheBlock((void*)virt_to_phys(fp->askb_rx[fp->fecpriv_current_rx]->tail), FEC_MAXBUF_SIZE);
#endif
fp->fecpriv_rxdesc[fp->fecpriv_current_rx].dataPointer = (unsigned int) virt_to_phys(fp->askb_rx[fp->fecpriv_current_rx]->tail);
fp->fecpriv_rxdesc[fp->fecpriv_current_rx].length = FEC_MAXBUF_SIZE;
- fp->fecpriv_rxdesc[fp->fecpriv_current_rx].statCtrl |= MCD_FEC_BUF_READY;
+ fp->fecpriv_rxdesc[fp->fecpriv_current_rx].statCtrl |= MCD_FEC_BUF_READY;
// flush data cache before initializing the descriptor and starting DMA
// DcacheFlushInvalidateCacheBlock((void*)virt_to_phys(fp->askb_rx[fp->fecpriv_current_rx]->tail), FEC_MAXBUF_SIZE);
@@ -1059,11 +1005,10 @@ void fec_interrupt_fec_rx_handler(struct
}
- // Tell the DMA to continue the reception
+ /* Tell the DMA to continue the reception */
MCD_continDma(fp->fecpriv_fec_rx_channel);
fp->fecpriv_rxflag = 0;
-
}
/************************************************************************
@@ -1080,76 +1025,70 @@ irqreturn_t fec_interrupt_handler(int ir
unsigned long base_addr = (unsigned long) dev->base_addr;
unsigned long events;
- // Read and clear the events
+ /* Read and clear the events */
events = FEC_EIR(base_addr) & FEC_EIMR(base_addr);
- if (events & FEC_EIR_HBERR)
- {
+ if (events & FEC_EIR_HBERR) {
fp->fecpriv_stat.tx_heartbeat_errors++;
- FEC_EIR(base_addr) = FEC_EIR_HBERR;
+ FEC_EIR(base_addr) = FEC_EIR_HBERR;
}
- // receive/transmit FIFO error
- if (((events & FEC_EIR_RFERR) != 0) || ((events & FEC_EIR_XFERR) != 0))
- {
- // kill DMA receive channel
- MCD_killDma (fp->fecpriv_fec_rx_channel);
- // kill running transmission by DMA
- MCD_killDma (fp->fecpriv_fec_tx_channel);
-
- // Reset FIFOs
- FEC_FECFRST(base_addr) |= FEC_SW_RST;
- FEC_FECFRST(base_addr) &= ~FEC_SW_RST;
-
- // reset receive FIFO status register
- FEC_FECRFSR(base_addr) = FEC_FECRFSR_FAE
- | FEC_FECRFSR_RXW
- | FEC_FECRFSR_UF;
-
- // reset transmit FIFO status register
- FEC_FECTFSR(base_addr) = FEC_FECTFSR_FAE
- | FEC_FECTFSR_TXW
- | FEC_FECTFSR_UF
- | FEC_FECTFSR_OF;
+ /* receive/transmit FIFO error */
+ if (((events & FEC_EIR_RFERR) != 0) || ((events & FEC_EIR_XFERR) != 0)) {
+ /* kill DMA receive channel */
+ MCD_killDma (fp->fecpriv_fec_rx_channel);
- // reset RFERR and XFERR event
- FEC_EIR(base_addr) = FEC_EIR_RFERR | FEC_EIR_XFERR;
+ /* kill running transmission by DMA */
+ MCD_killDma (fp->fecpriv_fec_tx_channel);
- // stop queue
- netif_stop_queue(dev);
-
- // execute reinitialization as tasklet
- tasklet_schedule(&fp->fecpriv_tasklet_reinit);
+ /* Reset FIFOs */
+ FEC_FECFRST(base_addr) |= FEC_SW_RST;
+ FEC_FECFRST(base_addr) &= ~FEC_SW_RST;
- fp->fecpriv_stat.rx_dropped++;
+ /* reset receive FIFO status register */
+ FEC_FECRFSR(base_addr) = FEC_FECRFSR_FAE |
+ FEC_FECRFSR_RXW |
+ FEC_FECRFSR_UF;
- }
+ /* reset transmit FIFO status register */
+ FEC_FECTFSR(base_addr) = FEC_FECTFSR_FAE |
+ FEC_FECTFSR_TXW |
+ FEC_FECTFSR_UF |
+ FEC_FECTFSR_OF;
- // transmit FIFO underrun
- if ((events & FEC_EIR_XFUN) != 0)
- {
- // reset XFUN event
- FEC_EIR(base_addr) = FEC_EIR_XFUN;
- fp->fecpriv_stat.tx_aborted_errors++;
- }
+ /* reset RFERR and XFERR event */
+ FEC_EIR(base_addr) = FEC_EIR_RFERR | FEC_EIR_XFERR;
+
+ /* stop queue */
+ netif_stop_queue(dev);
+
+ /* execute reinitialization as tasklet */
+ tasklet_schedule(&fp->fecpriv_tasklet_reinit);
+
+ fp->fecpriv_stat.rx_dropped++;
+ }
- // late collision
- if ((events & FEC_EIR_LC) != 0)
- {
- // reset LC event
- FEC_EIR(base_addr) = FEC_EIR_LC;
+ /* transmit FIFO underrun */
+ if ((events & FEC_EIR_XFUN) != 0) {
+ /* reset XFUN event */
+ FEC_EIR(base_addr) = FEC_EIR_XFUN;
fp->fecpriv_stat.tx_aborted_errors++;
- }
+ }
- // collision retry limit
- if ((events & FEC_EIR_RL) != 0)
- {
- // reset RL event
- FEC_EIR(base_addr) = FEC_EIR_RL;
+ /* late collision */
+ if ((events & FEC_EIR_LC) != 0) {
+ /* reset LC event */
+ FEC_EIR(base_addr) = FEC_EIR_LC;
fp->fecpriv_stat.tx_aborted_errors++;
- }
+ }
- return 0;
+ /* collision retry limit */
+ if ((events & FEC_EIR_RL) != 0) {
+ /* reset RL event */
+ FEC_EIR(base_addr) = FEC_EIR_RL;
+ fp->fecpriv_stat.tx_aborted_errors++;
+ }
+ return 0;
}
/************************************************************************
@@ -1166,52 +1105,46 @@ void fec_interrupt_fec_reinit(unsigned l
struct fec_priv *fp = netdev_priv(dev);
unsigned long base_addr = (unsigned long) dev->base_addr;
- // Initialize reception descriptors and start DMA for the reception
- for (i = 0; i < FEC_RX_BUF_NUMBER; i++)
- {
- if (!fp->askb_rx[i])
- {
- fp->askb_rx[i] = alloc_skb(FEC_MAXBUF_SIZE + 16, GFP_ATOMIC | GFP_DMA);
- if (!fp->askb_rx[i])
- {
- fp->fecpriv_rxdesc[i].dataPointer = 0;
- fp->fecpriv_rxdesc[i].statCtrl = 0;
- fp->fecpriv_rxdesc[i].length = 0;
- continue;
- }
+ /* Initialize reception descriptors and start DMA for the reception */
+ for (i = 0; i < FEC_RX_BUF_NUMBER; i++) {
+ if (!fp->askb_rx[i]) {
+ fp->askb_rx[i] = alloc_skb(FEC_MAXBUF_SIZE + 16, GFP_ATOMIC | GFP_DMA);
+ if (!fp->askb_rx[i]) {
+ fp->fecpriv_rxdesc[i].dataPointer = 0;
+ fp->fecpriv_rxdesc[i].statCtrl = 0;
+ fp->fecpriv_rxdesc[i].length = 0;
+ continue;
+ }
fp->askb_rx[i]->dev = dev;
- skb_reserve(fp->askb_rx[i], 16);
- }
- fp->fecpriv_rxdesc[i].dataPointer = (unsigned int) virt_to_phys(fp->askb_rx[i]->tail);
+ skb_reserve(fp->askb_rx[i], 16);
+ }
+ fp->fecpriv_rxdesc[i].dataPointer = (unsigned int) virt_to_phys(fp->askb_rx[i]->tail);
fp->fecpriv_rxdesc[i].statCtrl = MCD_FEC_BUF_READY | MCD_FEC_INTERRUPT;
- fp->fecpriv_rxdesc[i].length = FEC_MAXBUF_SIZE;
+ fp->fecpriv_rxdesc[i].length = FEC_MAXBUF_SIZE;
}
fp->fecpriv_rxdesc[i - 1].statCtrl |= MCD_FEC_WRAP;
fp->fecpriv_current_rx = 0;
- // restart frame transmission
- for (i = 0; i < FEC_TX_BUF_NUMBER; i++)
- {
- if (fp->fecpriv_txbuf[i])
- {
-
+ /* restart frame transmission */
+ for (i = 0; i < FEC_TX_BUF_NUMBER; i++) {
+ if (fp->fecpriv_txbuf[i]) {
kfree(fp->fecpriv_txbuf[i]);
fp->fecpriv_txbuf[i] = NULL;
- fp->fecpriv_stat.tx_dropped++;
+ fp->fecpriv_stat.tx_dropped++;
}
fp->fecpriv_txdesc[i].statCtrl = MCD_FEC_INTERRUPT;
}
fp->fecpriv_txdesc[i - 1].statCtrl |= MCD_FEC_WRAP;
fp->fecpriv_current_tx = fp->fecpriv_next_tx = 0;
- // flush entire data cache before restarting the DMA
+ /* flush entire data cache before restarting the DMA */
#if 0
/* JKM -- currently running with cache turned off */
DcacheFlushInvalidate();
#endif
- // restart DMA from beginning
+ /* restart DMA from beginning */
MCD_startDma(fp->fecpriv_fec_rx_channel,
(char *) fp->fecpriv_rxdesc, 0,
(unsigned char *) &(FEC_FECRFDR(base_addr)), 0,
@@ -1225,11 +1158,10 @@ void fec_interrupt_fec_reinit(unsigned l
FEC_TX_DMA_PRI, MCD_FECTX_DMA | MCD_INTERRUPT,
MCD_NO_CSUM | MCD_NO_BYTE_SWAP);
- // Enable FEC
- FEC_ECR(base_addr) |= FEC_ECR_ETHEREN;
+ /* Enable FEC */
+ FEC_ECR(base_addr) |= FEC_ECR_ETHEREN;
netif_wake_queue(dev);
-
}
/************************************************************************
@@ -1284,8 +1216,8 @@ void fec_interrupt_fec_rx_handler_fec1(v
}
#endif
-#ifndef MODULE
+#ifndef MODULE
/************************************************************************
* NAME: fec_mac_setup0
*
@@ -1300,7 +1232,6 @@ int __init fec_mac_setup0(char *s)
if(fec_str_to_mac(s, fec_mac_addr_fec0))
printk("The MAC address of FEC0 cannot be set from command line");
return 1;
-
}
#ifdef FEC_2
@@ -1316,11 +1247,9 @@ int __init fec_mac_setup1(char *s)
if(!s || !*s)
return 1;
-
if(fec_str_to_mac(s, fec_mac_addr_fec1))
printk("The MAC address of FEC1 cannot be set from command line");
return 1;
-
}
#endif
@@ -1332,52 +1261,46 @@ int __init fec_mac_setup1(char *s)
*************************************************************************/
int fec_str_to_mac( char *str_mac, unsigned char* addr)
{
- unsigned long val;
+ unsigned long val;
char c;
unsigned long octet[6], *octetptr = octet;
int i;
-again:
+again:
val = 0;
- while ((c = *str_mac) != '\0')
- {
- if ((c>='0')&&(c<='9'))
- {
+ while ((c = *str_mac) != '\0') {
+ if ((c>='0')&&(c<='9')) {
val = (val * 16) + (c - '0');
str_mac++;
continue;
}
- else
- if (((c>='a')&&(c<='f'))||((c>='A')&&(c<='F')))
- {
+ else if (((c>='a')&&(c<='f'))||((c>='A')&&(c<='F'))) {
val = (val << 4) + (c + 10 - (((c>='a')&&(c<='f')) ? 'a' : 'A'));
str_mac++;
continue;
}
break;
}
- if (*str_mac == ':')
- {
+ if (*str_mac == ':') {
*octetptr++ = val, str_mac++;
if (octetptr >= octet + 6)
return 1;
goto again;
}
- //Check for trailing characters.
+ /* Check for trailing characters */
if (*str_mac && !(*str_mac==' '))
return 1;
+
*octetptr++ = val;
- if ((octetptr - octet)==6)
- {
- for(i=0;i<=6;i++)
- addr[i]=octet[i];
+ if ((octetptr - octet)==6) {
+ for(i=0;i<=6;i++)
+ addr[i]=octet[i];
}
- else
+ else
return 1;
return 0;
-
}
#endif