diff mbox series

[V1] ARM: dts: am335x-bone-common: Increase MDIO reset deassert time

Message ID 20240531183817.2698445-1-colin.foster@in-advantage.com (mailing list archive)
State New
Headers show
Series [V1] ARM: dts: am335x-bone-common: Increase MDIO reset deassert time | expand

Commit Message

Colin Foster May 31, 2024, 6:38 p.m. UTC 
Prior to commit df16c1c51d81 ("net: phy: mdio_device: Reset device only
when necessary") MDIO reset deasserts were performed twice during boot.
Now that the second deassert is no longer performed, device probe
failures happen due to the change in timing with the following error
message:

SMSC LAN8710/LAN8720: probe of 4a101000.mdio:00 failed with error -5

Restore the original effective timing, which resolves the probe
failures.

Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
---
 arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Kevin Hilman July 9, 2024, 12:04 a.m. UTC | #1 
On Fri, 31 May 2024 13:38:17 -0500, Colin Foster wrote:
> Prior to commit df16c1c51d81 ("net: phy: mdio_device: Reset device only
> when necessary") MDIO reset deasserts were performed twice during boot.
> Now that the second deassert is no longer performed, device probe
> failures happen due to the change in timing with the following error
> message:
> 
> SMSC LAN8710/LAN8720: probe of 4a101000.mdio:00 failed with error -5
> 
> [...]

Applied, thanks!

[1/1] ARM: dts: am335x-bone-common: Increase MDIO reset deassert time
      commit: b9e33fd0c6f8a29643d3b6ec149d2a7693cbe902

Best regards,
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi b/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi
index f08f116bf75a..56418baa3e86 100644
--- a/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am335x-bone-common.dtsi
@@ -385,7 +385,7 @@  ethphy0: ethernet-phy@0 {
 		/* Support GPIO reset on revision C3 boards */
 		reset-gpios = <&gpio1 8 GPIO_ACTIVE_LOW>;
 		reset-assert-us = <300>;
-		reset-deassert-us = <6500>;
+		reset-deassert-us = <13000>;
 	};
 };