This is an automated email from the ASF dual-hosted git repository.

linguini1 pushed a commit to branch releases/13.1
in repository https://gitbox.apache.org/repos/asf/nuttx.git

commit c7cf132284e59c34ecf9d0c05ad06fa0ba02ff5b
Author: Daniel P. Carvalho <[email protected]>
AuthorDate: Mon Sep 21 19:38:27 2026 -0300

    arch/arm/stm32h5: fail the PHY initialization when the reset times out.
    
    The driver has the same code as the one of the STM32H7. When the PHY did
    not clear the reset bit in time, stm32_phyinit() returned the result of
    the last MDIO read. The bus reads all ones when the PHY does not answer
    yet, and that read succeeds, so the function returned OK and the driver
    went on with its default of 10 Mbps and half duplex, while the PHY could
    negotiate 100 Mbps and full duplex.
    
    Return -ETIMEDOUT, so that bringing the interface up fails and the
    failure is not hidden.
    
    It builds for nucleo-h563zi:netnsh, but it was not tested on hardware.
    
    Signed-off-by: Daniel P. Carvalho <[email protected]>
    Assisted-by: Claude:claude-sonnet-5
---
 arch/arm/src/stm32h5/stm32_ethernet.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/src/stm32h5/stm32_ethernet.c 
b/arch/arm/src/stm32h5/stm32_ethernet.c
index 473b27d418f..c0188f6bc6a 100644
--- a/arch/arm/src/stm32h5/stm32_ethernet.c
+++ b/arch/arm/src/stm32h5/stm32_ethernet.c
@@ -3369,7 +3369,7 @@ static int stm32_phyinit(struct stm32_ethmac_s *priv)
   if (to <= 0)
     {
       nerr("ERROR: Phy reset timeout\n");
-      return ret;
+      return -ETIMEDOUT;
     }
   else
     {

Reply via email to