embeddedsw icon indicating copy to clipboard operation
embeddedsw copied to clipboard
verified publisher icon Xilinx

Added support for REALTEK chips.

Open HFTrader opened this issue 7 years ago • 2 comments

Realtek behaves like Marvell so if the chip is found to be a Realtek (0x001c) then we replace the id as Marvell. This has been tested on a Zybo board with a Realtek RTL8211E. Everything seems to work fine, including speed autonegotiation. The LWIP echo server passes without the need to set the speed fixed to 1000 mbps as suggested in the Digilente website.

HFTrader avatar Jan 04 '18 03:01 HFTrader

Hi,

Thanks for the patch. I think it will be better to check for (phy identifier == marvell) || (phy identifier == realtek) instead of replacing the identifier value. I will make these changes and merge it into our next public release.

Regards, Harini

harinikatakam avatar Jan 11 '18 04:01 harinikatakam

Hi,

I've modified this patch to add a check for realtek and duplicate realtek autoneg function (omitting Marvell specific register writes in a couple of places). I'm merging it on our next version. I noticed that your commit does not have a Signed-off by line. Could you please provide one?

Regards, Harini

harinikatakam avatar Jan 23 '18 05:01 harinikatakam