On Mon, Mar 25, 2024 at 02:16:53PM +0200, Roger Quadros wrote: > Hi Tom / Joe, > > On 05/03/2024 15:24, Roger Quadros wrote: > > Currently, the GPIO Reset Device Tree properties of the > > PHY node are ignored when the PHY is probed via mdio-uclass driver [1]. > > > > To resolve this, for each child of the MDIO bus node, bind and probe > > the generic Ethernet PHY driver if CONFIG_DM_ETH_PHY is enabled. > > > > This should now show the generic_phy_driver in "dm tree" > > and also apply the GPIO reset before the MDIO bus driver scans the bus > > for the PHYs. > > > > ethernet 0 [ + ] am65_cpsw_nuss_port | |-- > > ethernet@8000000port@1 > > bootdev 3 [ ] eth_bootdev | | `-- > > port@1.bootdev > > mdio 0 [ + ] cpsw_mdio | `-- mdio@f00 > > eth_phy_ge 0 [ + ] eth_phy_generic_drv | |-- > > ethernet-phy@0 > > eth_phy_ge 1 [ + ] eth_phy_generic_drv | `-- > > ethernet-phy@1 > > > > To test this on Beagleplay, the following series is required > > > > [1] Switch am65-cpsw to DM MDIO > > > > https://lore.kernel.org/all/20240228-for-2024-07-am65-cpsw-mdio-v2-0-f74f972ea...@kernel.org/ > > > > The last 3 patches are marked [not-for-merge]. They are to show how the > > whole solution can work to fix Ethernet on Beagleplay, which has been broken > > so far. Those DT patches will be sent once the device tree changes are > > merged > > into Linux tree. > > > > Signed-off-by: Roger Quadros <rog...@kernel.org> > > --- > > Roger Quadros (5): > > net: mdio-uclass: Bind and probe generic Ethernet PHY driver > > configs/am62x_beagleplay_a53_defconfig: enable DM_ETH_PHY > > Any comments to the first 2 patches in this series?
Oh, so the CI failure, unfortunately, was because sometimes those tests hit some resource-related race condition I believe. And I took your comment at the time to mean you would v2 them. Yes, these patches seem fine as is and I'll pick up assorted changes for -next soon. Thanks for your patience. -- Tom
signature.asc
Description: PGP signature
