All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] RealTek RTL8211E u-boot driver
@ 2016-02-06 12:45 Michal Simek
  0 siblings, 0 replies; only message in thread
From: Michal Simek @ 2016-02-06 12:45 UTC (permalink / raw)
  To: u-boot

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: <http://lists.denx.de/pipermail/u-boot/attachments/20160206/6c4395cf/attachment.sig>

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2016-02-06 12:45 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-06 12:45 [U-Boot] RealTek RTL8211E u-boot driver Michal Simek

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.