[Feature Request] STM32Hxx port: Duplicate MDIO Clock Config setup in HAL

[Dweb2]

On the current version: ETH_MAC_MDIO_ClkConfig()

FreeRTOS-Plus-TCP/source/portable/NetworkInterface/STM32Hxx/stm32hxx_hal_eth.c

Line 2741 in 013b591

static void ETH_MAC_MDIO_ClkConfig( ETH_HandleTypeDef * heth )

and HAL_ETH_SetMDIOClockRange()

FreeRTOS-Plus-TCP/source/portable/NetworkInterface/STM32Hxx/stm32hxx_hal_eth.c

Line 2180 in 013b591

void HAL_ETH_SetMDIOClockRange( ETH_HandleTypeDef * heth )

are effectively identical functions which set the management clock. They get called from different contexts.

Maybe consider using just one function.

[AniruddhaKanhere]

Hello @Dweb2, thank you for bringing this to our attention. We are looking into it and will get back to you soon

[shubnil]

Hello @Dweb2, We missed this and will be looking into this now.

[ActoryOu]

Hi @Dweb2,
Thanks for your contribution. We removed duplicate function HAL_ETH_SetMDIOClockRange by #711.
I'm going to close this issue.

Thanks.