I just noticed that Cogent CSB272 is now incorrectly reporting 1000Mbps speed after the following patch is applied to common/miiphyutil.c
* Patches by Travis Sawyer, 12 Mar 2004: - Fix Gigabit Ethernet support for 440GX - Add Gigabit Ethernet Support to MII PHY utilities
Cogent CSB272 board uses Intel LXT971A PHY on board and current code is being fooled by this patch.
According to datasheet of Intel LXT971A, PHY, PHY_1000BTSR register is listed among registers as 1000BASE-T/100BASE-T2 Status Register but it is marked as not implemented and no further data is available. So, the check against 0xFFFF is not working properly.
I have been monitoring the linuxppc embedded list and there appears to be incompatible register sets for 1000Mbps PHYs as well.
Perhaps we need to make the detection of 1000Mbps PHY included optionally based on a CONFIG_ macro and 1000MBps PHY boards define that variable. I have no use for extra code.
Best regards, Tolunay