SGMII configuration depends on proper GT setting that's why when node has
phys property call PSGTR driver to configure it properly.

Signed-off-by: Michal Simek <[email protected]>
---

Changes in v3:
- Separate phy init from phy power on (IP reset has to be between)

Changes in v2:
- Handle also cases where phy reference doesn't exist which means
  that u-boot doesn't need to configure it (configured via psu_init for
  example)
- Error out where reference exists but driver is not compiled in - reported
  by Sean

 drivers/net/zynq_gem.c | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/drivers/net/zynq_gem.c b/drivers/net/zynq_gem.c
index 5cbe8d28304b..b751d28e611f 100644
--- a/drivers/net/zynq_gem.c
+++ b/drivers/net/zynq_gem.c
@@ -12,6 +12,7 @@
 #include <common.h>
 #include <cpu_func.h>
 #include <dm.h>
+#include <generic-phy.h>
 #include <log.h>
 #include <net.h>
 #include <netdev.h>
@@ -715,6 +716,19 @@ static int zynq_gem_probe(struct udevice *dev)
        void *bd_space;
        struct zynq_gem_priv *priv = dev_get_priv(dev);
        int ret;
+       struct phy phy;
+
+       if (priv->interface == PHY_INTERFACE_MODE_SGMII) {
+               ret = generic_phy_get_by_index(dev, 0, &phy);
+               if (!ret) {
+                       ret = generic_phy_init(&phy);
+                       if (ret)
+                               return ret;
+               } else if (ret != -ENOENT) {
+                       debug("could not get phy (err %d)\n", ret);
+                       return ret;
+               }
+       }
 
        ret = zynq_gem_reset_init(dev);
        if (ret)
@@ -771,6 +785,12 @@ static int zynq_gem_probe(struct udevice *dev)
        if (ret)
                goto err3;
 
+       if (priv->interface == PHY_INTERFACE_MODE_SGMII && phy.dev) {
+               ret = generic_phy_power_on(&phy);
+               if (ret)
+                       return ret;
+       }
+
        return ret;
 
 err3:
-- 
2.34.1

Reply via email to