[U-Boot] [PATCH v2 07/17] SPEAr : Network driver support added

Vipin KUMAR vipin.kumar at st.com
Tue May 4 08:33:24 CEST 2010


On 5/3/2010 5:18 AM, Tom Rix wrote:
> Vipin KUMAR wrote:
>> Designware network driver support added.
>> This is a Synopsys ethernet controller
>>
>> Signed-off-by: Vipin Kumar <vipin.kumar at st.com>
>> ---
>>  drivers/net/Makefile     |    1 +
>>  drivers/net/designware.c |  537
>> ++++++++++++++++++++++++++++++++++++++++++++++
>>  drivers/net/designware.h |  264 +++++++++++++++++++++++
>>  include/netdev.h         |    1 +
>>  4 files changed, 803 insertions(+), 0 deletions(-)
>>  create mode 100644 drivers/net/designware.c
>>  create mode 100644 drivers/net/designware.h
>>
>> diff --git a/drivers/net/Makefile b/drivers/net/Makefile
>> index 1ec0ba1..82df908 100644
>> --- a/drivers/net/Makefile
>> +++ b/drivers/net/Makefile
>> @@ -33,6 +33,7 @@ COBJS-$(CONFIG_BCM570x) += bcm570x.o
>> bcm570x_autoneg.o 5701rls.o
>>  COBJS-$(CONFIG_BFIN_MAC) += bfin_mac.o
>>  COBJS-$(CONFIG_CS8900) += cs8900.o
>>  COBJS-$(CONFIG_TULIP) += dc2114x.o
>> +COBJS-$(CONFIG_DESIGNWARE_ETH) += designware.o
>>  COBJS-$(CONFIG_DRIVER_DM9000) += dm9000x.o
>>  COBJS-$(CONFIG_DNET) += dnet.o
>>  COBJS-$(CONFIG_E1000) += e1000.o
>> diff --git a/drivers/net/designware.c b/drivers/net/designware.c
>> new file mode 100644
>> index 0000000..9361f77
>> --- /dev/null
>> +++ b/drivers/net/designware.c
>> @@ -0,0 +1,537 @@
>> +/*
>> + * (C) Copyright 2009
>> + * Vipin Kumar, ST Micoelectronics, vipin.kumar at st.com.
>> + *
>> + * (C) Copyright 2008
>> + * Deepak Sikri, ST Micoelectronics, deepak.sikri at st.com.
>> + *
>> + * 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.
>> + *
>> + * 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
>> + */
>> +
>> +/*
>> + * Designware ethernet IP driver for u-boot
>> + */
>> +
>> +#include <common.h>
>> +#include <miiphy.h>
>> +#include <malloc.h>
>> +#include <linux/err.h>
>> +#include <asm/io.h>
>> +#include "designware.h"
>> +
>> +static void tx_descs_init(struct eth_device *dev)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    struct eth_dma_regs *dma_p = priv->dma_regs_p;
>> +    struct dmamacdescr *desc_table_p = &priv->tx_mac_descrtable[0];
>> +    char *txbuffs = &priv->txbuffs[0];
>> +    struct dmamacdescr *desc_p;
>> +    u32 idx;
>> +
>> +    for (idx = 0; idx < CONFIG_TX_DESCR_NUM; idx++) {
>> +        desc_p = &desc_table_p[idx];
>> +        desc_p->dmamac_addr = &txbuffs[idx * CONFIG_ETH_BUFSIZE];
>> +        desc_p->dmamac_next = &desc_table_p[idx + 1];
>> +
>> +#if defined(CONFIG_DW_ALTDESCRIPTOR)
> 
> Add a comment describing CONFIG_DW_ALTDESCRIPTOR
> The README.spear would be an excellent place to put this
> and the other new CONFIG_ *
> 
> Update the README.spear to be current with the
> new spear1300 target and the other new information
> you want to share.

OK. I would add a comment here. Since this is implemented as generic
designware network peripheral and it can be reused, README.spear 
doesn't look like a good place to describe the various configurations
that are supported by the driver

Should I create a new doc/README.designware_eth to describe the
supported peripheral configurations ?

>> +        desc_p->txrx_status &= ~(DESC_TXSTS_TXINT | DESC_TXSTS_TXLAST |
>> +                DESC_TXSTS_TXFIRST | DESC_TXSTS_TXCRCDIS | \
>> +                DESC_TXSTS_TXCHECKINSCTRL | \
>> +                DESC_TXSTS_TXRINGEND | DESC_TXSTS_TXPADDIS);
>> +
>> +        desc_p->txrx_status |= DESC_TXSTS_TXCHAIN;
>> +        desc_p->dmamac_cntl = 0;
>> +        desc_p->txrx_status &= ~(DESC_TXSTS_MSK | DESC_TXSTS_OWNBYDMA);
>> +#else
>> +        desc_p->dmamac_cntl = DESC_TXCTRL_TXCHAIN;
>> +        desc_p->txrx_status = 0;
>> +#endif
>> +    }
>> +
>> +    /* Correcting the last pointer of the chain */
>> +    desc_p->dmamac_next = &desc_table_p[0];
>> +
>> +    writel((ulong)&desc_table_p[0], &dma_p->txdesclistaddr);
>> +}
>> +
>> +static void rx_descs_init(struct eth_device *dev)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    struct eth_dma_regs *dma_p = priv->dma_regs_p;
>> +    struct dmamacdescr *desc_table_p = &priv->rx_mac_descrtable[0];
>> +    char *rxbuffs = &priv->rxbuffs[0];
>> +    struct dmamacdescr *desc_p;
>> +    u32 idx;
>> +
>> +    for (idx = 0; idx < CONFIG_RX_DESCR_NUM; idx++) {
>> +        desc_p = &desc_table_p[idx];
>> +        desc_p->dmamac_addr = &rxbuffs[idx * CONFIG_ETH_BUFSIZE];
>> +        desc_p->dmamac_next = &desc_table_p[idx + 1];
>> +
>> +        desc_p->dmamac_cntl =
>> +            (MAC_MAX_FRAME_SZ & DESC_RXCTRL_SIZE1MASK) | \
>> +                      DESC_RXCTRL_RXCHAIN;
>> +
>> +        desc_p->txrx_status = DESC_RXSTS_OWNBYDMA;
>> +    }
>> +
>> +    /* Correcting the last pointer of the chain */
>> +    desc_p->dmamac_next = &desc_table_p[0];
>> +
>> +    writel((ulong)&desc_table_p[0], &dma_p->rxdesclistaddr);
>> +}
>> +
>> +static void descs_init(struct eth_device *dev)
>> +{
>> +    tx_descs_init(dev);
>> +    rx_descs_init(dev);
>> +}
>> +
>> +static int mac_reset(struct eth_device *dev)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    struct eth_mac_regs *mac_p = priv->mac_regs_p;
>> +    struct eth_dma_regs *dma_p = priv->dma_regs_p;
>> +
>> +    int timeout = CONFIG_MACRESET_TIMEOUT;
>> +
>> +    writel(DMAMAC_SRST, &dma_p->busmode);
>> +    writel(MII_PORTSELECT, &mac_p->conf);
>> +
>> +    do {
>> +        if (!(readl(&dma_p->busmode) & DMAMAC_SRST))
>> +            break;
>> +        udelay(1000);
>> +    } while (timeout--);
>> +
>> +    if (timeout < 0)
>> +        return -1;
>> +
>> +    return 0;
>> +}
>> +
>> +static void program_mac(struct eth_device *dev)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    struct eth_mac_regs *mac_p = priv->mac_regs_p;
>> +    u32 macid_lo, macid_hi;
>> +    u8 *mac_id = &dev->enetaddr[0];
>> +
>> +    macid_lo = mac_id[0] + (mac_id[1] << 8) + \
>> +           (mac_id[2] << 16) + (mac_id[3] << 24);
>> +    macid_hi = mac_id[4] + (mac_id[5] << 8);
>> +
>> +    writel(macid_hi, &mac_p->macaddr0hi);
>> +    writel(macid_lo, &mac_p->macaddr0lo);
>> +}
>> +
>> +static int dw_eth_init(struct eth_device *dev, bd_t *bis)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    struct eth_mac_regs *mac_p = priv->mac_regs_p;
>> +    struct eth_dma_regs *dma_p = priv->dma_regs_p;
>> +    u32 conf;
>> +
>> +    /* Reset ethernet hardware */
>> +    if (mac_reset(dev) < 0)
>> +        return -1;
>> +
>> +    program_mac(dev);
>> +
>> +    writel(FIXEDBURST | PRIORXTX_41 | BURST_16,
>> +            &dma_p->busmode);
>> +
>> +    writel(FLUSHTXFIFO | readl(&dma_p->opmode), &dma_p->opmode);
>> +    writel(STOREFORWARD | TXSECONDFRAME, &dma_p->opmode);
>> +
>> +    conf = FRAMEBURSTENABLE | DISABLERXOWN;
>> +
>> +    if (priv->speed != SPEED_1000M)
>> +        conf |= MII_PORTSELECT;
>> +
>> +    if (priv->duplex == FULL_DUPLEX)
>> +        conf |= FULLDPLXMODE;
>> +
>> +    writel(conf, &mac_p->conf);
>> +
>> +    descs_init(dev);
>> +
>> +    /*
>> +     * Start/Enable xfer at dma as well as mac level
>> +     */
>> +    writel(readl(&dma_p->opmode) | RXSTART, &dma_p->opmode);
>> +    writel(readl(&dma_p->opmode) | TXSTART, &dma_p->opmode);
>> +
>> +    writel(readl(&mac_p->conf) | RXENABLE, &mac_p->conf);
>> +    writel(readl(&mac_p->conf) | TXENABLE, &mac_p->conf);
>> +
>> +    return 0;
>> +}
>> +
>> +static int dw_eth_send(struct eth_device *dev, volatile void *packet,
>> +        int length)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    struct eth_dma_regs *dma_p = priv->dma_regs_p;
>> +    u32 desc_num = priv->tx_currdescnum;
>> +    struct dmamacdescr *desc_p = &priv->tx_mac_descrtable[desc_num];
>> +
>> +    /* Check if the descriptor is owned by CPU */
>> +    if (desc_p->txrx_status & DESC_TXSTS_OWNBYDMA) {
>> +        printf("CPU not owner of tx frame\n");
>> +        return -1;
>> +    }
>> +
>> +    memcpy((void *)desc_p->dmamac_addr, (void *)packet, length);
>> +
>> +#if defined(CONFIG_DW_ALTDESCRIPTOR)
>> +    desc_p->txrx_status |= DESC_TXSTS_TXFIRST | DESC_TXSTS_TXLAST;
>> +    desc_p->dmamac_cntl |= (length << DESC_TXCTRL_SIZE1SHFT) & \
>> +                   DESC_TXCTRL_SIZE1MASK;
>> +
>> +    desc_p->txrx_status &= ~(DESC_TXSTS_MSK);
>> +    desc_p->txrx_status |= DESC_TXSTS_OWNBYDMA;
>> +#else
>> +    desc_p->dmamac_cntl |= ((length << DESC_TXCTRL_SIZE1SHFT) & \
>> +                   DESC_TXCTRL_SIZE1MASK) | DESC_TXCTRL_TXLAST | \
>> +                   DESC_TXCTRL_TXFIRST;
>> +
>> +    desc_p->txrx_status = DESC_TXSTS_OWNBYDMA;
>> +#endif
>> +
>> +    /* Test the wrap-around condition. */
>> +    if (++desc_num >= CONFIG_TX_DESCR_NUM)
>> +        desc_num = 0;
>> +
>> +    priv->tx_currdescnum = desc_num;
>> +
>> +    /* Start the transmission */
>> +    writel(POLL_DATA, &dma_p->txpolldemand);
>> +
>> +    return 0;
>> +}
>> +
>> +static int dw_eth_recv(struct eth_device *dev)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    u32 desc_num = priv->rx_currdescnum;
>> +    struct dmamacdescr *desc_p = &priv->rx_mac_descrtable[desc_num];
>> +
>> +    u32 status = desc_p->txrx_status;
>> +    int length = 0;
>> +
>> +    /* Check  if the owner is the CPU */
>> +    if (!(status & DESC_RXSTS_OWNBYDMA)) {
>> +
>> +        length = (status & DESC_RXSTS_FRMLENMSK) >> \
>> +             DESC_RXSTS_FRMLENSHFT;
>> +
>> +        NetReceive(desc_p->dmamac_addr, length);
>> +
>> +        /*
>> +         * Make the current descriptor valid again and go to
>> +         * the next one
>> +         */
>> +        desc_p->txrx_status |= DESC_RXSTS_OWNBYDMA;
>> +
>> +        /* Test the wrap-around condition. */
>> +        if (++desc_num >= CONFIG_RX_DESCR_NUM)
>> +            desc_num = 0;
>> +    }
>> +
>> +    priv->rx_currdescnum = desc_num;
>> +
>> +    return length;
>> +}
>> +
>> +static void dw_eth_halt(struct eth_device *dev)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +
>> +    mac_reset(dev);
>> +    priv->tx_currdescnum = priv->rx_currdescnum = 0;
>> +}
>> +
>> +static int eth_mdio_read(struct eth_device *dev, u8 addr, u8 reg, u16
>> *val)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    struct eth_mac_regs *mac_p = priv->mac_regs_p;
>> +    u32 miiaddr;
>> +    int timeout = CONFIG_MDIO_TIMEOUT;
>> +
>> +    miiaddr = ((addr << MIIADDRSHIFT) & MII_ADDRMSK) | \
>> +          ((reg << MIIREGSHIFT) & MII_REGMSK);
>> +
>> +    writel(miiaddr | MII_CLKRANGE_150_250M | MII_BUSY, &mac_p->miiaddr);
>> +
>> +    do {
>> +        if (!(readl(&mac_p->miiaddr) & MII_BUSY))
>> +            break;
>> +        udelay(1000);
>> +    } while (timeout--);
>> +
>> +    *val = readl(&mac_p->miidata);
>> +
>> +    if (timeout < 0)
>> +        return -1;
>> +
>> +    return 0;
>> +}
>> +
>> +static int eth_mdio_write(struct eth_device *dev, u8 addr, u8 reg,
>> u16 val)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    struct eth_mac_regs *mac_p = priv->mac_regs_p;
>> +    u32 miiaddr;
>> +    int timeout = CONFIG_MDIO_TIMEOUT;
>> +    u16 value;
>> +
>> +    writel(val, &mac_p->miidata);
>> +    miiaddr = ((addr << MIIADDRSHIFT) & MII_ADDRMSK) | \
>> +          ((reg << MIIREGSHIFT) & MII_REGMSK) | MII_WRITE;
>> +
>> +    writel(miiaddr | MII_CLKRANGE_150_250M | MII_BUSY, &mac_p->miiaddr);
>> +
>> +    do {
>> +        if (!(readl(&mac_p->miiaddr) & MII_BUSY))
>> +            break;
>> +        udelay(1000);
>> +    } while (timeout--);
>> +
>> +    /* Needed as a fix for ST-Phy */
>> +    eth_mdio_read(dev, addr, reg, &value);
>> +
>> +    if (timeout < 0)
>> +        return -1;
>> +
>> +    return 0;
>> +}
>> +
>> +static int find_phy(struct eth_device *dev)
>> +{
>> +    int phy_addr = 0;
>> +    u16 ctrl, oldctrl;
>> +
>> +    do {
>> +        eth_mdio_read(dev, phy_addr, PHY_BMCR, &ctrl);
>> +        oldctrl = ctrl & PHY_BMCR_AUTON;
>> +
>> +        ctrl ^= PHY_BMCR_AUTON;
>> +        eth_mdio_write(dev, phy_addr, PHY_BMCR, ctrl);
>> +        eth_mdio_read(dev, phy_addr, PHY_BMCR, &ctrl);
>> +        ctrl &= PHY_BMCR_AUTON;
>> +
>> +        if (ctrl == oldctrl) {
>> +            phy_addr++;
>> +        } else {
>> +            ctrl ^= PHY_BMCR_AUTON;
>> +            eth_mdio_write(dev, phy_addr, PHY_BMCR, ctrl);
>> +
>> +            return phy_addr;
>> +        }
>> +    } while (phy_addr < 32);
>> +
>> +    return -1;
>> +}
>> +
>> +static int dw_reset_phy(struct eth_device *dev)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    u16 ctrl;
>> +    int timeout = CONFIG_PHYRESET_TIMEOUT;
>> +    u32 phy_addr = priv->address;
>> +
>> +    eth_mdio_write(dev, phy_addr, PHY_BMCR, PHY_BMCR_RESET);
>> +    do {
>> +        eth_mdio_read(dev, phy_addr, PHY_BMCR, &ctrl);
>> +        if (!(ctrl & PHY_BMCR_RESET))
>> +            break;
>> +        udelay(1000);
>> +    } while (timeout--);
>> +
>> +    if (timeout < 0)
>> +        return -1;
>> +
>> +#ifdef CONFIG_PHY_RESET_DELAY
>> +    udelay(CONFIG_PHY_RESET_DELAY);
>> +#endif
>> +    return 0;
>> +}
>> +
>> +static int configure_phy(struct eth_device *dev)
>> +{
>> +    struct dw_eth_dev *priv = dev->priv;
>> +    int phy_addr;
>> +    u16 bmcr, ctrl;
>> +#if defined(CONFIG_DW_AUTONEG)
>> +    u16 bmsr;
>> +    u32 timeout;
>> +    u16 anlpar, btsr;
> 
> CONFIG_DW_AUTONEG is never defined.
> This is dead code
> Remove all instances

The idea is that this is a generic driver which supports Autonegotiation
by enabling this #define in include/configs/xxx.h

>> +#endif
>> +    phy_addr = find_phy(dev);
>> +    priv->address = phy_addr;
>> +
>> +    if (phy_addr < 0)
>> +        return -1;
>> +
>> +    if (dw_reset_phy(dev) < 0)
>> +        return -1;
>> +
>> +#if defined(CONFIG_DW_AUTONEG)
>> +    bmcr = PHY_BMCR_AUTON | PHY_BMCR_RST_NEG | PHY_BMCR_100MB | \
>> +           PHY_BMCR_DPLX | PHY_BMCR_1000_MBPS;
>> +#else
>> +    bmcr = PHY_BMCR_100MB | PHY_BMCR_DPLX;
>> +
>> +#if defined(CONFIG_DW_SPEED10M)
>> +    bmcr &= ~PHY_BMCR_100MB;
> 
> CONFIG_DW_SPEED10M is never defined
> This is dead code
> Remove

Again, the driver supports 10Mbps. Its upto the board configuration to
use it

>> +#endif
>> +#if defined(CONFIG_DW_DUPLEXHALF)
> 
> CONFIG_DW_DUPLEXHALF is never defined
> Remove

Same explanation as above

>> +    bmcr &= ~PHY_BMCR_DPLX;
>> +#endif
>> +#endif
>> +    if (eth_mdio_write(dev, phy_addr, PHY_BMCR, bmcr) < 0)
>> +        return -1;
>> +
>> +    /* Read the phy status register and populate priv structure */
>> +#if defined(CONFIG_DW_AUTONEG)
>> +    timeout = CONFIG_AUTONEG_TIMEOUT;
>> +    do {
>> +        eth_mdio_read(dev, phy_addr, PHY_BMSR, &bmsr);
>> +        if (bmsr & PHY_BMSR_AUTN_COMP)
>> +            break;
>> +        udelay(1000);
>> +    } while (timeout--);
>> +
>> +    eth_mdio_read(dev, phy_addr, PHY_ANLPAR, &anlpar);
>> +    eth_mdio_read(dev, phy_addr, PHY_1000BTSR, &btsr);
>> +
>> +    if (btsr & (PHY_1000BTSR_1000FD | PHY_1000BTSR_1000HD)) {
>> +        priv->speed = SPEED_1000M;
>> +        if (btsr & PHY_1000BTSR_1000FD)
>> +            priv->duplex = FULL_DUPLEX;
>> +        else
>> +            priv->duplex = HALF_DUPLEX;
>> +    } else {
>> +        if (anlpar & PHY_ANLPAR_100)
>> +            priv->speed = SPEED_100M;
>> +        else
>> +            priv->speed = SPEED_10M;
>> +
>> +        if (anlpar & (PHY_ANLPAR_10FD | PHY_ANLPAR_TXFD))
>> +            priv->duplex = FULL_DUPLEX;
>> +        else
>> +            priv->duplex = HALF_DUPLEX;
>> +    }
>> +#else
>> +    if (eth_mdio_read(dev, phy_addr, PHY_BMCR, &ctrl) < 0)
>> +        return -1;
>> +
>> +    if (ctrl & PHY_BMCR_DPLX)
>> +        priv->duplex = FULL_DUPLEX;
>> +    else
>> +        priv->duplex = HALF_DUPLEX;
>> +
>> +    if (ctrl & PHY_BMCR_1000_MBPS)
>> +        priv->speed = SPEED_1000M;
>> +    else if (ctrl & PHY_BMCR_100_MBPS)
>> +        priv->speed = SPEED_100M;
>> +    else
>> +        priv->speed = SPEED_10M;
>> +#endif
>> +    return 0;
>> +}
>> +
>> +#if defined(CONFIG_MII) || defined(CONFIG_CMD_MII)
> 
> Checking for both CONFIG_MII and CONFIG_CMD_MII
> is standard practice. But shouldn't CONFIG_MII be enough?
> 
Yes, I also thought about this for a little while.
I think this is because CONFIG_CMD_MII can be defined independently

>> +static int dw_mii_read(char *devname, u8 addr, u8 reg, u16 *val)
>> +{
>> +    struct eth_device *dev;
>> +
>> +    dev = eth_get_dev_by_name(devname);
>> +    if (dev)
>> +        eth_mdio_read(dev, addr, reg, val);
>> +
>> +    return 0;
>> +}
>> +
>> +static int dw_mii_write(char *devname, u8 addr, u8 reg, u16 val)
>> +{
>> +    struct eth_device *dev;
>> +
>> +    dev = eth_get_dev_by_name(devname);
>> +    if (dev)
>> +        eth_mdio_write(dev, addr, reg, val);
>> +
>> +    return 0;
>> +}
>> +#endif
>> +
>> +int designware_initialize(u32 id, ulong base_addr)
>> +{
>> +    struct eth_device *dev;
>> +    struct dw_eth_dev *priv;
>> +
>> +    dev = (struct eth_device *) malloc(sizeof(struct eth_device));
>> +    if (!dev)
>> +        return -ENOMEM;
>> +
>> +    /*
>> +     * Since the priv structure contains the descriptors which need a
>> strict
>> +     * buswidth alignment, memalign is used to allocate memory
>> +     */
>> +    priv = (struct dw_eth_dev *) memalign(16, sizeof(struct
>> dw_eth_dev));
>> +    if (!priv) {
>> +        free(dev);
>> +        return -ENOMEM;
>> +    }
>> +
>> +    memset(dev, 0, sizeof(struct eth_device));
>> +    memset(priv, 0, sizeof(struct dw_eth_dev));
>> +
>> +    sprintf(dev->name, "mii%d", id);
>> +    dev->iobase = (int)base_addr;
>> +    dev->priv = priv;
>> +
>> +    eth_getenv_enetaddr_by_index(id, &dev->enetaddr[0]);
>> +
>> +    priv->dev = dev;
>> +    priv->mac_regs_p = (struct eth_mac_regs *)base_addr;
>> +    priv->dma_regs_p = (struct eth_dma_regs *)(base_addr +
>> +            DW_DMA_BASE_OFFSET);
>> +
>> +    if (mac_reset(dev) < 0)
>> +        return -1;
>> +
>> +    if (configure_phy(dev) < 0) {
>> +        printf("Phy could not be configured\n");
>> +        return -1;
>> +    }
>> +
>> +    dev->init = dw_eth_init;
>> +    dev->send = dw_eth_send;
>> +    dev->recv = dw_eth_recv;
>> +    dev->halt = dw_eth_halt;
>> +
>> +    eth_register(dev);
>> +
>> +#if defined(CONFIG_MII) || defined(CONFIG_CMD_MII)
>> +    miiphy_register(dev->name, dw_mii_read, dw_mii_write);
>> +#endif
>> +    return 1;
>> +}
>> diff --git a/drivers/net/designware.h b/drivers/net/designware.h
>> new file mode 100644
>> index 0000000..8f7e1ef
>> --- /dev/null
>> +++ b/drivers/net/designware.h
>> @@ -0,0 +1,264 @@
>> +/*
>> + * (C) Copyright 2009
>> + * Vipin Kumar, ST Micoelectronics, vipin.kumar at st.com.
>> + *
>> + * 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.
>> + *
>> + * 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
>> + */
>> +
>> +#ifndef _DW_ETH_H
>> +#define _DW_ETH_H
>> +
>> +#define CONFIG_TX_DESCR_NUM    16
>> +#define CONFIG_RX_DESCR_NUM    16
>> +#define CONFIG_ETH_BUFSIZE    2048
>> +#define TX_TOTAL_BUFSIZE    (CONFIG_ETH_BUFSIZE * CONFIG_TX_DESCR_NUM)
>> +#define RX_TOTAL_BUFSIZE    (CONFIG_ETH_BUFSIZE * CONFIG_RX_DESCR_NUM)
>> +
>> +#define CONFIG_MACRESET_TIMEOUT    (3 * CONFIG_SYS_HZ)
>> +#define CONFIG_MDIO_TIMEOUT    (3 * CONFIG_SYS_HZ)
>> +#define CONFIG_PHYRESET_TIMEOUT    (3 * CONFIG_SYS_HZ)
>> +#define CONFIG_AUTONEG_TIMEOUT    (5 * CONFIG_SYS_HZ)
>> +
>> +struct eth_mac_regs {
>> +    u32 conf;        /* 0x00 */
>> +    u32 framefilt;        /* 0x04 */
>> +    u32 hashtablehigh;    /* 0x08 */
>> +    u32 hashtablelow;    /* 0x0c */
>> +    u32 miiaddr;        /* 0x10 */
>> +    u32 miidata;        /* 0x14 */
>> +    u32 flowcontrol;    /* 0x18 */
>> +    u32 vlantag;        /* 0x1c */
>> +    u32 version;        /* 0x20 */
>> +    u8 reserved_1[0x38 - 0x24];
> 
> This hex subtraction could be simplified.
> Just the single immediate would be easier to read.
> 
This hex subtraction need not be performed. Actually, this is simply the
next register offset - current register offset + 4
This way manual additions are not needed and the reserved area in device
registers can be depicted easily. Do you still want me to do it the
other way ?

Regards
Vipin

> Tom
> 
>> +    u32 intreg;        /* 0x38 */
>> +    u32 intmask;        /* 0x3c */
>> +    u32 macaddr0hi;        /* 0x40 */
>> +    u32 macaddr0lo;        /* 0x44 */
>> +};
>> +
>> +/* MAC configuration register definitions */
>> +#define FRAMEBURSTENABLE    (1 << 21)
>> +#define MII_PORTSELECT        (1 << 15)
>> +#define FES_100            (1 << 14)
>> +#define DISABLERXOWN        (1 << 13)
>> +#define FULLDPLXMODE        (1 << 11)
>> +#define RXENABLE        (1 << 2)
>> +#define TXENABLE        (1 << 3)
>> +
>> +/* MII address register definitions */
>> +#define MII_BUSY        (1 << 0)
>> +#define MII_WRITE        (1 << 1)
>> +#define MII_CLKRANGE_60_100M    (0)
>> +#define MII_CLKRANGE_100_150M    (0x4)
>> +#define MII_CLKRANGE_20_35M    (0x8)
>> +#define MII_CLKRANGE_35_60M    (0xC)
>> +#define MII_CLKRANGE_150_250M    (0x10)
>> +#define MII_CLKRANGE_250_300M    (0x14)
>> +
>> +#define MIIADDRSHIFT        (11)
>> +#define MIIREGSHIFT        (6)
>> +#define MII_REGMSK        (0x1F << 6)
>> +#define MII_ADDRMSK        (0x1F << 11)
>> +
>> +
>> +struct eth_dma_regs {
>> +    u32 busmode;        /* 0x00 */
>> +    u32 txpolldemand;    /* 0x04 */
>> +    u32 rxpolldemand;    /* 0x08 */
>> +    u32 rxdesclistaddr;    /* 0x0c */
>> +    u32 txdesclistaddr;    /* 0x10 */
>> +    u32 status;        /* 0x14 */
>> +    u32 opmode;        /* 0x18 */
>> +    u32 intenable;        /* 0x1c */
>> +    u8 reserved[0x48 - 0x20];
>> +    u32 currhosttxdesc;    /* 0x48 */
>> +    u32 currhostrxdesc;    /* 0x4c */
>> +    u32 currhosttxbuffaddr;    /* 0x50 */
>> +    u32 currhostrxbuffaddr;    /* 0x54 */
>> +};
>> +
>> +#define DW_DMA_BASE_OFFSET    (0x1000)
>> +
>> +/* Bus mode register definitions */
>> +#define FIXEDBURST        (1 << 16)
>> +#define PRIORXTX_41        (3 << 14)
>> +#define PRIORXTX_31        (2 << 14)
>> +#define PRIORXTX_21        (1 << 14)
>> +#define PRIORXTX_11        (0 << 14)
>> +#define BURST_1            (1 << 8)
>> +#define BURST_2            (2 << 8)
>> +#define BURST_4            (4 << 8)
>> +#define BURST_8            (8 << 8)
>> +#define BURST_16        (16 << 8)
>> +#define BURST_32        (32 << 8)
>> +#define RXHIGHPRIO        (1 << 1)
>> +#define DMAMAC_SRST        (1 << 0)
>> +
>> +/* Poll demand definitions */
>> +#define POLL_DATA        (0xFFFFFFFF)
>> +
>> +/* Operation mode definitions */
>> +#define STOREFORWARD        (1 << 21)
>> +#define FLUSHTXFIFO        (1 << 20)
>> +#define TXSTART            (1 << 13)
>> +#define TXSECONDFRAME        (1 << 2)
>> +#define RXSTART            (1 << 1)
>> +
>> +/* Descriptior related definitions */
>> +#define MAC_MAX_FRAME_SZ    (2048)
>> +
>> +struct dmamacdescr {
>> +    u32 txrx_status;
>> +    u32 dmamac_cntl;
>> +    void *dmamac_addr;
>> +    struct dmamacdescr *dmamac_next;
>> +};
>> +
>> +/*
>> + * txrx_status definitions
>> + */
>> +
>> +/* tx status bits definitions */
>> +#if defined(CONFIG_DW_ALTDESCRIPTOR)
>> +
>> +#define DESC_TXSTS_OWNBYDMA        (1 << 31)
>> +#define DESC_TXSTS_TXINT        (1 << 30)
>> +#define DESC_TXSTS_TXLAST        (1 << 29)
>> +#define DESC_TXSTS_TXFIRST        (1 << 28)
>> +#define DESC_TXSTS_TXCRCDIS        (1 << 27)
>> +
>> +#define DESC_TXSTS_TXPADDIS        (1 << 26)
>> +#define DESC_TXSTS_TXCHECKINSCTRL    (3 << 22)
>> +#define DESC_TXSTS_TXRINGEND        (1 << 21)
>> +#define DESC_TXSTS_TXCHAIN        (1 << 20)
>> +#define DESC_TXSTS_MSK            (0x1FFFF << 0)
>> +
>> +#else
>> +
>> +#define DESC_TXSTS_OWNBYDMA        (1 << 31)
>> +#define DESC_TXSTS_MSK            (0x1FFFF << 0)
>> +
>> +#endif
>> +
>> +/* rx status bits definitions */
>> +#define DESC_RXSTS_OWNBYDMA        (1 << 31)
>> +#define DESC_RXSTS_DAFILTERFAIL        (1 << 30)
>> +#define DESC_RXSTS_FRMLENMSK        (0x3FFF << 16)
>> +#define DESC_RXSTS_FRMLENSHFT        (16)
>> +
>> +#define DESC_RXSTS_ERROR        (1 << 15)
>> +#define DESC_RXSTS_RXTRUNCATED        (1 << 14)
>> +#define DESC_RXSTS_SAFILTERFAIL        (1 << 13)
>> +#define DESC_RXSTS_RXIPC_GIANTFRAME    (1 << 12)
>> +#define DESC_RXSTS_RXDAMAGED        (1 << 11)
>> +#define DESC_RXSTS_RXVLANTAG        (1 << 10)
>> +#define DESC_RXSTS_RXFIRST        (1 << 9)
>> +#define DESC_RXSTS_RXLAST        (1 << 8)
>> +#define DESC_RXSTS_RXIPC_GIANT        (1 << 7)
>> +#define DESC_RXSTS_RXCOLLISION        (1 << 6)
>> +#define DESC_RXSTS_RXFRAMEETHER        (1 << 5)
>> +#define DESC_RXSTS_RXWATCHDOG        (1 << 4)
>> +#define DESC_RXSTS_RXMIIERROR        (1 << 3)
>> +#define DESC_RXSTS_RXDRIBBLING        (1 << 2)
>> +#define DESC_RXSTS_RXCRC        (1 << 1)
>> +
>> +/*
>> + * dmamac_cntl definitions
>> + */
>> +
>> +/* tx control bits definitions */
>> +#if defined(CONFIG_DW_ALTDESCRIPTOR)
>> +
>> +#define DESC_TXCTRL_SIZE1MASK        (0x1FFF << 0)
>> +#define DESC_TXCTRL_SIZE1SHFT        (0)
>> +#define DESC_TXCTRL_SIZE2MASK        (0x1FFF << 16)
>> +#define DESC_TXCTRL_SIZE2SHFT        (16)
>> +
>> +#else
>> +
>> +#define DESC_TXCTRL_TXINT        (1 << 31)
>> +#define DESC_TXCTRL_TXLAST        (1 << 30)
>> +#define DESC_TXCTRL_TXFIRST        (1 << 29)
>> +#define DESC_TXCTRL_TXCHECKINSCTRL    (3 << 27)
>> +#define DESC_TXCTRL_TXCRCDIS        (1 << 26)
>> +#define DESC_TXCTRL_TXRINGEND        (1 << 25)
>> +#define DESC_TXCTRL_TXCHAIN        (1 << 24)
>> +
>> +#define DESC_TXCTRL_SIZE1MASK        (0x7FF << 0)
>> +#define DESC_TXCTRL_SIZE1SHFT        (0)
>> +#define DESC_TXCTRL_SIZE2MASK        (0x7FF << 11)
>> +#define DESC_TXCTRL_SIZE2SHFT        (11)
>> +
>> +#endif
>> +
>> +/* rx control bits definitions */
>> +#if defined(CONFIG_DW_ALTDESCRIPTOR)
>> +
>> +#define DESC_RXCTRL_RXINTDIS        (1 << 31)
>> +#define DESC_RXCTRL_RXRINGEND        (1 << 15)
>> +#define DESC_RXCTRL_RXCHAIN        (1 << 14)
>> +
>> +#define DESC_RXCTRL_SIZE1MASK        (0x1FFF << 0)
>> +#define DESC_RXCTRL_SIZE1SHFT        (0)
>> +#define DESC_RXCTRL_SIZE2MASK        (0x1FFF << 16)
>> +#define DESC_RXCTRL_SIZE2SHFT        (16)
>> +
>> +#else
>> +
>> +#define DESC_RXCTRL_RXINTDIS        (1 << 31)
>> +#define DESC_RXCTRL_RXRINGEND        (1 << 25)
>> +#define DESC_RXCTRL_RXCHAIN        (1 << 24)
>> +
>> +#define DESC_RXCTRL_SIZE1MASK        (0x7FF << 0)
>> +#define DESC_RXCTRL_SIZE1SHFT        (0)
>> +#define DESC_RXCTRL_SIZE2MASK        (0x7FF << 11)
>> +#define DESC_RXCTRL_SIZE2SHFT        (11)
>> +
>> +#endif
>> +
>> +struct dw_eth_dev {
>> +    u32 address;
>> +    u32 speed;
>> +    u32 duplex;
>> +    u32 tx_currdescnum;
>> +    u32 rx_currdescnum;
>> +    u32 padding;
>> +
>> +    struct dmamacdescr tx_mac_descrtable[CONFIG_TX_DESCR_NUM];
>> +    struct dmamacdescr rx_mac_descrtable[CONFIG_RX_DESCR_NUM];
>> +
>> +    char txbuffs[TX_TOTAL_BUFSIZE];
>> +    char rxbuffs[RX_TOTAL_BUFSIZE];
>> +
>> +    struct eth_mac_regs *mac_regs_p;
>> +    struct eth_dma_regs *dma_regs_p;
>> +
>> +    struct eth_device *dev;
>> +} __attribute__ ((aligned(8)));
>> +
>> +/* Speed specific definitions */
>> +#define SPEED_10M        1
>> +#define SPEED_100M        2
>> +#define SPEED_1000M        3
>> +
>> +/* Duplex mode specific definitions */
>> +#define HALF_DUPLEX        1
>> +#define FULL_DUPLEX        2
>> +
>> +#endif
>> diff --git a/include/netdev.h b/include/netdev.h
>> index 1dd80f0..808fdde 100644
>> --- a/include/netdev.h
>> +++ b/include/netdev.h
>> @@ -47,6 +47,7 @@ int bfin_EMAC_initialize(bd_t *bis);
>>  int cs8900_initialize(u8 dev_num, int base_addr);
>>  int dc21x4x_initialize(bd_t *bis);
>>  int davinci_emac_initialize(void);
>> +int designware_initialize(u32 id, ulong base_addr);
>>  int dnet_eth_initialize(int id, void *regs, unsigned int phy_addr);
>>  int e1000_initialize(bd_t *bis);
>>  int eepro100_initialize(bd_t *bis);
> 
> 



More information about the U-Boot mailing list