* [U-Boot] [PATCH] net: fix m88e1111s PHY auto negotiation timeout
@ 2016-02-16 6:54 Thomas Chou
2016-02-18 7:47 ` Stefan Roese
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Chou @ 2016-02-16 6:54 UTC (permalink / raw)
To: u-boot
After commit a058052c358c
("net: phy: do not read configuration register on reset")
both 3c120 and 10m50 devboard which use Marvel m88e1111s PHY got this
error message,
Net: eth0: ethernet at 400
=> ping 192.168.1.5
ethernet at 400 Waiting for PHY auto negotiation to complete.... TIMEOUT !
Using ethernet at 400 device
host 192.168.1.5 is alive
This is because the auto negotiation control bit was cleared by the new
phy_reset(). But the m88e1111s_config() does the soft-reset already,
there is no need to reset twice. The extra phy_reset() should be
removed.
Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
---
drivers/net/phy/marvell.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
index eab1558..5badfac 100644
--- a/drivers/net/phy/marvell.c
+++ b/drivers/net/phy/marvell.c
@@ -271,8 +271,6 @@ static int m88e1111s_config(struct phy_device *phydev)
genphy_config_aneg(phydev);
- phy_reset(phydev);
-
return 0;
}
--
2.5.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] net: fix m88e1111s PHY auto negotiation timeout
2016-02-16 6:54 [U-Boot] [PATCH] net: fix m88e1111s PHY auto negotiation timeout Thomas Chou
@ 2016-02-18 7:47 ` Stefan Roese
2016-02-18 9:38 ` Huan Wang
2016-02-25 1:33 ` Thomas Chou
0 siblings, 2 replies; 4+ messages in thread
From: Stefan Roese @ 2016-02-18 7:47 UTC (permalink / raw)
To: u-boot
Hi Thomas,
On 16.02.2016 07:54, Thomas Chou wrote:
> After commit a058052c358c
> ("net: phy: do not read configuration register on reset")
>
> both 3c120 and 10m50 devboard which use Marvel m88e1111s PHY got this
> error message,
>
> Net: eth0: ethernet at 400
> => ping 192.168.1.5
> ethernet at 400 Waiting for PHY auto negotiation to complete.... TIMEOUT !
> Using ethernet at 400 device
> host 192.168.1.5 is alive
>
> This is because the auto negotiation control bit was cleared by the new
> phy_reset(). But the m88e1111s_config() does the soft-reset already,
> there is no need to reset twice. The extra phy_reset() should be
> removed.
>
> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
> ---
> drivers/net/phy/marvell.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
> index eab1558..5badfac 100644
> --- a/drivers/net/phy/marvell.c
> +++ b/drivers/net/phy/marvell.c
> @@ -271,8 +271,6 @@ static int m88e1111s_config(struct phy_device *phydev)
>
> genphy_config_aneg(phydev);
>
> - phy_reset(phydev);
> -
> return 0;
> }
>
>
I've also stumbled over this problem on some boards equipped with the
Marvell PHY. Could you please take a look at these patches instead:
http://patchwork.ozlabs.org/patch/581316/
http://patchwork.ozlabs.org/patch/581317/
Do they work for you?
Joe, could you please make sure to push these for this release? As its
a real bugfix.
Thanks,
Stefan
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] net: fix m88e1111s PHY auto negotiation timeout
2016-02-18 7:47 ` Stefan Roese
@ 2016-02-18 9:38 ` Huan Wang
2016-02-25 1:33 ` Thomas Chou
1 sibling, 0 replies; 4+ messages in thread
From: Huan Wang @ 2016-02-18 9:38 UTC (permalink / raw)
To: u-boot
> On 16.02.2016 07:54, Thomas Chou wrote:
> > After commit a058052c358c
> > ("net: phy: do not read configuration register on reset")
> >
> > both 3c120 and 10m50 devboard which use Marvel m88e1111s PHY got this
> > error message,
> >
> > Net: eth0: ethernet at 400
> > => ping 192.168.1.5
> > ethernet at 400 Waiting for PHY auto negotiation to complete.... TIMEOUT !
> > Using ethernet at 400 device
> > host 192.168.1.5 is alive
> >
> > This is because the auto negotiation control bit was cleared by the
> > new phy_reset(). But the m88e1111s_config() does the soft-reset
> > already, there is no need to reset twice. The extra phy_reset() should
> > be removed.
> >
> > Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
> > ---
> > drivers/net/phy/marvell.c | 2 --
> > 1 file changed, 2 deletions(-)
> >
> > diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
> > index eab1558..5badfac 100644
> > --- a/drivers/net/phy/marvell.c
> > +++ b/drivers/net/phy/marvell.c
> > @@ -271,8 +271,6 @@ static int m88e1111s_config(struct phy_device
> > *phydev)
> >
> > genphy_config_aneg(phydev);
> >
> > - phy_reset(phydev);
> > -
> > return 0;
> > }
> >
> >
>
> I've also stumbled over this problem on some boards equipped with the
> Marvell PHY. Could you please take a look at these patches instead:
>
> http://patchwork.ozlabs.org/patch/581316/
> http://patchwork.ozlabs.org/patch/581317/
>
> Do they work for you?
>
> Joe, could you please make sure to push these for this release? As its a
> real bugfix.
>
>
[Alison Wang]
After commit a058052c358c
("net: phy: do not read configuration register on reset")
I find auto negotiation timeout error also occurs on Atheros's PHY AR8033 too. :(
Best Regards,
Alison Wang
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] net: fix m88e1111s PHY auto negotiation timeout
2016-02-18 7:47 ` Stefan Roese
2016-02-18 9:38 ` Huan Wang
@ 2016-02-25 1:33 ` Thomas Chou
1 sibling, 0 replies; 4+ messages in thread
From: Thomas Chou @ 2016-02-25 1:33 UTC (permalink / raw)
To: u-boot
Hi Stefan,
On 2016?02?18? 15:47, Stefan Roese wrote:
> I've also stumbled over this problem on some boards equipped with the
> Marvell PHY. Could you please take a look at these patches instead:
>
> http://patchwork.ozlabs.org/patch/581316/
> http://patchwork.ozlabs.org/patch/581317/
>
> Do they work for you?
Yes, they work great. Thanks a lot. I should have found your patches
earlier. :)
Best regards,
Thomas
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-02-25 1:33 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-16 6:54 [U-Boot] [PATCH] net: fix m88e1111s PHY auto negotiation timeout Thomas Chou
2016-02-18 7:47 ` Stefan Roese
2016-02-18 9:38 ` Huan Wang
2016-02-25 1:33 ` Thomas Chou
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.