From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Sat, 6 Feb 2016 13:45:32 +0100 Subject: [U-Boot] RealTek RTL8211E u-boot driver Message-ID: <56B5EAEC.6070504@monstr.eu> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 at e000b000 Waiting for PHY auto negotiation to complete....... done ethernet at 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 -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 181 bytes Desc: OpenPGP digital signature URL: