net: ethernet: bgmac: device tree phy enablement
authorJon Mason <jon.mason@broadcom.com>
Fri, 4 Nov 2016 05:11:00 +0000 (01:11 -0400)
committerDavid S. Miller <davem@davemloft.net>
Mon, 7 Nov 2016 18:11:22 +0000 (13:11 -0500)
Change the bgmac driver to allow for phy's defined by the device tree

Signed-off-by: Jon Mason <jon.mason@broadcom.com>
Acked-by: Rafał Miłecki <rafal@milecki.pl>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bgmac-bcma.c
drivers/net/ethernet/broadcom/bgmac-platform.c
drivers/net/ethernet/broadcom/bgmac.c
drivers/net/ethernet/broadcom/bgmac.h

index c16ec3a51876aacee21ad81dc714a343afb75ba7..4a4ffc0c4c65d6d1777d8a24ffc5b0e0cdc63aeb 100644 (file)
@@ -80,6 +80,24 @@ static void bcma_bgmac_cmn_maskset32(struct bgmac *bgmac, u16 offset, u32 mask,
        bcma_maskset32(bgmac->bcma.cmn, offset, mask, set);
 }
 
+static int bcma_phy_connect(struct bgmac *bgmac)
+{
+       struct phy_device *phy_dev;
+       char bus_id[MII_BUS_ID_SIZE + 3];
+
+       /* Connect to the PHY */
+       snprintf(bus_id, sizeof(bus_id), PHY_ID_FMT, bgmac->mii_bus->id,
+                bgmac->phyaddr);
+       phy_dev = phy_connect(bgmac->net_dev, bus_id, bgmac_adjust_link,
+                             PHY_INTERFACE_MODE_MII);
+       if (IS_ERR(phy_dev)) {
+               dev_err(bgmac->dev, "PHY connection failed\n");
+               return PTR_ERR(phy_dev);
+       }
+
+       return 0;
+}
+
 static const struct bcma_device_id bgmac_bcma_tbl[] = {
        BCMA_CORE(BCMA_MANUF_BCM, BCMA_CORE_4706_MAC_GBIT,
                  BCMA_ANY_REV, BCMA_ANY_CLASS),
@@ -275,6 +293,10 @@ static int bgmac_probe(struct bcma_device *core)
        bgmac->cco_ctl_maskset = bcma_bgmac_cco_ctl_maskset;
        bgmac->get_bus_clock = bcma_bgmac_get_bus_clock;
        bgmac->cmn_maskset32 = bcma_bgmac_cmn_maskset32;
+       if (bgmac->mii_bus)
+               bgmac->phy_connect = bcma_phy_connect;
+       else
+               bgmac->phy_connect = bgmac_phy_connect_direct;
 
        err = bgmac_enet_probe(bgmac);
        if (err)
index be52f270c2c1448f058a4399af44c0d4fb21b722..46429402dfc1548ceb51cf4b658339167e4be3f8 100644 (file)
@@ -16,6 +16,7 @@
 #include <linux/bcma/bcma.h>
 #include <linux/etherdevice.h>
 #include <linux/of_address.h>
+#include <linux/of_mdio.h>
 #include <linux/of_net.h>
 #include "bgmac.h"
 
@@ -86,6 +87,20 @@ static void platform_bgmac_cmn_maskset32(struct bgmac *bgmac, u16 offset,
        WARN_ON(1);
 }
 
+static int platform_phy_connect(struct bgmac *bgmac)
+{
+       struct phy_device *phy_dev;
+
+       phy_dev = of_phy_get_and_connect(bgmac->net_dev, bgmac->dev->of_node,
+                                        bgmac_adjust_link);
+       if (!phy_dev) {
+               dev_err(bgmac->dev, "PHY connection failed\n");
+               return -ENODEV;
+       }
+
+       return 0;
+}
+
 static int bgmac_probe(struct platform_device *pdev)
 {
        struct device_node *np = pdev->dev.of_node;
@@ -102,7 +117,6 @@ static int bgmac_probe(struct platform_device *pdev)
        /* Set the features of the 4707 family */
        bgmac->feature_flags |= BGMAC_FEAT_CLKCTLST;
        bgmac->feature_flags |= BGMAC_FEAT_NO_RESET;
-       bgmac->feature_flags |= BGMAC_FEAT_FORCE_SPEED_2500;
        bgmac->feature_flags |= BGMAC_FEAT_CMDCFG_SR_REV4;
        bgmac->feature_flags |= BGMAC_FEAT_TX_MASK_SETUP;
        bgmac->feature_flags |= BGMAC_FEAT_RX_MASK_SETUP;
@@ -151,6 +165,12 @@ static int bgmac_probe(struct platform_device *pdev)
        bgmac->cco_ctl_maskset = platform_bgmac_cco_ctl_maskset;
        bgmac->get_bus_clock = platform_bgmac_get_bus_clock;
        bgmac->cmn_maskset32 = platform_bgmac_cmn_maskset32;
+       if (of_parse_phandle(np, "phy-handle", 0)) {
+               bgmac->phy_connect = platform_phy_connect;
+       } else {
+               bgmac->phy_connect = bgmac_phy_connect_direct;
+               bgmac->feature_flags |= BGMAC_FEAT_FORCE_SPEED_2500;
+       }
 
        return bgmac_enet_probe(bgmac);
 }
index 31ca204b38d2040d677f09b97d5a8c5f284141d2..7f66ea7638229d9396959a258d803cbef8bf4bb9 100644 (file)
@@ -1388,7 +1388,7 @@ static const struct ethtool_ops bgmac_ethtool_ops = {
  * MII
  **************************************************/
 
-static void bgmac_adjust_link(struct net_device *net_dev)
+void bgmac_adjust_link(struct net_device *net_dev)
 {
        struct bgmac *bgmac = netdev_priv(net_dev);
        struct phy_device *phy_dev = net_dev->phydev;
@@ -1411,8 +1411,9 @@ static void bgmac_adjust_link(struct net_device *net_dev)
                phy_print_status(phy_dev);
        }
 }
+EXPORT_SYMBOL_GPL(bgmac_adjust_link);
 
-static int bgmac_phy_connect_direct(struct bgmac *bgmac)
+int bgmac_phy_connect_direct(struct bgmac *bgmac)
 {
        struct fixed_phy_status fphy_status = {
                .link = 1,
@@ -1437,24 +1438,7 @@ static int bgmac_phy_connect_direct(struct bgmac *bgmac)
 
        return err;
 }
-
-static int bgmac_phy_connect(struct bgmac *bgmac)
-{
-       struct phy_device *phy_dev;
-       char bus_id[MII_BUS_ID_SIZE + 3];
-
-       /* Connect to the PHY */
-       snprintf(bus_id, sizeof(bus_id), PHY_ID_FMT, bgmac->mii_bus->id,
-                bgmac->phyaddr);
-       phy_dev = phy_connect(bgmac->net_dev, bus_id, &bgmac_adjust_link,
-                             PHY_INTERFACE_MODE_MII);
-       if (IS_ERR(phy_dev)) {
-               dev_err(bgmac->dev, "PHY connection failed\n");
-               return PTR_ERR(phy_dev);
-       }
-
-       return 0;
-}
+EXPORT_SYMBOL_GPL(bgmac_phy_connect_direct);
 
 int bgmac_enet_probe(struct bgmac *info)
 {
@@ -1507,10 +1491,7 @@ int bgmac_enet_probe(struct bgmac *info)
 
        netif_napi_add(net_dev, &bgmac->napi, bgmac_poll, BGMAC_WEIGHT);
 
-       if (!bgmac->mii_bus)
-               err = bgmac_phy_connect_direct(bgmac);
-       else
-               err = bgmac_phy_connect(bgmac);
+       err = bgmac_phy_connect(bgmac);
        if (err) {
                dev_err(bgmac->dev, "Cannot connect to phy\n");
                goto err_dma_free;
index 80836b4c9f3814cf115f85a8eb093e54538acd99..bfd9f40d71f7f55fe597aacf8846cdcee4668890 100644 (file)
@@ -513,10 +513,13 @@ struct bgmac {
        u32 (*get_bus_clock)(struct bgmac *bgmac);
        void (*cmn_maskset32)(struct bgmac *bgmac, u16 offset, u32 mask,
                              u32 set);
+       int (*phy_connect)(struct bgmac *bgmac);
 };
 
 int bgmac_enet_probe(struct bgmac *info);
 void bgmac_enet_remove(struct bgmac *bgmac);
+void bgmac_adjust_link(struct net_device *net_dev);
+int bgmac_phy_connect_direct(struct bgmac *bgmac);
 
 struct mii_bus *bcma_mdio_mii_register(struct bcma_device *core, u8 phyaddr);
 void bcma_mdio_mii_unregister(struct mii_bus *mii_bus);
@@ -583,4 +586,9 @@ static inline void bgmac_set(struct bgmac *bgmac, u16 offset, u32 set)
 {
        bgmac_maskset(bgmac, offset, ~0, set);
 }
+
+static inline int bgmac_phy_connect(struct bgmac *bgmac)
+{
+       return bgmac->phy_connect(bgmac);
+}
 #endif /* _BGMAC_H */