On Tue, Nov 28, 2023 at 3:21 PM Fabio Estevam festevam@gmail.com wrote:
Hi Tim and Maxim,
On Tue, Nov 28, 2023 at 8:05 PM Tim Harvey tharvey@gateworks.com wrote:
Hi Maxim,
I've tried this series on my imx8mm-venice boards and find that basic ping/dhcp/tftp breaks: u-boot=> net list eth0 : ethernet@30be0000 00:d0:12:78:f8:01 active u-boot=> setenv ipaddr 192.168.1.1 u-boot=> ping 192.168.1.146 eth0: ethernet@30be0000 00:d0:12:78:f8:01 active Using ethernet@30be0000 device pinging addr: 192.168.1.146 ping_tmo: ping failed; host 192.168.1.146 is not alive u-boot=> dhcp init already done for ethernet@30be0000 dhcp_tmo 20/20 dhcp_tmo 19/20 dhcp_tmo 18/20 dhcp_tmo 17/20 dhcp_tmo 16/20 dhcp_tmo 15/20 dhcp_tmo 14/20 dhcp_tmo 13/20 dhcp_tmo 12/20 dhcp_tmo 11/20 dhcp_tmo 10/20 dhcp_tmo 9/20 dhcp_tmo 8/20 dhcp_tmo 7/20 dhcp_tmo 6/20 dhcp_tmo 5/20 dhcp_tmo 4/20 dhcp_tmo 3/20 dhcp_tmo 2/20 dhcp_tmo 1/20 dhcp_tmo 0/20 DHCP client timeout
I got the same result on a custom imx8mn-based board.
u-boot=> tftpboot $loadaddr venice/Image init already done for ethernet@30be0000 TFTP from server 192.168.1.146; our IP address is 192.168.1.1 Filename 'venice/Image'. Load address: 0x48200000 Loading: ^ cntl-c after a minute or so of no output
Is there some basic config here that I may have wrong?
Also, while trying to see if I can determine where things broke I noticed there is build breakage starting with 'net/lwip: integrate lwIP library' In file included from net/lwip/lwip-external/src/core/init.c:38: ./net/lwip/lwip-external/src/include/lwip/opt.h:51:10: fatal error: lwipopts.h: No such file or directory 51 | #include "lwipopts.h" | ^~~~~~~~~~~~ compilation terminated. make[2]: *** [scripts/Makefile.build:256: net/lwip/lwip-external/src/core/init.o ] Error 1 make[1]: *** [scripts/Makefile.build:397: net/lwip] Error 2 make: *** [Makefile:1858: net] Error 2
I haven't noticed this build error here though. I applied Maxim's series against master.
Fabio,
You won't see this build issue with the entire series applied... only with the patches up to and including 'net/lwip: integrate lwIP library'. My point was there are points in the series where compilation fails which is not ok.
Tim