brcm47xx: b44: bgmac: some small changes to the phylib patches
authorHauke Mehrtens <hauke@hauke-m.de>
Wed, 18 Sep 2013 16:49:52 +0000 (16:49 +0000)
committerHauke Mehrtens <hauke@hauke-m.de>
Wed, 18 Sep 2013 16:49:52 +0000 (16:49 +0000)
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
SVN-Revision: 38044

target/linux/brcm47xx/patches-3.10/770-bgmac-phylib.patch
target/linux/brcm47xx/patches-3.10/780-b44-phylib.patch

index 4a42aeb..13a7ea5 100644 (file)
@@ -35,8 +35,8 @@
 -      default:
 -              return -EOPNOTSUPP;
 -      }
-+      if (!netif_running(net_dev))
-+              return -EINVAL;
++      if (!netif_running(net_dev))
++              return -EINVAL;
 +
 +      if (!bgmac->phydev)
 +              return -EINVAL;
@@ -45,7 +45,7 @@
  }
  
  static const struct net_device_ops bgmac_netdev_ops = {
-@@ -1271,61 +1258,18 @@ static int bgmac_get_settings(struct net
+@@ -1271,61 +1258,16 @@ static int bgmac_get_settings(struct net
  {
        struct bgmac *bgmac = netdev_priv(net_dev);
  
@@ -97,7 +97,6 @@
  }
  
 -#if 0
-+
  static int bgmac_set_settings(struct net_device *net_dev,
                              struct ethtool_cmd *cmd)
  {
 +      return phy_ethtool_sset(bgmac->phydev, cmd);
  }
 -#endif
-+
  
  static void bgmac_get_drvinfo(struct net_device *net_dev,
                              struct ethtool_drvinfo *info)
-@@ -1336,6 +1280,7 @@ static void bgmac_get_drvinfo(struct net
+@@ -1336,6 +1278,7 @@ static void bgmac_get_drvinfo(struct net
  
  static const struct ethtool_ops bgmac_ethtool_ops = {
        .get_settings           = bgmac_get_settings,
        .get_drvinfo            = bgmac_get_drvinfo,
  };
  
-@@ -1354,10 +1299,42 @@ static int bgmac_mii_write(struct mii_bu
+@@ -1354,10 +1297,36 @@ static int bgmac_mii_write(struct mii_bu
        return bgmac_phy_write(bus->priv, mii_id, regnum, value);
  }
  
 +{
 +      struct bgmac *bgmac = netdev_priv(dev);
 +      struct phy_device *phydev = bgmac->phydev;
-+      int status_changed = 0;
++      bool status_changed = 0;
 +
 +      BUG_ON(!phydev);
 +
 +              bgmac->old_duplex = phydev->duplex;
 +      }
 +
-+      if (status_changed) {
-+              pr_info("%s: link %s", dev->name, phydev->link ?
-+                      "UP" : "DOWN");
-+              if (phydev->link)
-+                      pr_cont(" - %d/%s", phydev->speed,
-+                      phydev->duplex == DUPLEX_FULL ? "full" : "half");
-+              pr_cont("\n");
-+      }
++      if (status_changed)
++              phy_print_status(phydev);
 +}
 +
  static int bgmac_mii_register(struct bgmac *bgmac)
  
        mii_bus = mdiobus_alloc();
        if (!mii_bus)
-@@ -1388,7 +1365,29 @@ static int bgmac_mii_register(struct bgm
+@@ -1388,7 +1357,28 @@ static int bgmac_mii_register(struct bgm
  
        bgmac->mii_bus = mii_bus;
  
 +                           PHY_INTERFACE_MODE_MII);
 +
 +      if (IS_ERR(phydev)) {
-+              netdev_err(net_dev, "could not attach PHY: %s", phy_id);
++              netdev_err(net_dev, "could not attach PHY: %s\n", phy_id);
 +              bgmac->phyaddr = BGMAC_PHY_NOREGS;
 +              return PTR_ERR(phydev);
 +      }
 +      bgmac->old_duplex = -1;
 +      bgmac->phyaddr = phydev->addr;
 +
-+      netdev_info(net_dev, "attached PHY driver [%s] "
-+              "(mii_bus:phy_addr=%s)\n",
-+              phydev->drv->name, dev_name(&phydev->dev));
++      netdev_info(net_dev, "attached PHY driver [%s] (mii_bus:phy_addr=%s)\n",
++                  phydev->drv->name, dev_name(&phydev->dev));
 +
 +      return 0;
  
index 244978b..0295ca0 100644 (file)
  }
  
  static void b44_get_ringparam(struct net_device *dev,
-@@ -2102,20 +2027,81 @@ static const struct ethtool_ops b44_etht
+@@ -2102,20 +2027,74 @@ static const struct ethtool_ops b44_etht
  
  static int b44_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
  {
 +{
 +      struct b44 *bp = netdev_priv(dev);
 +      struct phy_device *phydev = bp->phydev;
-+      int status_changed = 0;
++      bool status_changed = 0;
 +
 +      BUG_ON(!phydev);
 +
 +              bp->old_duplex = phydev->duplex;
 +      }
 +
-+      if (status_changed) {
-+              pr_info("%s: link %s", dev->name, phydev->link ?
-+                      "UP" : "DOWN");
-+              if (phydev->link)
-+                      pr_cont(" - %d/%s", phydev->speed,
-+                      phydev->duplex == DUPLEX_FULL ? "full" : "half");
-+              pr_cont("\n");
-+      }
++      if (status_changed)
++              phy_print_status(phydev);
 +}
 +
 +static int b44_mii_probe(struct net_device *dev)
 +      phydev = phy_connect(dev, phy_id, &b44_adjust_link,
 +                           PHY_INTERFACE_MODE_MII);
 +      if (IS_ERR(phydev)) {
-+              netdev_err(dev, "could not attach PHY: %s", phy_id);
++              netdev_err(dev, "could not attach PHY: %s\n", phy_id);
 +              bp->phy_addr = B44_PHY_ADDR_NO_PHY;
 +              return PTR_ERR(phydev);
 +      }
 +      bp->old_duplex = -1;
 +      bp->phy_addr = phydev->addr;
 +
-+      netdev_info(dev, "attached PHY driver [%s] "
-+              "(mii_bus:phy_addr=%s)\n",
-+              phydev->drv->name, dev_name(&phydev->dev));
++      netdev_info(dev, "attached PHY driver [%s] (mii_bus:phy_addr=%s)\n",
++                  phydev->drv->name, dev_name(&phydev->dev));
 +
 +      return 0;
 +}
  static int b44_get_invariants(struct b44 *bp)
  {
        struct ssb_device *sdev = bp->sdev;
-@@ -2235,12 +2221,38 @@ static int b44_init_one(struct ssb_devic
+@@ -2235,12 +2214,40 @@ static int b44_init_one(struct ssb_devic
                goto err_out_powerdown;
        }
  
 +      bp->mii_bus->read = b44_mii_read;
 +      bp->mii_bus->write = b44_mii_write;
 +      bp->mii_bus->name = "b44_eth_mii";
++      bp->mii_bus->parent = sdev->dev;
++      bp->mii_bus->phy_mask = ~(1 << bp->phy_addr);
 +      snprintf(bp->mii_bus->id, MII_BUS_ID_SIZE, "%x", instance);
 +      bp->mii_bus->irq = kmalloc(sizeof(int) * PHY_MAX_ADDR, GFP_KERNEL);
 +      if (!bp->mii_bus->irq) {
  
        /* By default, advertise all speed/duplex settings. */
        bp->flags |= (B44_FLAG_ADV_10HALF | B44_FLAG_ADV_10FULL |
-@@ -2272,6 +2284,16 @@ static int b44_init_one(struct ssb_devic
+@@ -2272,6 +2279,16 @@ static int b44_init_one(struct ssb_devic
  
        return 0;
  
  err_out_powerdown:
        ssb_bus_may_powerdown(sdev->bus);
  
-@@ -2285,8 +2307,12 @@ out:
+@@ -2285,8 +2302,12 @@ out:
  static void b44_remove_one(struct ssb_device *sdev)
  {
        struct net_device *dev = ssb_get_drvdata(sdev);