17 Jun
2020
17 Jun
'20
10:36 p.m.
Hi Vladimir,
On Wed, Jun 17, 2020 at 4:25 PM Vladimir Oltean olteanv@gmail.com wrote:
Not Grygorii's fault, I'm sure. He just noticed that there wasn't an of node for the PHY, and created one.
Today I learned, with the phy_connect API, you're supposed to populate the phydev node _yourself_. Like zynq_gem does:
https://gitlab.denx.de/u-boot/u-boot/-/blob/master/drivers/net/zynq_gem.c#L3...
Too bad almost nobody else does......
With the new dm_eth_phy_connect API, that happens automatically:
https://gitlab.denx.de/u-boot/u-boot/-/blob/master/net/mdio-uclass.c#L172
So, yeah. Fix the Ethernet driver, or switch to DM_MDIO.
Thanks for your hint! I managed to fix the FEC driver and now it can correctly find the PHY node.
Cheers