From patchwork Mon Feb 9 17:57:42 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: frederic rodo X-Patchwork-Id: 22757 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by ozlabs.org (Postfix) with ESMTP id E1CEDDDDA8 for ; Tue, 10 Feb 2009 04:57:51 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754155AbZBIR5q (ORCPT ); Mon, 9 Feb 2009 12:57:46 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753528AbZBIR5q (ORCPT ); Mon, 9 Feb 2009 12:57:46 -0500 Received: from mail-ew0-f21.google.com ([209.85.219.21]:42659 "EHLO mail-ew0-f21.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753037AbZBIR5p (ORCPT ); Mon, 9 Feb 2009 12:57:45 -0500 Received: by ewy14 with SMTP id 14so2677090ewy.13 for ; Mon, 09 Feb 2009 09:57:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:date:message-id:subject :from:to:cc:content-type; bh=vG7T3S191yupZELLWI+scXsyxQdrN4BU39R5x2M5O/g=; b=qs2KfbPjxwYjTNOo+pjkcIVRrzEDoXXf+31SssgHCc3r94o4ddV81sPTU1f7p10ObL eNbmR+3bP1iR3V0djdFHeYY1m/OgJ15eixvpTQ2AGFMvy3W0M3U+WJ+zmRF2u1cTGiVk lEcThOKaoVRAAQSCDJ0WeEhiU/B6lxa1/u58U= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:date:message-id:subject:from:to:cc:content-type; b=Zw/6hXrcpIskS491XwbE6g5x6XMiAdOJJnUbvvql1zC4VuLYR707/yRnQzFflHIOU4 VB7eKRyKcld2a8sCIRhnobqWCVgK8R/nZzPT1nbIuIhxZEV/vTF0d7eN1hNgTwkTeMCm BcCHbqKtUhdfX2Ir6jlEWAdten8aWtLm/ogRA= MIME-Version: 1.0 Received: by 10.210.76.4 with SMTP id y4mr1067766eba.176.1234202262254; Mon, 09 Feb 2009 09:57:42 -0800 (PST) Date: Mon, 9 Feb 2009 18:57:42 +0100 Message-ID: Subject: [PATCH] MX27 FEC add mii bus From: frederic rodo To: Sascha Hauer Cc: linux-arm-kernel@lists.arm.linux.org.uk, netdev@vger.kernel.org, gerg@snapgear.com Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Hi, I have made the corrections you have asked, and I test it successfully with my custom board and a pcm038 thanks From c68bdbea385e3f97cf7c53dd6c754808d0af8d50 Mon Sep 17 00:00:00 2001 From: Frederic Rodo Date: Mon, 9 Feb 2009 17:47:09 +0100 Subject: [PATCH] fec: add mii bus Signed-off-by: Frederic Rodo --- drivers/net/Kconfig | 1 + drivers/net/fec.c | 374 ++++++++++++++++++++++++++++++++++++++++++++++++--- include/linux/fec.h | 26 ++++ 3 files changed, 382 insertions(+), 19 deletions(-) diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig index 2d7cd61..c6a7774 100644 --- a/drivers/net/Kconfig +++ b/drivers/net/Kconfig @@ -1830,6 +1830,7 @@ config 68360_ENET config FEC bool "FEC ethernet controller (of ColdFire and some i.MX CPUs)" depends on M523x || M527x || M5272 || M528x || M520x || MACH_MX27 + select PHYLIB if MACH_MX27 help Say Y here if you want to use the built-in 10/100 Fast ethernet controller on some Motorola ColdFire and Freescale i.MX processors. diff --git a/drivers/net/fec.c b/drivers/net/fec.c index ea23baa..e087125 100644 --- a/drivers/net/fec.c +++ b/drivers/net/fec.c @@ -40,6 +40,7 @@ #include #include #include +#include #include @@ -51,6 +52,7 @@ #include "fec.h" #ifdef CONFIG_ARCH_MXC +#include #include #define FEC_ALIGNMENT 0xf #else @@ -110,8 +112,6 @@ static unsigned char fec_mac_default[] = { #define FEC_FLASHMAC 0 #endif -#endif /* FEC_LEGACY */ - /* Forward declarations of some structures to support different PHYs */ @@ -130,6 +130,8 @@ typedef struct { const phy_cmd_t *shutdown; } phy_info_t; +#endif /* FEC_LEGACY */ + /* The number of Tx and Rx buffers. These are allocated from the page * pool. The code may assume these are power of two, so it it best * to keep them that size. @@ -213,13 +215,13 @@ struct fec_enet_private { uint tx_full; /* hold while accessing the HW like ringbuffer for tx/rx but not MAC */ spinlock_t hw_lock; +#ifdef FEC_LEGACY /* hold while accessing the mii_list_t() elements */ spinlock_t mii_lock; uint phy_id; uint phy_id_done; uint phy_status; - uint phy_speed; phy_info_t const *phy; struct work_struct phy_task; @@ -228,16 +230,25 @@ struct fec_enet_private { uint phy_addr; + int old_link; +#else + struct platform_device *pdev; + struct mii_bus *mii_bus; + struct phy_device *phy_dev; + int mii_timeout; +#endif + uint phy_speed; int index; int opened; int link; - int old_link; int full_duplex; }; static int fec_enet_open(struct net_device *dev); static int fec_enet_start_xmit(struct sk_buff *skb, struct net_device *dev); +#ifdef FEC_LEGACY static void fec_enet_mii(struct net_device *dev); +#endif static irqreturn_t fec_enet_interrupt(int irq, void * dev_id); static void fec_enet_tx(struct net_device *dev); static void fec_enet_rx(struct net_device *dev); @@ -248,6 +259,11 @@ static void fec_stop(struct net_device *dev); static void fec_set_mac_address(struct net_device *dev); +/* Transmitter timeout. +*/ +#define TX_TIMEOUT (2*HZ) + +#ifdef FEC_LEGACY /* MII processing. We keep this as simple as possible. Requests are * placed on the list (if there is room). When the request is finished * by the MII, an optional function may be called. @@ -274,10 +290,6 @@ static int mii_queue(struct net_device *dev, int request, (VAL & 0xffff)) #define mk_mii_end 0 -/* Transmitter timeout. -*/ -#define TX_TIMEOUT (2*HZ) - /* Register definitions for the PHY. */ @@ -310,6 +322,18 @@ static int mii_queue(struct net_device *dev, int request, #define PHY_STAT_100HDX 0x4000 /* 100 Mbit half duplex selected */ #define PHY_STAT_100FDX 0x8000 /* 100 Mbit full duplex selected */ +#else +/* FEC MMFR bits definition */ +#define FEC_MMFR_ST (1 << 30) +#define FEC_MMFR_OP_READ (2 << 28) +#define FEC_MMFR_OP_WRITE (1 << 28) +#define FEC_MMFR_PA(v) ((v & 0x1f) << 23) +#define FEC_MMFR_RA(v) ((v & 0x1f) << 18) +#define FEC_MMFR_TA (2 << 16) +#define FEC_MMFR_DATA(v) (v & 0xffff) + +#define FEC_MII_TRIES 10000 +#endif static int fec_enet_start_xmit(struct sk_buff *skb, struct net_device *dev) @@ -490,10 +514,12 @@ fec_enet_interrupt(int irq, void * dev_id) fec_enet_tx(dev); } +#ifdef FEC_LEGACY if (int_events & FEC_ENET_MII) { ret = IRQ_HANDLED; fec_enet_mii(dev); } +#endif } while (int_events); @@ -711,6 +737,7 @@ while (!((status = bdp->cbd_sc) & BD_ENET_RX_EMPTY)) { } +#ifdef FEC_LEGACY /* called from interrupt context */ static void fec_enet_mii(struct net_device *dev) @@ -1238,6 +1265,7 @@ static phy_info_t const * const phy_info[] = { NULL }; +#endif /* FEC_LEGACY */ /* ------------------------------------------------------------------------- */ #ifdef HAVE_mii_link_interrupt static irqreturn_t @@ -1726,6 +1754,7 @@ static void __inline__ fec_phy_ack_intr(void) /* ------------------------------------------------------------------------- */ +#ifdef FEC_LEGACY static void mii_display_status(struct net_device *dev) { struct fec_enet_private *fep = netdev_priv(dev); @@ -1927,9 +1956,7 @@ mii_discover_phy(uint mii_reg, struct net_device *dev) printk("FEC: No PHY device found.\n"); /* Disable external MII interface */ fecp->fec_mii_speed = fep->phy_speed = 0; -#ifdef FREC_LEGACY fec_disable_phy_intr(); -#endif } } @@ -1954,6 +1981,281 @@ mii_link_interrupt(int irq, void * dev_id) return IRQ_HANDLED; } #endif +#else +static void fec_enet_handle_link_change(struct net_device *dev) +{ + struct fec_enet_private *fep = netdev_priv(dev); + struct phy_device *phy_dev = fep->phy_dev; + unsigned long flags; + + int status_change = 0; + + spin_lock_irqsave(&fep->hw_lock, flags); + + /* Prevent a state halted on mii error */ + if (fep->mii_timeout && phy_dev->state == PHY_HALTED) { + phy_dev->state = PHY_RESUMING; + goto spin_unlock; + } + + if (phy_dev->link) { + if (fep->full_duplex != phy_dev->duplex) { + fec_restart(dev, phy_dev->duplex); + status_change = 1; + } + } + + if (phy_dev->link != fep->link) { + fep->link = phy_dev->link; + if (phy_dev->link) + fec_restart(dev, phy_dev->duplex); + else + fec_stop(dev); + status_change = 1; + } +spin_unlock: + spin_unlock_irqrestore(&fep->hw_lock, flags); + + if (status_change) + phy_print_status(phy_dev); +} + +/* Phy section + * NOTE: a mii transaction is during around 25 us, so polling it... + */ +static int fec_enet_mdio_read(struct mii_bus *bus, int mii_id, int regnum) +{ + struct fec_enet_private *fep = bus->priv; + volatile fec_t *ep; + int tries = FEC_MII_TRIES; + + fep->mii_timeout = 0; + ep = fep->hwp; + + /* clear MII end of transfer bit*/ + ep->fec_ievent = FEC_ENET_MII; + + /* start a read op */ + ep->fec_mii_data = FEC_MMFR_ST | FEC_MMFR_OP_READ | + FEC_MMFR_PA(mii_id) | FEC_MMFR_RA(regnum) | + FEC_MMFR_TA; + + /* wait for end of transfer */ + while (!(ep->fec_ievent & FEC_ENET_MII) && --tries) + cpu_relax(); + + if (!tries) { + fep->mii_timeout = 1; + return -ETIMEDOUT; + } + + /* return value */ + return FEC_MMFR_DATA(ep->fec_mii_data); +} + +static int fec_enet_mdio_write(struct mii_bus *bus, int mii_id, int regnum, + u16 value) +{ + struct fec_enet_private *fep = bus->priv; + volatile fec_t *ep; + int tries = FEC_MII_TRIES; + + fep->mii_timeout = 0; + ep = fep->hwp; + + /* clear MII end of transfer bit*/ + ep->fec_ievent = FEC_ENET_MII; + + /* start a read op */ + ep->fec_mii_data = FEC_MMFR_ST | FEC_MMFR_OP_READ | + FEC_MMFR_PA(mii_id) | FEC_MMFR_RA(regnum) | + FEC_MMFR_TA | FEC_MMFR_DATA(value); + + /* wait for end of transfer */ + while (!(ep->fec_ievent & FEC_ENET_MII) && --tries) + cpu_relax(); + + if (!tries) { + fep->mii_timeout = 1; + return -ETIMEDOUT; + } + return 0; +} + +static int fec_enet_mdio_reset(struct mii_bus *bus) +{ + return 0; +} + +static int fec_enet_mii_probe(struct net_device *dev) +{ + struct fec_enet_private *fep = netdev_priv(dev); + struct phy_device *phy_dev = NULL; + int phy_addr; + + /* find the first phy */ + for (phy_addr = 0; phy_addr < PHY_MAX_ADDR; phy_addr++) { + if (fep->mii_bus->phy_map[phy_addr]) { + phy_dev = fep->mii_bus->phy_map[phy_addr]; + break; + } + } + + if (!phy_dev) { + printk(KERN_ERR "%s: no PHY found\n", dev->name); + return -1; + } + + /* attach the mac to the phy */ + phy_dev = phy_connect(dev, phy_dev->dev.bus_id, + &fec_enet_handle_link_change, 0, + PHY_INTERFACE_MODE_MII); + if (IS_ERR(phy_dev)) { + printk(KERN_ERR "%s: Could not attach to PHY\n", dev->name); + return PTR_ERR(phy_dev); + } + + /* mask with MAC supported features */ + phy_dev->supported &= PHY_BASIC_FEATURES; + phy_dev->advertising = phy_dev->supported; + + fep->phy_dev = phy_dev; + fep->link = 0; + fep->full_duplex = 0; + + return 0; +} + +static int fec_enet_mii_init(struct platform_device *pdev) +{ + struct net_device *dev = platform_get_drvdata(pdev); + struct fec_enet_private *fep = netdev_priv(dev); + struct fec_enet_platform_data *pdata; + volatile fec_t *ep; + int err = -ENXIO, i; + + ep = fep->hwp; + + fep->mii_timeout = 0; + /* + * Set MII speed to 2.5 MHz + */ + fep->phy_speed = ((((clk_get_rate(fep->clk) / 2 + 4999999) + / 2500000) / 2) & 0x3F) << 1; + ep->fec_mii_speed = fep->phy_speed; + + fep->mii_bus = mdiobus_alloc(); + if (fep->mii_bus == NULL) { + err = -ENOMEM; + goto err_out; + } + + fep->mii_bus->name = "fec_enet_mii_bus"; + fep->mii_bus->read = &fec_enet_mdio_read; + fep->mii_bus->write = &fec_enet_mdio_write; + fep->mii_bus->reset = &fec_enet_mdio_reset; + snprintf(fep->mii_bus->id, MII_BUS_ID_SIZE, "%x", pdev->id); + fep->mii_bus->priv = fep; + fep->mii_bus->parent = &pdev->dev; + pdata = pdev->dev.platform_data; + + if (pdata) + fep->mii_bus->phy_mask = pdata->phy_mask; + + fep->mii_bus->irq = kmalloc(sizeof(int) * PHY_MAX_ADDR, GFP_KERNEL); + if (!fep->mii_bus->irq) { + err = -ENOMEM; + goto err_out_free_mdiobus; + } + + for (i = 0; i < PHY_MAX_ADDR; i++) + fep->mii_bus->irq[i] = PHY_POLL; + + platform_set_drvdata(dev, fep->mii_bus); + + if (mdiobus_register(fep->mii_bus)) + goto err_out_free_mdio_irq; + + if (fec_enet_mii_probe(dev) != 0) + goto err_out_unregister_bus; + + return 0; + +err_out_unregister_bus: + mdiobus_unregister(fep->mii_bus); +err_out_free_mdio_irq: + kfree(fep->mii_bus->irq); +err_out_free_mdiobus: + mdiobus_free(fep->mii_bus); +err_out: + return err; +} + +static void fec_enet_mii_remove(struct fec_enet_private *fep) +{ + if (fep->phy_dev) + phy_disconnect(fep->phy_dev); + mdiobus_unregister(fep->mii_bus); + kfree(fep->mii_bus->irq); + mdiobus_free(fep->mii_bus); +} + + +static int fec_enet_get_settings(struct net_device *dev, + struct ethtool_cmd *cmd) +{ + struct fec_enet_private *fep = netdev_priv(dev); + struct phy_device *phydev = fep->phy_dev; + + if (!phydev) + return -ENODEV; + + return phy_ethtool_gset(phydev, cmd); +} + +static int fec_enet_set_settings(struct net_device *dev, + struct ethtool_cmd *cmd) +{ + struct fec_enet_private *fep = netdev_priv(dev); + struct phy_device *phydev = fep->phy_dev; + + if (!phydev) + return -ENODEV; + + return phy_ethtool_sset(phydev, cmd); +} + +static void fec_enet_get_drvinfo(struct net_device *dev, + struct ethtool_drvinfo *info) +{ + struct fec_enet_private *fep = netdev_priv(dev); + + strcpy(info->driver, fep->pdev->dev.driver->name); + strcpy(info->version, "Revision: 1.0"); + strcpy(info->bus_info, fep->pdev->dev.bus_id); +} + +static struct ethtool_ops fec_enet_ethtool_ops = { + .get_settings = fec_enet_get_settings, + .set_settings = fec_enet_set_settings, + .get_drvinfo = fec_enet_get_drvinfo, + .get_link = ethtool_op_get_link, +}; + +static int fec_enet_ioctl(struct net_device *dev, struct ifreq *rq, int cmd) +{ + struct fec_enet_private *fep = netdev_priv(dev); + struct phy_device *phydev = fep->phy_dev; + + if (!netif_running(dev)) + return -EINVAL; + + if (!phydev) + return -ENODEV; + + return phy_mii_ioctl(phydev, if_mii(rq), cmd); +} +#endif /* FEC_LEGACY */ static int fec_enet_open(struct net_device *dev) @@ -1965,9 +2267,10 @@ fec_enet_open(struct net_device *dev) */ fec_set_mac_address(dev); - fep->sequence_done = 0; fep->link = 0; +#ifdef FEC_LEGACY + fep->sequence_done = 0; if (fep->phy) { mii_do_cmd(dev, fep->phy->ack_int); mii_do_cmd(dev, fep->phy->config); @@ -1996,6 +2299,10 @@ fec_enet_open(struct net_device *dev) fec_restart(dev, 1); } +#else + /* schedule a link state check */ + phy_start(fep->phy_dev); +#endif netif_start_queue(dev); fep->opened = 1; return 0; /* Success */ @@ -2009,6 +2316,9 @@ fec_enet_close(struct net_device *dev) /* Don't know what to do yet. */ fep->opened = 0; +#ifndef FEC_LEGACY + phy_stop(fep->phy_dev); +#endif netif_stop_queue(dev); fec_stop(dev); @@ -2135,7 +2445,9 @@ int __init fec_enet_init(struct net_device *dev, int index) } spin_lock_init(&fep->hw_lock); +#ifdef FEC_LEGACY spin_lock_init(&fep->mii_lock); +#endif /* Create an Ethernet device instance. */ @@ -2260,28 +2572,28 @@ int __init fec_enet_init(struct net_device *dev, int index) dev->stop = fec_enet_close; dev->set_multicast_list = set_multicast_list; +#ifdef FEC_LEGACY for (i=0; ido_ioctl = fec_enet_ioctl; + dev->ethtool_ops = &fec_enet_ethtool_ops; + fecp->fec_r_cntrl = OPT_FRAME_SIZE | 0x04; fecp->fec_x_cntrl = 0x00; +#endif - /* - * Set MII speed to 2.5 MHz - */ - fep->phy_speed = ((((clk_get_rate(fep->clk) / 2 + 4999999) - / 2500000) / 2) & 0x3F) << 1; - fecp->fec_mii_speed = fep->phy_speed; +#ifdef FEC_LEGACY fec_restart(dev, 0); #endif /* Clear and enable interrupts */ fecp->fec_ievent = 0xffc00000; +#ifdef FEC_LEGACY fecp->fec_imask = (FEC_ENET_TXF | FEC_ENET_RXF | FEC_ENET_MII); /* Queue up command to detect the PHY and initialize the @@ -2290,6 +2602,9 @@ int __init fec_enet_init(struct net_device *dev, int index) fep->phy_id_done = 0; fep->phy_addr = 0; mii_queue(dev, mk_mii_read(MII_REG_PHYIR1), mii_discover_phy); +#else + fecp->fec_imask = (FEC_ENET_TXF | FEC_ENET_RXF); +#endif return 0; } @@ -2316,7 +2631,7 @@ fec_restart(struct net_device *dev, int duplex) /* Clear any outstanding interrupt. */ - fecp->fec_ievent = 0xffc00000; + fecp->fec_ievent = 0xffc00000 & ~FEC_ENET_MII; /* Set station address. */ @@ -2406,7 +2721,11 @@ fec_restart(struct net_device *dev, int duplex) /* Enable interrupts we wish to service. */ +#ifdef FEC_LEGACY fecp->fec_imask = (FEC_ENET_TXF | FEC_ENET_RXF | FEC_ENET_MII); +#else + fecp->fec_imask = (FEC_ENET_TXF | FEC_ENET_RXF); +#endif } static void @@ -2438,7 +2757,11 @@ fec_stop(struct net_device *dev) */ fecp->fec_ievent = FEC_ENET_MII; +#ifdef FEC_LEGACY fecp->fec_imask = FEC_ENET_MII; +#else + fecp->fec_imask = 0; +#endif fecp->fec_mii_speed = fep->phy_speed; } @@ -2500,6 +2823,7 @@ fec_probe(struct platform_device *pdev) memset(fep, 0, sizeof(*fep)); ndev->base_addr = (unsigned long)ioremap(r->start, resource_size(r)); + fep->pdev = pdev; if (!ndev->base_addr) { ret = -ENOMEM; @@ -2535,13 +2859,24 @@ fec_probe(struct platform_device *pdev) if (ret) goto failed_init; + ret = fec_enet_mii_init(pdev); + if (ret) + goto failed_mii_init; + ret = register_netdev(ndev); if (ret) goto failed_register; + printk(KERN_INFO "%s: Freescale FEC PHY driver [%s] " + "(mii_bus:phy_addr=%s, irq=%d)\n", ndev->name, + fep->phy_dev->drv->name, fep->phy_dev->dev.bus_id, + fep->phy_dev->irq); + return 0; failed_register: + fec_enet_mii_remove(fep); +failed_mii_init: failed_init: clk_disable(fep->clk); clk_put(fep->clk); @@ -2568,6 +2903,7 @@ fec_drv_remove(struct platform_device *pdev) platform_set_drvdata(pdev, NULL); fec_stop(ndev); + fec_enet_mii_remove(fep); clk_disable(fep->clk); clk_put(fep->clk); iounmap((void __iomem *)ndev->base_addr); diff --git a/include/linux/fec.h b/include/linux/fec.h new file mode 100644 index 0000000..60a4fc9 --- /dev/null +++ b/include/linux/fec.h @@ -0,0 +1,26 @@ +/* + * Copyright 2009 Frederic Rodo, fred.rodo@gmail.com + * + * 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., 51 Franklin Street, Fifth Floor, Boston, + * MA 02110-1301, USA. + */ + +#ifndef __LINUX_FEC_H +#define __LINUX_FEC_H + +struct fec_enet_platform_data { + u32 phy_mask; /* Set x bit to mask phy at address x */ +}; + +#endif /* __LINUX_FEC_H */