Re: [PATCH] ibm_newemac: Increase MDIO timeouts

2008-04-25 Thread Benjamin Herrenschmidt
On Fri, 2008-04-25 at 18:58 -0500, Josh Boyer wrote: > > > I guess them being "two short" is why they needed to be > doubled. :-) > > > > Well, in that case, it makes sense, it's still pretty short and > better > > safe than sorry. On day I may look at actually measuring PHYs and > see if > > it

Re: [PATCH] ibm_newemac: Increase MDIO timeouts

2008-04-25 Thread Josh Boyer
On Sat, 26 Apr 2008 08:22:38 +1000 Benjamin Herrenschmidt <[EMAIL PROTECTED]> wrote: > > On Fri, 2008-04-25 at 16:57 -0400, Bill Fink wrote: > > On Wed, 23 Apr 2008, Benjamin Herrenschmidt wrote: > > > > > This patch doubles the MDIO timeouts in EMAC as there are field > > > cases where they are

Re: [PATCH] ibm_newemac: Increase MDIO timeouts

2008-04-25 Thread Benjamin Herrenschmidt
On Fri, 2008-04-25 at 16:57 -0400, Bill Fink wrote: > On Wed, 23 Apr 2008, Benjamin Herrenschmidt wrote: > > > This patch doubles the MDIO timeouts in EMAC as there are field > > cases where they are two short to communicate with some PHYs. > > I guess them being "two short" is why they needed t

Re: [PATCH] ibm_newemac: Increase MDIO timeouts

2008-04-25 Thread Bill Fink
On Wed, 23 Apr 2008, Benjamin Herrenschmidt wrote: > This patch doubles the MDIO timeouts in EMAC as there are field > cases where they are two short to communicate with some PHYs. I guess them being "two short" is why they needed to be doubled. :-)

Re: [PATCH] ibm_newemac: Increase MDIO timeouts

2008-04-24 Thread Jeff Garzik
Benjamin Herrenschmidt wrote: This patch doubles the MDIO timeouts in EMAC as there are field cases where they are two short to communicate with some PHYs. Signed-off-by: Benjamin Herrenschmidt <[EMAIL PROTECTED]> applied ___ Linuxppc-dev mailing l

[PATCH] ibm_newemac: Increase MDIO timeouts

2008-04-22 Thread Benjamin Herrenschmidt
This patch doubles the MDIO timeouts in EMAC as there are field cases where they are two short to communicate with some PHYs. Signed-off-by: Benjamin Herrenschmidt <[EMAIL PROTECTED]> --- drivers/net/ibm_newemac/core.c |8 1 file changed, 4 insertions(+), 4 deletions(-) --- linux-w