drivers: ethernet: eth_stm32_hal: fix API_V2 multicast

After API_V2 auto-negotiation support was added by zephyrproject-rtos#86621
setup_mac_filter() was called before HAL_ETH_Init(), which resulted in
received multicast packets being discarded.

This removes that call to setup_mac_filter() when API_V2 in enabled and
calls it from eth_init_api_v2() instead, after HAL_ETH_Init().

I've verified the problem and tested the fix on a Nucleo-H563ZI with a
simple application that just starts up and makes an mDNS query (which
depends on working multicast).

Signed-off-by: Kevin ORourke <[email protected]>

Author: kevinior

drivers/ethernet/eth_stm32_hal.c

@@ -967,10 +967,10 @@ static int eth_initialize(const struct device *dev)
 		&dma_tx_buffer[0][0], ETH_TXBUFNB);
 	HAL_ETH_DMARxDescListInit(heth, dma_rx_desc_tab,
 		&dma_rx_buffer[0][0], ETH_RXBUFNB);
-#endif /* !CONFIG_ETH_STM32_HAL_API_V1 */
 
 	setup_mac_filter(heth);
 
+#endif /* !CONFIG_ETH_STM32_HAL_API_V1 */
 
 	LOG_DBG("MAC %02x:%02x:%02x:%02x:%02x:%02x",
 		dev_data->mac_addr[0], dev_data->mac_addr[1],
@@ -1062,6 +1062,8 @@ static int eth_init_api_v2(const struct device *dev)
 		return -EINVAL;
 	}
 
+	setup_mac_filter(heth);
+
 #if defined(CONFIG_PTP_CLOCK_STM32_HAL)
 	/* Enable timestamping of RX packets. We enable all packets to be
 	 * timestamped to cover both IEEE 1588 and gPTP.