Merge commit 'gcl/merge' into next
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Thu, 18 Jun 2009 01:22:08 +0000 (11:22 +1000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Thu, 18 Jun 2009 01:22:08 +0000 (11:22 +1000)
Manual merge of:
drivers/net/fec_mpc52xx.c

1  2 
drivers/net/fec_mpc52xx.c
drivers/net/fec_mpc52xx_phy.c

index 7d443405bbe271e6ea47220775e5d42d4955414c,5ddf03325d1634f8ba91176c20c0d96f649f6ff5..cc786333d95c87cd84d34a1b189c43f9b60367c1
@@@ -25,7 -25,6 +25,7 @@@
  #include <linux/hardirq.h>
  #include <linux/delay.h>
  #include <linux/of_device.h>
 +#include <linux/of_mdio.h>
  #include <linux/of_platform.h>
  
  #include <linux/netdevice.h>
  
  #define DRIVER_NAME "mpc52xx-fec"
  
 -#define FEC5200_PHYADDR_NONE  (-1)
 -#define FEC5200_PHYADDR_7WIRE (-2)
 -
  /* Private driver data structure */
  struct mpc52xx_fec_priv {
 +      struct net_device *ndev;
        int duplex;
        int speed;
        int r_irq;
        int msg_enable;
  
        /* MDIO link details */
 -      int phy_addr;
 -      unsigned int phy_speed;
 +      unsigned int mdio_speed;
 +      struct device_node *phy_node;
        struct phy_device *phydev;
        enum phy_state link;
 +      int seven_wire_mode;
  };
  
  
@@@ -211,25 -211,85 +211,25 @@@ static void mpc52xx_fec_adjust_link(str
                phy_print_status(phydev);
  }
  
 -static int mpc52xx_fec_init_phy(struct net_device *dev)
 -{
 -      struct mpc52xx_fec_priv *priv = netdev_priv(dev);
 -      struct phy_device *phydev;
 -      char phy_id[BUS_ID_SIZE];
 -
 -      snprintf(phy_id, sizeof(phy_id), "%x:%02x",
 -                      (unsigned int)dev->base_addr, priv->phy_addr);
 -
 -      priv->link = PHY_DOWN;
 -      priv->speed = 0;
 -      priv->duplex = -1;
 -
 -      phydev = phy_connect(dev, phy_id, &mpc52xx_fec_adjust_link, 0, PHY_INTERFACE_MODE_MII);
 -      if (IS_ERR(phydev)) {
 -              dev_err(&dev->dev, "phy_connect failed\n");
 -              return PTR_ERR(phydev);
 -      }
 -      dev_info(&dev->dev, "attached phy %i to driver %s\n",
 -                      phydev->addr, phydev->drv->name);
 -
 -      priv->phydev = phydev;
 -
 -      return 0;
 -}
 -
 -static int mpc52xx_fec_phy_start(struct net_device *dev)
 -{
 -      struct mpc52xx_fec_priv *priv = netdev_priv(dev);
 -      int err;
 -
 -      if (priv->phy_addr < 0)
 -              return 0;
 -
 -      err = mpc52xx_fec_init_phy(dev);
 -      if (err) {
 -              dev_err(&dev->dev, "mpc52xx_fec_init_phy failed\n");
 -              return err;
 -      }
 -
 -      /* reset phy - this also wakes it from PDOWN */
 -      phy_write(priv->phydev, MII_BMCR, BMCR_RESET);
 -      phy_start(priv->phydev);
 -
 -      return 0;
 -}
 -
 -static void mpc52xx_fec_phy_stop(struct net_device *dev)
 -{
 -      struct mpc52xx_fec_priv *priv = netdev_priv(dev);
 -
 -      if (!priv->phydev)
 -              return;
 -
 -      phy_disconnect(priv->phydev);
 -      /* power down phy */
 -      phy_stop(priv->phydev);
 -      phy_write(priv->phydev, MII_BMCR, BMCR_PDOWN);
 -}
 -
 -static void mpc52xx_fec_phy_hw_init(struct mpc52xx_fec_priv *priv)
 -{
 -      struct mpc52xx_fec __iomem *fec = priv->fec;
 -
 -      if (priv->phydev)
 -              return;
 -
 -      out_be32(&fec->mii_speed, priv->phy_speed);
 -}
 -
  static int mpc52xx_fec_open(struct net_device *dev)
  {
        struct mpc52xx_fec_priv *priv = netdev_priv(dev);
        int err = -EBUSY;
  
 +      if (priv->phy_node) {
 +              priv->phydev = of_phy_connect(priv->ndev, priv->phy_node,
 +                                            mpc52xx_fec_adjust_link, 0, 0);
 +              if (!priv->phydev) {
 +                      dev_err(&dev->dev, "of_phy_connect failed\n");
 +                      return -ENODEV;
 +              }
 +              phy_start(priv->phydev);
 +      }
 +
        if (request_irq(dev->irq, &mpc52xx_fec_interrupt, IRQF_SHARED,
                        DRIVER_NAME "_ctrl", dev)) {
                dev_err(&dev->dev, "ctrl interrupt request failed\n");
 -              goto out;
 +              goto free_phy;
        }
        if (request_irq(priv->r_irq, &mpc52xx_fec_rx_interrupt, 0,
                        DRIVER_NAME "_rx", dev)) {
                goto free_irqs;
        }
  
 -      err = mpc52xx_fec_phy_start(dev);
 -      if (err)
 -              goto free_skbs;
 -
        bcom_enable(priv->rx_dmatsk);
        bcom_enable(priv->tx_dmatsk);
  
  
        return 0;
  
 - free_skbs:
 -      mpc52xx_fec_free_rx_buffers(dev, priv->rx_dmatsk);
 -
   free_irqs:
        free_irq(priv->t_irq, dev);
   free_2irqs:
        free_irq(priv->r_irq, dev);
   free_ctrl_irq:
        free_irq(dev->irq, dev);
 - out:
 + free_phy:
 +      if (priv->phydev) {
 +              phy_stop(priv->phydev);
 +              phy_disconnect(priv->phydev);
 +              priv->phydev = NULL;
 +      }
  
        return err;
  }
@@@ -290,12 -352,7 +290,12 @@@ static int mpc52xx_fec_close(struct net
        free_irq(priv->r_irq, dev);
        free_irq(priv->t_irq, dev);
  
 -      mpc52xx_fec_phy_stop(dev);
 +      if (priv->phydev) {
 +              /* power down phy */
 +              phy_stop(priv->phydev);
 +              phy_disconnect(priv->phydev);
 +              priv->phydev = NULL;
 +      }
  
        return 0;
  }
@@@ -639,7 -696,7 +639,7 @@@ static void mpc52xx_fec_hw_init(struct 
        /* set phy speed.
         * this can't be done in phy driver, since it needs to be called
         * before fec stuff (even on resume) */
 -      mpc52xx_fec_phy_hw_init(priv);
 +      out_be32(&fec->mii_speed, priv->mdio_speed);
  }
  
  /**
@@@ -675,7 -732,7 +675,7 @@@ static void mpc52xx_fec_start(struct ne
        rcntrl = FEC_RX_BUFFER_SIZE << 16;      /* max frame length */
        rcntrl |= FEC_RCNTRL_FCE;
  
 -      if (priv->phy_addr != FEC5200_PHYADDR_7WIRE)
 +      if (!priv->seven_wire_mode)
                rcntrl |= FEC_RCNTRL_MII_MODE;
  
        if (priv->duplex == DUPLEX_FULL)
@@@ -741,6 -798,8 +741,6 @@@ static void mpc52xx_fec_stop(struct net
  
        /* Stop FEC */
        out_be32(&fec->ecntrl, in_be32(&fec->ecntrl) & ~FEC_ECNTRL_ETHER_EN);
 -
 -      return;
  }
  
  /* reset fec and bestcomm tasks */
@@@ -758,11 -817,9 +758,11 @@@ static void mpc52xx_fec_reset(struct ne
  
        mpc52xx_fec_hw_init(dev);
  
 -      phy_stop(priv->phydev);
 -      phy_write(priv->phydev, MII_BMCR, BMCR_RESET);
 -      phy_start(priv->phydev);
 +      if (priv->phydev) {
 +              phy_stop(priv->phydev);
 +              phy_write(priv->phydev, MII_BMCR, BMCR_RESET);
 +              phy_start(priv->phydev);
 +      }
  
        bcom_fec_rx_reset(priv->rx_dmatsk);
        bcom_fec_tx_reset(priv->tx_dmatsk);
@@@ -862,6 -919,8 +862,6 @@@ mpc52xx_fec_probe(struct of_device *op
        struct net_device *ndev;
        struct mpc52xx_fec_priv *priv = NULL;
        struct resource mem;
 -      struct device_node *phy_node;
 -      const phandle *phy_handle;
        const u32 *prop;
        int prop_size;
  
                return -ENOMEM;
  
        priv = netdev_priv(ndev);
 +      priv->ndev = ndev;
  
        /* Reserve FEC control zone */
        rv = of_address_to_resource(op->node, 0, &mem);
        ndev->ethtool_ops       = &mpc52xx_fec_ethtool_ops;
        ndev->watchdog_timeo    = FEC_WATCHDOG_TIMEOUT;
        ndev->base_addr         = mem.start;
 +      SET_NETDEV_DEV(ndev, &op->dev);
  
        spin_lock_init(&priv->lock);
  
         */
  
        /* Start with safe defaults for link connection */
 -      priv->phy_addr = FEC5200_PHYADDR_NONE;
        priv->speed = 100;
        priv->duplex = DUPLEX_HALF;
-       priv->mdio_speed = ((mpc52xx_find_ipb_freq(op->node) >> 20) / 5) << 1;
 -      priv->phy_speed = ((mpc5xxx_get_bus_frequency(op->node) >> 20) / 5) << 1;
 -
 -      /* the 7-wire property means don't use MII mode */
 -      if (of_find_property(op->node, "fsl,7-wire-mode", NULL))
 -              priv->phy_addr = FEC5200_PHYADDR_7WIRE;
++      priv->mdio_speed = ((mpc5xxx_get_bus_frequency(op->node) >> 20) / 5) << 1;
  
        /* The current speed preconfigures the speed of the MII link */
        prop = of_get_property(op->node, "current-speed", &prop_size);
                priv->duplex = prop[1] ? DUPLEX_FULL : DUPLEX_HALF;
        }
  
 -      /* If there is a phy handle, setup link to that phy */
 -      phy_handle = of_get_property(op->node, "phy-handle", &prop_size);
 -      if (phy_handle && (prop_size >= sizeof(phandle))) {
 -              phy_node = of_find_node_by_phandle(*phy_handle);
 -              prop = of_get_property(phy_node, "reg", &prop_size);
 -              if (prop && (prop_size >= sizeof(u32)))
 -                      if ((*prop >= 0) && (*prop < PHY_MAX_ADDR))
 -                              priv->phy_addr = *prop;
 -              of_node_put(phy_node);
 +      /* If there is a phy handle, then get the PHY node */
 +      priv->phy_node = of_parse_phandle(op->node, "phy-handle", 0);
 +
 +      /* the 7-wire property means don't use MII mode */
 +      if (of_find_property(op->node, "fsl,7-wire-mode", NULL)) {
 +              priv->seven_wire_mode = 1;
 +              dev_info(&ndev->dev, "using 7-wire PHY mode\n");
        }
  
        /* Hardware init */
        mpc52xx_fec_hw_init(ndev);
 -
        mpc52xx_fec_reset_stats(ndev);
  
 -      SET_NETDEV_DEV(ndev, &op->dev);
 -
 -      /* Register the new network device */
        rv = register_netdev(ndev);
        if (rv < 0)
                goto probe_error;
  
 -      /* Now report the link setup */
 -      switch (priv->phy_addr) {
 -       case FEC5200_PHYADDR_NONE:
 -              dev_info(&ndev->dev, "Fixed speed MII link: %i%cD\n",
 -                       priv->speed, priv->duplex ? 'F' : 'H');
 -              break;
 -       case FEC5200_PHYADDR_7WIRE:
 -              dev_info(&ndev->dev, "using 7-wire PHY mode\n");
 -              break;
 -       default:
 -              dev_info(&ndev->dev, "Using PHY at MDIO address %i\n",
 -                       priv->phy_addr);
 -      }
 -
        /* We're done ! */
        dev_set_drvdata(&op->dev, ndev);
  
        /* Error handling - free everything that might be allocated */
  probe_error:
  
 +      if (priv->phy_node)
 +              of_node_put(priv->phy_node);
 +      priv->phy_node = NULL;
 +
        irq_dispose_mapping(ndev->irq);
  
        if (priv->rx_dmatsk)
@@@ -1015,10 -1093,6 +1015,10 @@@ mpc52xx_fec_remove(struct of_device *op
  
        unregister_netdev(ndev);
  
 +      if (priv->phy_node)
 +              of_node_put(priv->phy_node);
 +      priv->phy_node = NULL;
 +
        irq_dispose_mapping(ndev->irq);
  
        bcom_fec_rx_release(priv->rx_dmatsk);
index fec9f245116b7824c2099ccdecf6d4e2b1597b23,176e9b8d7101fc6bbf6869a8cbb25a1a2be58c06..31e6d62b785d81beb13ea7b4f76792975a385127
  #include <linux/netdevice.h>
  #include <linux/phy.h>
  #include <linux/of_platform.h>
 +#include <linux/of_mdio.h>
  #include <asm/io.h>
  #include <asm/mpc52xx.h>
  #include "fec_mpc52xx.h"
  
  struct mpc52xx_fec_mdio_priv {
        struct mpc52xx_fec __iomem *regs;
 +      int mdio_irqs[PHY_MAX_ADDR];
  };
  
  static int mpc52xx_fec_mdio_transfer(struct mii_bus *bus, int phy_id,
@@@ -29,7 -27,7 +29,7 @@@
  {
        struct mpc52xx_fec_mdio_priv *priv = bus->priv;
        struct mpc52xx_fec __iomem *fec;
 -      int tries = 100;
 +      int tries = 3;
  
        value |= (phy_id << FEC_MII_DATA_PA_SHIFT) & FEC_MII_DATA_PA_MSK;
        value |= (reg << FEC_MII_DATA_RA_SHIFT) & FEC_MII_DATA_RA_MSK;
@@@ -40,7 -38,7 +40,7 @@@
  
        /* wait for it to finish, this takes about 23 us on lite5200b */
        while (!(in_be32(&fec->ievent) & FEC_IEVENT_MII) && --tries)
 -              udelay(5);
 +              msleep(1);
  
        if (!tries)
                return -ETIMEDOUT;
@@@ -66,6 -64,7 +66,6 @@@ static int mpc52xx_fec_mdio_probe(struc
  {
        struct device *dev = &of->dev;
        struct device_node *np = of->node;
 -      struct device_node *child = NULL;
        struct mii_bus *bus;
        struct mpc52xx_fec_mdio_priv *priv;
        struct resource res = {};
        bus->write = mpc52xx_fec_mdio_write;
  
        /* setup irqs */
 -      bus->irq = kmalloc(sizeof(bus->irq[0]) * PHY_MAX_ADDR, GFP_KERNEL);
 -      if (bus->irq == NULL) {
 -              err = -ENOMEM;
 -              goto out_free;
 -      }
 -      for (i=0; i<PHY_MAX_ADDR; i++)
 -              bus->irq[i] = PHY_POLL;
 -
 -      while ((child = of_get_next_child(np, child)) != NULL) {
 -              int irq = irq_of_parse_and_map(child, 0);
 -              if (irq != NO_IRQ) {
 -                      const u32 *id = of_get_property(child, "reg", NULL);
 -                      if (id)
 -                              bus->irq[*id] = irq;
 -              }
 -      }
 +      bus->irq = priv->mdio_irqs;
  
        /* setup registers */
        err = of_address_to_resource(np, 0, &res);
  
        /* set MII speed */
        out_be32(&priv->regs->mii_speed,
-               ((mpc52xx_find_ipb_freq(of->node) >> 20) / 5) << 1);
+               ((mpc5xxx_get_bus_frequency(of->node) >> 20) / 5) << 1);
  
 -      err = mdiobus_register(bus);
 +      err = of_mdiobus_register(bus, np);
        if (err)
                goto out_unmap;