stm32/eth.c: Remove redundant code.

robert-hh · robert-hh · commit 30fb6a733f7b · 2024-02-26T09:32:06.000+01:00
The MAC clock was initialized both in eth_init() and eth_mac_init().
The latter is not required.

Signed-off-by: robert-hh &lt;robert@hammelrath.com&gt;
diff --git a/ports/stm32/eth.c b/ports/stm32/eth.c
@@ -246,19 +246,12 @@ STATIC int eth_mac_init(eth_t *self) {
     #endif
     mpu_config_end(irq_state);
 
-    // Enable peripheral clock
+    // Set MAC to reset state
     #if defined(STM32H5)
-    __HAL_RCC_ETH_CLK_ENABLE();
-    __HAL_RCC_ETHTX_CLK_ENABLE();
-    __HAL_RCC_ETHRX_CLK_ENABLE();
     __HAL_RCC_ETH_FORCE_RESET();
     #elif defined(STM32H7)
-    __HAL_RCC_ETH1MAC_CLK_ENABLE();
-    __HAL_RCC_ETH1TX_CLK_ENABLE();
-    __HAL_RCC_ETH1RX_CLK_ENABLE();
     __HAL_RCC_ETH1MAC_FORCE_RESET();
     #else
-    __HAL_RCC_ETH_CLK_ENABLE();
     __HAL_RCC_ETHMAC_FORCE_RESET();
     #endif
 
