linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* MPC5200 FEC Ethernet only halfduplex ?
@ 2006-04-03 10:22 Thomas Schnürer
  2006-04-03 14:44 ` Dale Farnsworth
  0 siblings, 1 reply; 2+ messages in thread
From: Thomas Schnürer @ 2006-04-03 10:22 UTC (permalink / raw)
  To: linuxppc-dev

[-- Attachment #1: Type: text/plain, Size: 803 bytes --]

Im working with Linux on our MPC5200 Board, with the latest Rev. B3 Processor.
I was updating our Linux BSP from BestComm 2.1 to 2.2 some time ago and used your patches which increased usability of the FEC very much.
 
There is a line in the driver code which I dont understand, it says:
+ fec_restart(dev, 0); /* always use half duplex mode only */
 
Why can only half duplex be used ? I made a test with the iperf tool and saw that with the driver in its current state I get 90 Mbit/s in one direction and only 10 kBit/ other way. When changing the fec_restart call to (dev, 1) it turns to 90 Mbit and 250 kBit/s.
Then I changed for couriosity the BestComm task prioritisation and suddenly I get 90 Mbit in both directions.
So whats the reason for this setting above ?
 
Thanks in advance!

[-- Attachment #2: Type: text/html, Size: 1414 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: MPC5200 FEC Ethernet only halfduplex ?
  2006-04-03 10:22 MPC5200 FEC Ethernet only halfduplex ? Thomas Schnürer
@ 2006-04-03 14:44 ` Dale Farnsworth
  0 siblings, 0 replies; 2+ messages in thread
From: Dale Farnsworth @ 2006-04-03 14:44 UTC (permalink / raw)
  To: TSchnuerer, linuxppc-dev

TSchnuerer@men.de wrote:
> Im working with Linux on our MPC5200 Board, with the latest Rev. B3 
> Processor.
> I was updating our Linux BSP from BestComm 2.1 to 2.2 some time ago and
> used your patches which increased usability of the FEC very much.
> 
> There is a line in the driver code which I dont understand, it says:
> + fec_restart(dev, 0); /* always use half duplex mode only */
> 
> Why can only half duplex be used ?

The reason is simple, but not satisfying.  The driver has no
mechanism to detect PHY status changes and thus no way to determine
when a connection changes from full to half duplex.  It needs a
poll function for PHY status change.  Doing only half-duplex
avoids this issue--at a big performance cost.

I think a poll function was added in the Denx version.

The MPC5200 FEC driver desperately needs a rewrite.

-Dale

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2006-04-03 14:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-04-03 10:22 MPC5200 FEC Ethernet only halfduplex ? Thomas Schnürer
2006-04-03 14:44 ` Dale Farnsworth

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).