Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

W5500 build with RP2350 failed #109

Closed
Dexus opened this issue Jan 20, 2025 · 5 comments
Closed

W5500 build with RP2350 failed #109

Dexus opened this issue Jan 20, 2025 · 5 comments

Comments

@Dexus
Copy link

Dexus commented Jan 20, 2025

RP2040_PICO_CNC_mdns_webui.json

Command failed: make
In file included from /home/dev/RP2040/networking/networking.h:55,
from /home/dev/RP2040/networking/wiznet/enet.c:50:
/home/dev/RP2040/lwipopts.h:46: warning: "LWIP_CHKSUM_ALGORITHM" redefined
46 | #define LWIP_CHKSUM_ALGORITHM 3
|
In file included from /home/sdk/pico-sdk/lib/lwip/src/include/lwip/arch.h:48,
from /home/sdk/pico-sdk/lib/lwip/src/include/lwip/debug.h:40,
from /home/sdk/pico-sdk/lib/lwip/src/include/lwip/opt.h:52,
from /home/sdk/pico-sdk/lib/lwip/src/include/lwip/netif.h:40,
from /home/dev/RP2040/networking/wiznet/enet.c:31:
/home/dev/RP2040/networking/wiznet/arch/cc.h:96: note: this is the location of the previous definition
96 | #define LWIP_CHKSUM_ALGORITHM 0
|
/home/dev/RP2040/networking/wiznet/enet.c:176:38: warning: 'struct mdns_service' declared inside parameter list will not be visible outside of this definition or declaration
176 | static void mdns_device_info (struct mdns_service *service, void *txt_userdata)
| ^~~~~~~~~~~~
/home/dev/RP2040/networking/wiznet/enet.c: In function 'mdns_device_info':
/home/dev/RP2040/networking/wiznet/enet.c:181:5: warning: implicit declaration of function 'mdns_resp_add_service_txtitem' [-Wimplicit-function-declaration]
181 | mdns_resp_add_service_txtitem(service, "model=grblHAL", 13);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/dev/RP2040/networking/wiznet/enet.c: At top level:
/home/dev/RP2040/networking/wiznet/enet.c:186:39: warning: 'struct mdns_service' declared inside parameter list will not be visible outside of this definition or declaration
186 | static void mdns_service_info (struct mdns_service *service, void *txt_userdata)
| ^~~~~~~~~~~~
/home/dev/RP2040/networking/wiznet/enet.c: In function 'netif_status_callback':
/home/dev/RP2040/networking/wiznet/enet.c:238:9: warning: implicit declaration of function 'mdns_resp_init' [-Wimplicit-function-declaration]
238 | mdns_resp_init();
| ^~~~~~~~~~~~~~
/home/dev/RP2040/networking/wiznet/enet.c:240:29: warning: implicit declaration of function 'mdns_resp_add_netif' [-Wimplicit-function-declaration]
240 | if((services.mdns = mdns_resp_add_netif(netif_default, network.hostname, MDNS_TTL) == ERR_OK)) {
| ^~~~~~~~~~~~~~~~~~~
/home/dev/RP2040/networking/wiznet/enet.c:242:13: warning: implicit declaration of function 'mdns_resp_add_service' [-Wimplicit-function-declaration]
242 | mdns_resp_add_service(netif_default, network.hostname, "_device-info", DNSSD_PROTO_TCP, 0, MDNS_TTL, mdns_device_info, "version=" GRBL_VERSION);
| ^~~~~~~~~~~~~~~~~~~~~
/home/dev/RP2040/networking/wiznet/enet.c:242:84: error: 'DNSSD_PROTO_TCP' undeclared (first use in this function); did you mean 'IP_PROTO_TCP'?
242 | mdns_resp_add_service(netif_default, network.hostname, "_device-info", DNSSD_PROTO_TCP, 0, MDNS_TTL, mdns_device_info, "version=" GRBL_VERSION);
| ^~~~~~~~~~~~~~~
| IP_PROTO_TCP
/home/dev/RP2040/networking/wiznet/enet.c:242:84: note: each undeclared identifier is reported only once for each function it appears in
make[2]: *** [CMakeFiles/grblHAL.dir/build.make:482: CMakeFiles/grblHAL.dir/networking/wiznet/enet.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1873: CMakeFiles/grblHAL.dir/all] Error 2
make: *** [Makefile:91: all] Error 2
@Dexus
Copy link
Author

Dexus commented Jan 20, 2025

ok, its not the w5500 its mdns

@terjeio
Copy link
Contributor

terjeio commented Jan 20, 2025

Its due to a typo in the Web Builder backend plus an incompatible mdns lwip app in the SDK. I'll have to add a check for the lwip version used and change the grblHAL code accordingly.
Anyway the Web Builder at least compiles the code now, I have yet to test if it still works.

@terjeio
Copy link
Contributor

terjeio commented Jan 22, 2025

Updated code committed.

@terjeio terjeio closed this as completed Jan 22, 2025
@Dexus
Copy link
Author

Dexus commented Jan 22, 2025

@terjeio

for WIFI:

Command failed: make
In file included from /home/dev/RP2040/driver.c:115:
/home/dev/RP2040/wifi.h:28:10: fatal error: lwip/ip_addr.h: No such file or directory
28 | #include "lwip/ip_addr.h"
| ^~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/grblHAL.dir/build.make:90: CMakeFiles/grblHAL.dir/driver.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1954: CMakeFiles/grblHAL.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

while w5500 works

@terjeio
Copy link
Contributor

terjeio commented Jan 23, 2025

Another backend issue fixed - it was not set up for the new Pico2 W. Was ok for Pico W.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants