Hi guys,

I am having one issue with RealTek RTL8211E u-boot driver.
I found you have added support for this phy in past.

The problem which I have is with rtl8211x_parse_status() where you are
working with page 11.
What was wrong on genphy_parse_link function?

The problem I do see is that SPDDONE bit is 1 but link bit is zero.
That's why phydev->link is setup to 0 which ending up in driver.

Zynq> dhcp
ethernet@e000b000 Waiting for PHY auto negotiation to complete....... done
ethernet@e000b000: No link.

There is sort of 1ms delay between spddone bit and link up.

No problem to add waiting loop in parse status to cover it or just move
to to genphy_parse_link.
Linux realtek driver is using genphy_read_status that's why I think this
is right way to go.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to