From: Suman Anna <s-a...@ti.com>

Enhance the AM65 CPSW NUSS driver to perform a MDIO reset using a GPIO
line. Logic is also added to perform a pre and post delay around reset
using the optional 'reset-delay-us' and 'reset-post-delay-us' properties.
This is similar to the reset being performed in the Linux kernel. The
reset is done once when the CPSW MDIO bus is being initialized.

Signed-off-by: Suman Anna <s-a...@ti.com>
Signed-off-by: Siddharth Vadapalli <s-vadapa...@ti.com>
---
 drivers/net/ti/am65-cpsw-nuss.c | 38 ++++++++++++++++++++++++++++++++-
 1 file changed, 37 insertions(+), 1 deletion(-)

diff --git a/drivers/net/ti/am65-cpsw-nuss.c b/drivers/net/ti/am65-cpsw-nuss.c
index 456c3eb5b2..f4e5809380 100644
--- a/drivers/net/ti/am65-cpsw-nuss.c
+++ b/drivers/net/ti/am65-cpsw-nuss.c
@@ -9,6 +9,7 @@
 #include <common.h>
 #include <malloc.h>
 #include <asm/cache.h>
+#include <asm/gpio.h>
 #include <asm/io.h>
 #include <asm/processor.h>
 #include <clk.h>
@@ -26,6 +27,7 @@
 #include <soc.h>
 #include <syscon.h>
 #include <linux/bitops.h>
+#include <linux/delay.h>
 #include <linux/soc/ti/ti-udma.h>
 
 #include "cpsw_mdio.h"
@@ -96,6 +98,8 @@
 
 #define AM65_CPSW_CPPI_PKT_TYPE                        0x7
 
+#define DEFAULT_GPIO_RESET_DELAY               10
+
 struct am65_cpsw_port {
        fdt_addr_t      port_base;
        fdt_addr_t      port_sgmii_base;
@@ -120,6 +124,10 @@ struct am65_cpsw_common {
        struct mii_dev          *bus;
        u32                     bus_freq;
 
+       struct gpio_desc        mdio_gpio_reset;
+       u32                     reset_delay_us;
+       u32                     reset_post_delay_us;
+
        struct dma              dma_tx;
        struct dma              dma_rx;
        u32                     rx_next;
@@ -679,6 +687,16 @@ static int am65_cpsw_mdio_init(struct udevice *dev)
        if (!priv->has_phy || cpsw_common->bus)
                return 0;
 
+       if (IS_ENABLED(CONFIG_DM_GPIO)) {
+               if (dm_gpio_is_valid(&cpsw_common->mdio_gpio_reset)) {
+                       dm_gpio_set_value(&cpsw_common->mdio_gpio_reset, 1);
+                       udelay(cpsw_common->reset_delay_us);
+                       dm_gpio_set_value(&cpsw_common->mdio_gpio_reset, 0);
+                       if (cpsw_common->reset_post_delay_us > 0)
+                               udelay(cpsw_common->reset_post_delay_us);
+               }
+       }
+
        ret = am65_cpsw_mdio_setup(dev);
        if (ret)
                return ret;
@@ -818,7 +836,7 @@ out:
 static int am65_cpsw_probe_nuss(struct udevice *dev)
 {
        struct am65_cpsw_common *cpsw_common = dev_get_priv(dev);
-       ofnode ports_np, node;
+       ofnode ports_np, node, mdio_np;
        int ret, i;
        struct udevice *port_dev;
 
@@ -845,6 +863,24 @@ static int am65_cpsw_probe_nuss(struct udevice *dev)
                                AM65_CPSW_CPSW_NU_ALE_BASE;
        cpsw_common->mdio_base = cpsw_common->ss_base + AM65_CPSW_MDIO_BASE;
 
+       if (IS_ENABLED(CONFIG_DM_GPIO)) {
+               /* get bus level PHY reset GPIO details */
+               mdio_np = dev_read_subnode(dev, "mdio");
+               if (!ofnode_valid(mdio_np)) {
+                       ret = -ENOENT;
+                       goto out;
+               }
+
+               cpsw_common->reset_delay_us = ofnode_read_u32_default(mdio_np, 
"reset-delay-us",
+                                                                     
DEFAULT_GPIO_RESET_DELAY);
+               cpsw_common->reset_post_delay_us = 
ofnode_read_u32_default(mdio_np,
+                                                                          
"reset-post-delay-us",
+                                                                          0);
+               ret = gpio_request_by_name_nodev(mdio_np, "reset-gpios", 0,
+                                                &cpsw_common->mdio_gpio_reset,
+                                                GPIOD_IS_OUT | 
GPIOD_IS_OUT_ACTIVE);
+       }
+
        ports_np = dev_read_subnode(dev, "ethernet-ports");
        if (!ofnode_valid(ports_np)) {
                ret = -ENOENT;
-- 
2.34.1

Reply via email to