Re: [U-Boot] [PATCH 4/5] net: phy: Marvell 88E151x: Increase delay after init

2016-12-09 Thread Joe Hershberger 
On Fri, Dec 9, 2016 at 4:41 AM, Phil Edworthy  wrote:
> On Marvell 88E1512, the delay is not enough when connected
> to some external switches (e.g. Netgear GS108E).
>
> Signed-off-by: Phil Edworthy 

Acked-by: Joe Hershberger 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 4/5] net: phy: Marvell 88E151x: Increase delay after init

2016-12-09 Thread Stefan Roese 

On 09.12.2016 11:41, Phil Edworthy wrote:

On Marvell 88E1512, the delay is not enough when connected
to some external switches (e.g. Netgear GS108E).

Signed-off-by: Phil Edworthy 
---
 drivers/net/phy/marvell.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
index 8214760..48ebb50 100644
--- a/drivers/net/phy/marvell.c
+++ b/drivers/net/phy/marvell.c
@@ -320,7 +320,7 @@ static int m88e1518_config(struct phy_device *phydev)
/* Reset page selection */
phy_write(phydev, MDIO_DEVAD_NONE, 22, 0);

-   udelay(100);
+   udelay(500);
}

return m88es_config(phydev);


Reviewed-by: Stefan Roese 

Thanks,
Stefan
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 4/5] net: phy: Marvell 88E151x: Increase delay after init

2016-12-09 Thread Phil Edworthy 
On Marvell 88E1512, the delay is not enough when connected
to some external switches (e.g. Netgear GS108E).

Signed-off-by: Phil Edworthy 
---
 drivers/net/phy/marvell.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
index 8214760..48ebb50 100644
--- a/drivers/net/phy/marvell.c
+++ b/drivers/net/phy/marvell.c
@@ -320,7 +320,7 @@ static int m88e1518_config(struct phy_device *phydev)
/* Reset page selection */
phy_write(phydev, MDIO_DEVAD_NONE, 22, 0);
 
-   udelay(100);
+   udelay(500);
}
 
return m88es_config(phydev);
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot