]> git.sur5r.net Git - u-boot/blobdiff - drivers/net/xilinx_emac.c
AT91SAM9263EK support
[u-boot] / drivers / net / xilinx_emac.c
index d44f31e8aff1140cd3a4f67a7a4fbeac737cbff7..c7f1a2a8d7dd4816a7c5dd1557b78ac8ee3b2f51 100644 (file)
-/*
- * (C) Copyright 2007 Michal Simek
- *
- * Michal SIMEK <monstr@monstr.eu>
- *
- * See file CREDITS for list of people who contributed to this
- * project.
+/******************************************************************************
  *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
+ * XILINX IS PROVIDING THIS DESIGN, CODE, OR INFORMATION "AS IS"
+ * AS A COURTESY TO YOU, SOLELY FOR USE IN DEVELOPING PROGRAMS AND
+ * SOLUTIONS FOR XILINX DEVICES. BY PROVIDING THIS DESIGN, CODE,
+ * OR INFORMATION AS ONE POSSIBLE IMPLEMENTATION OF THIS FEATURE,
+ * APPLICATION OR STANDARD, XILINX IS MAKING NO REPRESENTATION
+ * THAT THIS IMPLEMENTATION IS FREE FROM ANY CLAIMS OF INFRINGEMENT,
+ * AND YOU ARE RESPONSIBLE FOR OBTAINING ANY RIGHTS YOU MAY REQUIRE
+ * FOR YOUR IMPLEMENTATION. XILINX EXPRESSLY DISCLAIMS ANY
+ * WARRANTY WHATSOEVER WITH RESPECT TO THE ADEQUACY OF THE
+ * IMPLEMENTATION, INCLUDING BUT NOT LIMITED TO ANY WARRANTIES OR
+ * REPRESENTATIONS THAT THIS IMPLEMENTATION IS FREE FROM CLAIMS OF
+ * INFRINGEMENT, IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+ * FOR A PARTICULAR PURPOSE.
  *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
+ * (C) Copyright 2007-2008 Michal Simek
+ * Michal SIMEK <monstr@monstr.eu>
  *
- * Based on Xilinx drivers
+ * (c) Copyright 2003 Xilinx Inc.
+ * All rights reserved.
  *
- */
+ ******************************************************************************/
 
 #include <config.h>
 #include <common.h>
 #include <net.h>
 #include <asm/io.h>
-#include <asm/asm.h>
-#include "xilinx_emac.h"
 
-#ifdef XILINX_EMAC
+#include <asm/asm.h>
 
 #undef DEBUG
 
+typedef struct {
+       u32 regbaseaddress;     /* Base address of registers */
+       u32 databaseaddress;    /* Base address of data for FIFOs */
+} xpacketfifov100b;
+
+typedef struct {
+       u32 baseaddress;        /* Base address (of IPIF) */
+       u32 isstarted;          /* Device is currently started 0-no, 1-yes */
+       xpacketfifov100b recvfifo;      /* FIFO used to receive frames */
+       xpacketfifov100b sendfifo;      /* FIFO used to send frames */
+} xemac;
+
+#define XIIF_V123B_IISR_OFFSET 32UL /* IP interrupt status register */
+#define XIIF_V123B_RESET_MASK          0xAUL
+#define XIIF_V123B_RESETR_OFFSET       64UL /* reset register */
+
+/* This constant is used with the Reset Register */
+#define XPF_RESET_FIFO_MASK            0x0000000A
+#define XPF_COUNT_STATUS_REG_OFFSET    4UL
+
+/* These constants are used with the Occupancy/Vacancy Count Register. This
+ * register also contains FIFO status */
+#define XPF_COUNT_MASK                 0x0000FFFF
+#define XPF_DEADLOCK_MASK              0x20000000
+
+/* Offset of the MAC registers from the IPIF base address */
+#define XEM_REG_OFFSET         0x1100UL
+
+/*
+ * Register offsets for the Ethernet MAC. Each register is 32 bits.
+ */
+#define XEM_ECR_OFFSET (XEM_REG_OFFSET + 0x4)  /* MAC Control */
+#define XEM_SAH_OFFSET (XEM_REG_OFFSET + 0xC)  /* Station addr, high */
+#define XEM_SAL_OFFSET (XEM_REG_OFFSET + 0x10) /* Station addr, low */
+#define XEM_RPLR_OFFSET        (XEM_REG_OFFSET + 0x1C) /* Rx packet length */
+#define XEM_TPLR_OFFSET        (XEM_REG_OFFSET + 0x20) /* Tx packet length */
+#define XEM_TSR_OFFSET (XEM_REG_OFFSET + 0x24) /* Tx status */
+
+#define XEM_PFIFO_OFFSET       0x2000UL
+/* Tx registers */
+#define XEM_PFIFO_TXREG_OFFSET (XEM_PFIFO_OFFSET + 0x0)
+/* Rx registers */
+#define XEM_PFIFO_RXREG_OFFSET (XEM_PFIFO_OFFSET + 0x10)
+/* Tx keyhole */
+#define XEM_PFIFO_TXDATA_OFFSET        (XEM_PFIFO_OFFSET + 0x100)
+/* Rx keyhole */
+#define XEM_PFIFO_RXDATA_OFFSET        (XEM_PFIFO_OFFSET + 0x200)
+
+/*
+ * EMAC Interrupt Registers (Status and Enable) masks. These registers are
+ * part of the IPIF IP Interrupt registers
+ */
+/* A mask for all transmit interrupts, used in polled mode */
+#define XEM_EIR_XMIT_ALL_MASK  (XEM_EIR_XMIT_DONE_MASK |\
+                               XEM_EIR_XMIT_ERROR_MASK | \
+                               XEM_EIR_XMIT_SFIFO_EMPTY_MASK |\
+                               XEM_EIR_XMIT_LFIFO_FULL_MASK)
+
+/* Xmit complete */
+#define XEM_EIR_XMIT_DONE_MASK         0x00000001UL
+/* Recv complete */
+#define XEM_EIR_RECV_DONE_MASK         0x00000002UL
+/* Xmit error */
+#define XEM_EIR_XMIT_ERROR_MASK                0x00000004UL
+/* Recv error */
+#define XEM_EIR_RECV_ERROR_MASK                0x00000008UL
+/* Xmit status fifo empty */
+#define XEM_EIR_XMIT_SFIFO_EMPTY_MASK  0x00000010UL
+/* Recv length fifo empty */
+#define XEM_EIR_RECV_LFIFO_EMPTY_MASK  0x00000020UL
+/* Xmit length fifo full */
+#define XEM_EIR_XMIT_LFIFO_FULL_MASK   0x00000040UL
+/* Recv length fifo overrun */
+#define XEM_EIR_RECV_LFIFO_OVER_MASK   0x00000080UL
+/* Recv length fifo underrun */
+#define XEM_EIR_RECV_LFIFO_UNDER_MASK  0x00000100UL
+/* Xmit status fifo overrun */
+#define XEM_EIR_XMIT_SFIFO_OVER_MASK   0x00000200UL
+/* Transmit status fifo underrun */
+#define XEM_EIR_XMIT_SFIFO_UNDER_MASK  0x00000400UL
+/* Transmit length fifo overrun */
+#define XEM_EIR_XMIT_LFIFO_OVER_MASK   0x00000800UL
+/* Transmit length fifo underrun */
+#define XEM_EIR_XMIT_LFIFO_UNDER_MASK  0x00001000UL
+/* Transmit pause pkt received */
+#define XEM_EIR_XMIT_PAUSE_MASK                0x00002000UL
+
+/*
+ * EMAC Control Register (ECR)
+ */
+/* Full duplex mode */
+#define XEM_ECR_FULL_DUPLEX_MASK       0x80000000UL
+/* Reset transmitter */
+#define XEM_ECR_XMIT_RESET_MASK                0x40000000UL
+/* Enable transmitter */
+#define XEM_ECR_XMIT_ENABLE_MASK       0x20000000UL
+/* Reset receiver */
+#define XEM_ECR_RECV_RESET_MASK                0x10000000UL
+/* Enable receiver */
+#define XEM_ECR_RECV_ENABLE_MASK       0x08000000UL
+/* Enable PHY */
+#define XEM_ECR_PHY_ENABLE_MASK                0x04000000UL
+/* Enable xmit pad insert */
+#define XEM_ECR_XMIT_PAD_ENABLE_MASK   0x02000000UL
+/* Enable xmit FCS insert */
+#define XEM_ECR_XMIT_FCS_ENABLE_MASK   0x01000000UL
+/* Enable unicast addr */
+#define XEM_ECR_UNICAST_ENABLE_MASK    0x00020000UL
+/* Enable broadcast addr */
+#define XEM_ECR_BROAD_ENABLE_MASK      0x00008000UL
+
+/*
+ * Transmit Status Register (TSR)
+ */
+/* Transmit excess deferral */
+#define XEM_TSR_EXCESS_DEFERRAL_MASK   0x80000000UL
+/* Transmit late collision */
+#define XEM_TSR_LATE_COLLISION_MASK    0x01000000UL
+
 #define ENET_MAX_MTU           PKTSIZE
 #define ENET_ADDR_LENGTH       6
 
 static unsigned int etherrxbuff[PKTSIZE_ALIGN/4]; /* Receive buffer */
 
-static u8 EMACAddr[ENET_ADDR_LENGTH] = { 0x00, 0x0a, 0x35, 0x00, 0x22, 0x01 };
+static u8 emacaddr[ENET_ADDR_LENGTH] = { 0x00, 0x0a, 0x35, 0x00, 0x22, 0x01 };
 
-static XEmac Emac;
+static xemac emac;
 
 void eth_halt(void)
 {
-       return;
+       debug ("eth_halt\n");
 }
 
 int eth_init(bd_t * bis)
 {
-       u32 HelpReg;
-#ifdef DEBUG
-       printf("EMAC Initialization Started\n\r");
-#endif
-       if (Emac.IsStarted) {
+       u32 helpreg;
+       debug ("EMAC Initialization Started\n\r");
+
+       if (emac.isstarted) {
                puts("Emac is started\n");
                return 0;
        }
 
-       memset (&Emac, 0, sizeof (XEmac));
+       memset (&emac, 0, sizeof (xemac));
 
-       Emac.BaseAddress = XILINX_EMAC_BASEADDR;
+       emac.baseaddress = XILINX_EMAC_BASEADDR;
 
        /* Setting up FIFOs */
-       Emac.RecvFifo.RegBaseAddress = Emac.BaseAddress +
+       emac.recvfifo.regbaseaddress = emac.baseaddress +
                                        XEM_PFIFO_RXREG_OFFSET;
-       Emac.RecvFifo.DataBaseAddress = Emac.BaseAddress +
+       emac.recvfifo.databaseaddress = emac.baseaddress +
                                        XEM_PFIFO_RXDATA_OFFSET;
-       out_be32 (Emac.RecvFifo.RegBaseAddress, XPF_RESET_FIFO_MASK);
+       out_be32 (emac.recvfifo.regbaseaddress, XPF_RESET_FIFO_MASK);
 
-       Emac.SendFifo.RegBaseAddress = Emac.BaseAddress +
+       emac.sendfifo.regbaseaddress = emac.baseaddress +
                                        XEM_PFIFO_TXREG_OFFSET;
-       Emac.SendFifo.DataBaseAddress = Emac.BaseAddress +
+       emac.sendfifo.databaseaddress = emac.baseaddress +
                                        XEM_PFIFO_TXDATA_OFFSET;
-       out_be32 (Emac.SendFifo.RegBaseAddress, XPF_RESET_FIFO_MASK);
+       out_be32 (emac.sendfifo.regbaseaddress, XPF_RESET_FIFO_MASK);
 
        /* Reset the entire IPIF */
-       out_be32 (Emac.BaseAddress + XIIF_V123B_RESETR_OFFSET,
+       out_be32 (emac.baseaddress + XIIF_V123B_RESETR_OFFSET,
                                        XIIF_V123B_RESET_MASK);
 
        /* Stopping EMAC for setting up MAC */
-       HelpReg = in_be32 (Emac.BaseAddress + XEM_ECR_OFFSET);
-       HelpReg &= ~(XEM_ECR_XMIT_ENABLE_MASK | XEM_ECR_RECV_ENABLE_MASK);
-       out_be32 (Emac.BaseAddress + XEM_ECR_OFFSET, HelpReg);
+       helpreg = in_be32 (emac.baseaddress + XEM_ECR_OFFSET);
+       helpreg &= ~(XEM_ECR_XMIT_ENABLE_MASK | XEM_ECR_RECV_ENABLE_MASK);
+       out_be32 (emac.baseaddress + XEM_ECR_OFFSET, helpreg);
 
        if (!getenv("ethaddr")) {
-               memcpy(bis->bi_enetaddr, EMACAddr, ENET_ADDR_LENGTH);
+               memcpy(bis->bi_enetaddr, emacaddr, ENET_ADDR_LENGTH);
        }
 
        /* Set the device station address high and low registers */
-       HelpReg = (bis->bi_enetaddr[0] << 8) | bis->bi_enetaddr[1];
-       out_be32 (Emac.BaseAddress + XEM_SAH_OFFSET, HelpReg);
-       HelpReg = (bis->bi_enetaddr[2] << 24) | (bis->bi_enetaddr[3] << 16) |
+       helpreg = (bis->bi_enetaddr[0] << 8) | bis->bi_enetaddr[1];
+       out_be32 (emac.baseaddress + XEM_SAH_OFFSET, helpreg);
+       helpreg = (bis->bi_enetaddr[2] << 24) | (bis->bi_enetaddr[3] << 16) |
                        (bis->bi_enetaddr[4] << 8) | bis->bi_enetaddr[5];
-       out_be32 (Emac.BaseAddress + XEM_SAL_OFFSET, HelpReg);
-
+       out_be32 (emac.baseaddress + XEM_SAL_OFFSET, helpreg);
 
-       HelpReg = XEM_ECR_UNICAST_ENABLE_MASK | XEM_ECR_BROAD_ENABLE_MASK |
+       helpreg = XEM_ECR_UNICAST_ENABLE_MASK | XEM_ECR_BROAD_ENABLE_MASK |
                XEM_ECR_FULL_DUPLEX_MASK | XEM_ECR_XMIT_FCS_ENABLE_MASK |
                XEM_ECR_XMIT_PAD_ENABLE_MASK | XEM_ECR_PHY_ENABLE_MASK;
-       out_be32 (Emac.BaseAddress + XEM_ECR_OFFSET, HelpReg);
+       out_be32 (emac.baseaddress + XEM_ECR_OFFSET, helpreg);
 
-       Emac.IsStarted = 1;
+       emac.isstarted = 1;
 
        /* Enable the transmitter, and receiver */
-       HelpReg = in_be32 (Emac.BaseAddress + XEM_ECR_OFFSET);
-       HelpReg &= ~(XEM_ECR_XMIT_RESET_MASK | XEM_ECR_RECV_RESET_MASK);
-       HelpReg |= (XEM_ECR_XMIT_ENABLE_MASK | XEM_ECR_RECV_ENABLE_MASK);
-       out_be32 (Emac.BaseAddress + XEM_ECR_OFFSET, HelpReg);
+       helpreg = in_be32 (emac.baseaddress + XEM_ECR_OFFSET);
+       helpreg &= ~(XEM_ECR_XMIT_RESET_MASK | XEM_ECR_RECV_RESET_MASK);
+       helpreg |= (XEM_ECR_XMIT_ENABLE_MASK | XEM_ECR_RECV_ENABLE_MASK);
+       out_be32 (emac.baseaddress + XEM_ECR_OFFSET, helpreg);
 
        printf("EMAC Initialization complete\n\r");
        return 0;
@@ -118,12 +230,12 @@ int eth_init(bd_t * bis)
 
 int eth_send(volatile void *ptr, int len)
 {
-       u32 IntrStatus;
-       u32 XmitStatus;
-       u32 FifoCount;
-       u32 WordCount;
-       u32 ExtraByteCount;
-       u32 *WordBuffer = (u32 *) ptr;
+       u32 intrstatus;
+       u32 xmitstatus;
+       u32 fifocount;
+       u32 wordcount;
+       u32 extrabytecount;
+       u32 *wordbuffer = (u32 *) ptr;
 
        if (len > ENET_MAX_MTU)
                len = ENET_MAX_MTU;
@@ -135,24 +247,18 @@ int eth_send(volatile void *ptr, int len)
         * continue. The upper layer software should reset the device to resolve
         * the error.
         */
-       IntrStatus = in_be32 ((Emac.BaseAddress) + XIIF_V123B_IISR_OFFSET);
-       if (IntrStatus & (XEM_EIR_XMIT_SFIFO_OVER_MASK |
+       intrstatus = in_be32 ((emac.baseaddress) + XIIF_V123B_IISR_OFFSET);
+       if (intrstatus & (XEM_EIR_XMIT_SFIFO_OVER_MASK |
                        XEM_EIR_XMIT_LFIFO_OVER_MASK)) {
-#ifdef DEBUG
-               puts ("Transmitting overrun error\n");
-#endif
+               debug ("Transmitting overrun error\n");
                return 0;
-       } else if (IntrStatus & (XEM_EIR_XMIT_SFIFO_UNDER_MASK |
+       } else if (intrstatus & (XEM_EIR_XMIT_SFIFO_UNDER_MASK |
                        XEM_EIR_XMIT_LFIFO_UNDER_MASK)) {
-#ifdef DEBUG
-               puts ("Transmitting underrun error\n");
-#endif
+               debug ("Transmitting underrun error\n");
                return 0;
-       } else if (in_be32 (Emac.SendFifo.RegBaseAddress +
+       } else if (in_be32 (emac.sendfifo.regbaseaddress +
                        XPF_COUNT_STATUS_REG_OFFSET) & XPF_DEADLOCK_MASK) {
-#ifdef DEBUG
-               puts("Transmitting fifo error\n");
-#endif
+               debug ("Transmitting fifo error\n");
                return 0;
        }
 
@@ -165,102 +271,92 @@ int eth_send(volatile void *ptr, int len)
         * Clear the latched LFIFO_FULL bit so next time around the most
         * current status is represented
         */
-       if (IntrStatus & XEM_EIR_XMIT_LFIFO_FULL_MASK) {
-               out_be32 ((Emac.BaseAddress) + XIIF_V123B_IISR_OFFSET, IntrStatus
-                               & XEM_EIR_XMIT_LFIFO_FULL_MASK);
-#ifdef DEBUG
-               puts ("Fifo is full\n");
-#endif
+       if (intrstatus & XEM_EIR_XMIT_LFIFO_FULL_MASK) {
+               out_be32 ((emac.baseaddress) + XIIF_V123B_IISR_OFFSET,
+                       intrstatus & XEM_EIR_XMIT_LFIFO_FULL_MASK);
+               debug ("Fifo is full\n");
                return 0;
        }
 
        /* get the count of how many words may be inserted into the FIFO */
-       FifoCount = in_be32 (Emac.SendFifo.RegBaseAddress +
+       fifocount = in_be32 (emac.sendfifo.regbaseaddress +
                                XPF_COUNT_STATUS_REG_OFFSET) & XPF_COUNT_MASK;
-       WordCount = len >> 2;
-       ExtraByteCount = len & 0x3;
+       wordcount = len >> 2;
+       extrabytecount = len & 0x3;
 
-       if (FifoCount < WordCount) {
-#ifdef DEBUG
-               puts ("Sending packet is larger then size of FIFO\n");
-#endif
+       if (fifocount < wordcount) {
+               debug ("Sending packet is larger then size of FIFO\n");
                return 0;
        }
 
-       for (FifoCount = 0; FifoCount < WordCount; FifoCount++) {
-               out_be32 (Emac.SendFifo.DataBaseAddress, WordBuffer[FifoCount]);
+       for (fifocount = 0; fifocount < wordcount; fifocount++) {
+               out_be32 (emac.sendfifo.databaseaddress, wordbuffer[fifocount]);
        }
-       if (ExtraByteCount > 0) {
-               u32 LastWord = 0;
-               u8 *ExtraBytesBuffer = (u8 *) (WordBuffer + WordCount);
-
-               if (ExtraByteCount == 1) {
-                       LastWord = ExtraBytesBuffer[0] << 24;
-               } else if (ExtraByteCount == 2) {
-                       LastWord = ExtraBytesBuffer[0] << 24 |
-                               ExtraBytesBuffer[1] << 16;
-               } else if (ExtraByteCount == 3) {
-                       LastWord = ExtraBytesBuffer[0] << 24 |
-                               ExtraBytesBuffer[1] << 16 |
-                               ExtraBytesBuffer[2] << 8;
+       if (extrabytecount > 0) {
+               u32 lastword = 0;
+               u8 *extrabytesbuffer = (u8 *) (wordbuffer + wordcount);
+
+               if (extrabytecount == 1) {
+                       lastword = extrabytesbuffer[0] << 24;
+               } else if (extrabytecount == 2) {
+                       lastword = extrabytesbuffer[0] << 24 |
+                               extrabytesbuffer[1] << 16;
+               } else if (extrabytecount == 3) {
+                       lastword = extrabytesbuffer[0] << 24 |
+                               extrabytesbuffer[1] << 16 |
+                               extrabytesbuffer[2] << 8;
                }
-               out_be32 (Emac.SendFifo.DataBaseAddress, LastWord);
+               out_be32 (emac.sendfifo.databaseaddress, lastword);
        }
 
        /* Loop on the MAC's status to wait for any pause to complete */
-       IntrStatus = in_be32 ((Emac.BaseAddress) + XIIF_V123B_IISR_OFFSET);
-       while ((IntrStatus & XEM_EIR_XMIT_PAUSE_MASK) != 0) {
-               IntrStatus = in_be32 ((Emac.BaseAddress) +
+       intrstatus = in_be32 ((emac.baseaddress) + XIIF_V123B_IISR_OFFSET);
+       while ((intrstatus & XEM_EIR_XMIT_PAUSE_MASK) != 0) {
+               intrstatus = in_be32 ((emac.baseaddress) +
                                        XIIF_V123B_IISR_OFFSET);
                /* Clear the pause status from the transmit status register */
-               out_be32 ((Emac.BaseAddress) + XIIF_V123B_IISR_OFFSET,
-                               IntrStatus & XEM_EIR_XMIT_PAUSE_MASK);
+               out_be32 ((emac.baseaddress) + XIIF_V123B_IISR_OFFSET,
+                               intrstatus & XEM_EIR_XMIT_PAUSE_MASK);
        }
 
        /*
         * Set the MAC's transmit packet length register to tell it to transmit
         */
-       out_be32 (Emac.BaseAddress + XEM_TPLR_OFFSET, len);
+       out_be32 (emac.baseaddress + XEM_TPLR_OFFSET, len);
 
        /*
         * Loop on the MAC's status to wait for the transmit to complete.
         * The transmit status is in the FIFO when the XMIT_DONE bit is set.
         */
        do {
-               IntrStatus = in_be32 ((Emac.BaseAddress) +
+               intrstatus = in_be32 ((emac.baseaddress) +
                                                XIIF_V123B_IISR_OFFSET);
        }
-       while ((IntrStatus & XEM_EIR_XMIT_DONE_MASK) == 0);
+       while ((intrstatus & XEM_EIR_XMIT_DONE_MASK) == 0);
 
-       XmitStatus = in_be32 (Emac.BaseAddress + XEM_TSR_OFFSET);
+       xmitstatus = in_be32 (emac.baseaddress + XEM_TSR_OFFSET);
 
-       if (IntrStatus & (XEM_EIR_XMIT_SFIFO_OVER_MASK |
+       if (intrstatus & (XEM_EIR_XMIT_SFIFO_OVER_MASK |
                                        XEM_EIR_XMIT_LFIFO_OVER_MASK)) {
-#ifdef DEBUG
-               puts ("Transmitting overrun error\n");
-#endif
+               debug ("Transmitting overrun error\n");
                return 0;
-       } else if (IntrStatus & (XEM_EIR_XMIT_SFIFO_UNDER_MASK |
+       } else if (intrstatus & (XEM_EIR_XMIT_SFIFO_UNDER_MASK |
                                        XEM_EIR_XMIT_LFIFO_UNDER_MASK)) {
-#ifdef DEBUG
-               puts ("Transmitting underrun error\n");
-#endif
+               debug ("Transmitting underrun error\n");
                return 0;
        }
 
        /* Clear the interrupt status register of transmit statuses */
-       out_be32 ((Emac.BaseAddress) + XIIF_V123B_IISR_OFFSET,
-                               IntrStatus & XEM_EIR_XMIT_ALL_MASK);
+       out_be32 ((emac.baseaddress) + XIIF_V123B_IISR_OFFSET,
+                               intrstatus & XEM_EIR_XMIT_ALL_MASK);
 
        /*
         * Collision errors are stored in the transmit status register
         * instead of the interrupt status register
         */
-       if ((XmitStatus & XEM_TSR_EXCESS_DEFERRAL_MASK) ||
-                               (XmitStatus & XEM_TSR_LATE_COLLISION_MASK)) {
-#ifdef DEBUG
-               puts ("Transmitting collision error\n");
-#endif
+       if ((xmitstatus & XEM_TSR_EXCESS_DEFERRAL_MASK) ||
+                               (xmitstatus & XEM_TSR_LATE_COLLISION_MASK)) {
+               debug ("Transmitting collision error\n");
                return 0;
        }
        return 1;
@@ -268,38 +364,35 @@ int eth_send(volatile void *ptr, int len)
 
 int eth_rx(void)
 {
-       u32 PktLength;
-       u32 IntrStatus;
-       u32 FifoCount;
-       u32 WordCount;
-       u32 ExtraByteCount;
-       u32 LastWord;
-       u8 *ExtraBytesBuffer;
-
-       if (in_be32 (Emac.RecvFifo.RegBaseAddress + XPF_COUNT_STATUS_REG_OFFSET)
+       u32 pktlength;
+       u32 intrstatus;
+       u32 fifocount;
+       u32 wordcount;
+       u32 extrabytecount;
+       u32 lastword;
+       u8 *extrabytesbuffer;
+
+       if (in_be32 (emac.recvfifo.regbaseaddress + XPF_COUNT_STATUS_REG_OFFSET)
                        & XPF_DEADLOCK_MASK) {
-               out_be32 (Emac.RecvFifo.RegBaseAddress, XPF_RESET_FIFO_MASK);
-#ifdef DEBUG
-               puts ("Receiving FIFO deadlock\n");
-#endif
+               out_be32 (emac.recvfifo.regbaseaddress, XPF_RESET_FIFO_MASK);
+               debug ("Receiving FIFO deadlock\n");
                return 0;
        }
 
        /*
-        * Get the interrupt status to know what happened (whether an error occurred
-        * and/or whether frames have been received successfully). When clearing the
-        * intr status register, clear only statuses that pertain to receive.
+        * Get the interrupt status to know what happened (whether an error
+        * occurred and/or whether frames have been received successfully).
+        * When clearing the intr status register, clear only statuses that
+        * pertain to receive.
         */
-       IntrStatus = in_be32 ((Emac.BaseAddress) + XIIF_V123B_IISR_OFFSET);
+       intrstatus = in_be32 ((emac.baseaddress) + XIIF_V123B_IISR_OFFSET);
        /*
         * Before reading from the length FIFO, make sure the length FIFO is not
         * empty. We could cause an underrun error if we try to read from an
         * empty FIFO.
         */
-       if (!(IntrStatus & XEM_EIR_RECV_DONE_MASK)) {
-#ifdef DEBUG
-               /* puts("Receiving FIFO is empty\n"); */
-#endif
+       if (!(intrstatus & XEM_EIR_RECV_DONE_MASK)) {
+               /* debug ("Receiving FIFO is empty\n"); */
                return 0;
        }
 
@@ -307,8 +400,8 @@ int eth_rx(void)
         * Determine, from the MAC, the length of the next packet available
         * in the data FIFO (there should be a non-zero length here)
         */
-       PktLength = in_be32 (Emac.BaseAddress + XEM_RPLR_OFFSET);
-       if (!PktLength) {
+       pktlength = in_be32 (emac.baseaddress + XEM_RPLR_OFFSET);
+       if (!pktlength) {
                return 0;
        }
 
@@ -320,53 +413,50 @@ int eth_rx(void)
         * in the IPIF, which means it may indicate a non-empty condition even
         * though there is something in the FIFO.
         */
-       out_be32 ((Emac.BaseAddress) + XIIF_V123B_IISR_OFFSET,
+       out_be32 ((emac.baseaddress) + XIIF_V123B_IISR_OFFSET,
                                                XEM_EIR_RECV_DONE_MASK);
 
-       FifoCount = in_be32 (Emac.RecvFifo.RegBaseAddress +
+       fifocount = in_be32 (emac.recvfifo.regbaseaddress +
                                XPF_COUNT_STATUS_REG_OFFSET) & XPF_COUNT_MASK;
 
-       if ((FifoCount * 4) < PktLength) {
-#ifdef DEBUG
-               puts ("Receiving FIFO is smaller than packet size.\n");
-#endif
+       if ((fifocount * 4) < pktlength) {
+               debug ("Receiving FIFO is smaller than packet size.\n");
                return 0;
        }
 
-       WordCount = PktLength >> 2;
-       ExtraByteCount = PktLength & 0x3;
+       wordcount = pktlength >> 2;
+       extrabytecount = pktlength & 0x3;
 
-       for (FifoCount = 0; FifoCount < WordCount; FifoCount++) {
-               etherrxbuff[FifoCount] =
-                               in_be32 (Emac.RecvFifo.DataBaseAddress);
+       for (fifocount = 0; fifocount < wordcount; fifocount++) {
+               etherrxbuff[fifocount] =
+                               in_be32 (emac.recvfifo.databaseaddress);
        }
 
        /*
         * if there are extra bytes to handle, read the last word from the FIFO
         * and insert the extra bytes into the buffer
         */
-       if (ExtraByteCount > 0) {
-               ExtraBytesBuffer = (u8 *) (etherrxbuff + WordCount);
+       if (extrabytecount > 0) {
+               extrabytesbuffer = (u8 *) (etherrxbuff + wordcount);
 
-               LastWord = in_be32 (Emac.RecvFifo.DataBaseAddress);
+               lastword = in_be32 (emac.recvfifo.databaseaddress);
 
                /*
                 * one extra byte in the last word, put the byte into the next
                 * location of the buffer, bytes in a word of the FIFO are
                 * ordered from most significant byte to least
                 */
-               if (ExtraByteCount == 1) {
-                       ExtraBytesBuffer[0] = (u8) (LastWord >> 24);
-               } else if (ExtraByteCount == 2) {
-                       ExtraBytesBuffer[0] = (u8) (LastWord >> 24);
-                       ExtraBytesBuffer[1] = (u8) (LastWord >> 16);
-               } else if (ExtraByteCount == 3) {
-                       ExtraBytesBuffer[0] = (u8) (LastWord >> 24);
-                       ExtraBytesBuffer[1] = (u8) (LastWord >> 16);
-                       ExtraBytesBuffer[2] = (u8) (LastWord >> 8);
+               if (extrabytecount == 1) {
+                       extrabytesbuffer[0] = (u8) (lastword >> 24);
+               } else if (extrabytecount == 2) {
+                       extrabytesbuffer[0] = (u8) (lastword >> 24);
+                       extrabytesbuffer[1] = (u8) (lastword >> 16);
+               } else if (extrabytecount == 3) {
+                       extrabytesbuffer[0] = (u8) (lastword >> 24);
+                       extrabytesbuffer[1] = (u8) (lastword >> 16);
+                       extrabytesbuffer[2] = (u8) (lastword >> 8);
                }
        }
-       NetReceive((uchar *)etherrxbuff, PktLength);
+       NetReceive((uchar *)etherrxbuff, pktlength);
        return 1;
 }
-#endif