* 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).