From c734a6283c3f26d945df66ddaf9c327b32b08c65 Mon Sep 17 00:00:00 2001 From: Marc Rechte Date: Fri, 24 Feb 2017 08:00:11 +0100 Subject: [PATCH] kinetis_enet.c add #define for number of loops for auto negotiation to complete --- arch/arm/src/kinetis/kinetis_enet.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/arch/arm/src/kinetis/kinetis_enet.c b/arch/arm/src/kinetis/kinetis_enet.c index 554ccf734d..a640de5cc6 100644 --- a/arch/arm/src/kinetis/kinetis_enet.c +++ b/arch/arm/src/kinetis/kinetis_enet.c @@ -127,6 +127,7 @@ #define KINETIS_TXTIMEOUT (60*CLK_TCK) #define MII_MAXPOLLS (0x1ffff) #define LINK_WAITUS (500*1000) +#define LINK_NLOOPS (10) /* PHY definitions. * @@ -1763,7 +1764,7 @@ static inline int kinetis_initphy(struct kinetis_driver_s *priv) /* Wait for auto negotiation to complete */ - for (retries = 0; retries < 10; retries++) + for (retries = 0; retries < LINK_NLOOPS; retries++) { ret = kinetis_readmii(priv, phyaddr, MII_MSR, &phydata); if (ret < 0) @@ -1792,8 +1793,8 @@ static inline int kinetis_initphy(struct kinetis_driver_s *priv) PHY chip have mechanisms to configure link OK. We should leave autconf on, and find a way to re-configure MCU whenever the link is ready. */ - ninfo("%s: Autonegotiation failed (is cable plugged-in ?), default to 10Mbs mode\n", \ - BOARD_PHY_NAME); + ninfo("%s: Autonegotiation failed [%d] (is cable plugged-in ?), default to 10Mbs mode\n", \ + BOARD_PHY_NAME, retries); /* Stop auto negotiation */