zephyrproject-rtos / zephyr

Primary Git Repository for the Zephyr Project. Zephyr is a new generation, scalable, optimized, secure RTOS for multiple hardware architectures.
https://docs.zephyrproject.org
Apache License 2.0
10.9k stars 6.64k forks source link

samples/net/sockets/echo_server/ sample.net.sockets.echo_server.usbnet fails to build for stm32h747i_disco/stm32h747xx/m7 on main #72433

Closed aescolar closed 6 months ago

aescolar commented 6 months ago

Describe the bug samples/net/sockets/echo_server/ fails to build for stm32h747i_disco/stm32h747xx/m7

To Reproduce Steps to reproduce the behavior:

  1. twister -p stm32h747i_disco/stm32h747xx/m7 -T samples/net/sockets/echo_server/ -s sample.net.sockets.echo_server.usbnet
  2. See error

Expected behavior No build errors

Impact main CI failing blocking PRs which trigger this sample/boards combination

Logs and console output https://github.com/zephyrproject-rtos/zephyr/actions/runs/8984763903/job/24679138638?pr=68127

Environment (please complete the following information):

Additional context Introduced by https://github.com/zephyrproject-rtos/zephyr/pull/71012 Bisected to ac165f663e365e19eb6323219cf74f9d5259f805

aescolar commented 6 months ago

CC @spectrum70

aescolar commented 6 months ago

CC @erwango @FRASTM

foss0guru commented 6 months ago

Hi @aescolar the following patch solves the issue

--- a/boards/st/stm32h747i_disco/stm32h747i_disco_stm32h747xx_m7.dts
+++ b/boards/st/stm32h747i_disco/stm32h747i_disco_stm32h747xx_m7.dts
@@ -130,9 +130,7 @@
         */
        status = "okay";
        pinctrl-0 = <&eth_ref_clk_pa1
-                    &eth_mdio_pa2
                     &eth_crs_dv_pa7
-                    &eth_mdc_pc1
                     &eth_rxd0_pc4
                     &eth_rxd1_pc5
                     &eth_tx_en_pg11
@@ -141,6 +139,18 @@
        pinctrl-names = "default";
 };

+&mdio {
+       status = "okay";
+       pinctrl-0 = <&eth_mdio_pa2 &eth_mdc_pc1>;
+       pinctrl-names = "default";
+
+       ethernet-phy@0 {
+               compatible = "ethernet-phy";
+               reg = <0x00>;
+               status = "okay";
+       };
+};
+
 &rng {
        status = "okay";
 };