Compiling error #include <api/IPAddress.h>

[bnarit]

I have to remove api/ from the #include directives in the library in order to compile it:

Arduino_10BASE_T1S/src/Arduino_10BASE_T1S_PHY_Interface.h

Line 17 in 759cc47

#include <api/IPAddress.h>

Arduino_10BASE_T1S/src/Arduino_10BASE_T1S_UDP.h

Lines 22 to 23 in 759cc47

	#include <api/Udp.h>
	#include <api/IPAddress.h>

//#include <api/Udp.h>
//#include <api/IPAddress.h>
#include <Udp.h>
#include <IPAddress.h>

[facchinm]

Hi @bnarit ,
which board are you using? The api include path is a best practice when the core depends from https://github.com/arduino/ArduinoCore-API , but we could relax this for this library in particular (since it's meant to target multiple architectures)

[bnarit]

I have fixed the compiler error, but haven't tested the system.

My board is implemented on RP2350.

The most of the problem are the conflicts on the T1S lib and lwip itself.

What I found are...

1. t1s_lwip_itoa or lwip_itoa didn't implement, so i wrapped it from

   char *t1s_lwip_itoa(char *buf, size_t bufsize, int num) {
     snprintf(buf, bufsize, "%d", num);  // base 10 only
     return buf;
   }

2. I also need to define away lwip_htonl and lwip_htons to prevent multiple definitions on linking stage.

   #ifndef lwip_htonl
   #define lwip_htonl(x) PP_HTONL(x)
   #endif
   
   #ifndef lwip_htons
   #define lwip_htons(x) PP_HTONS(x)
   #endif

3. the remove api/ from the #include directives as mentioned above.

Overall, I can successfully compile, but I'm not sure if this is the correct way.

[bnarit]

Now, I'm starting realized that this lib is based on ArduinoCore-API, which from my search isn't support RP2350, yet.
And this is all work around which isn't guarantee anything.

Another option is moving to Arduino Zephyr, but since this project require to maintain bare metal implementation, it will be on consideration.

[bnarit]

Now I can compile, get it attach to HWSPI of RP2350A. it seems to straight forward.

The HWSPI is working fine, interrupt attached. but still not functional. I will move to another thread.

[facchinm]

I reopened the issue to properly tackle the problem for all users in mainline library 🙂

[bnarit]

Thank you, now I can compile and run the lib.

The issue is on the stack size, which is quite easy to fix.

Just wrap them in another thread, since the main loop() thread is limit, by creating new thread work.

However, it receives some package and I am working on the package interpreter.

[bnarit]

Ok, my bug is there are two LWIP.
In earlephilhower/arduino-pico and in this Arduino_10BASE_T1S.

So, I remove prebuild lib ... libipv4.a in:

.platformio\packages\framework-arduinopico\lib\rp2350

then run the following command to build an empty lib:

"\.platformio\packages\toolchain-rp2040-earlephilhower\bin\arm-none-eabi-ar.exe" --% rcs libipv4.a

and then reconfig cfg/lwipopts.h by:

• removing all Rename APIs.
• add non existing flags and debug

#define LWIP_DBG_TYPES_ON (LWIP_DBG_TRACE) 
extern  void lwip_diag_serial(const char *fmt, ...) ;
#define LWIP_PLATFORM_DIAG(x) lwip_diag_serial x
#ifndef lwip_htonl
#define lwip_htonl(x) (((x) & 0xff) << 24 | \
                       ((x) & 0xff00) << 8 | \
                       ((x) & 0xff0000) >> 8 | \
                       ((x) & 0xff000000) >> 24)
#endif

#ifndef lwip_htons
#define lwip_htons(x) (((x) << 8) | ((x) >> 8))
#endif

• now we can use lwip_itoa, no need to re-implement it.

• and i put

extern "C" void lwip_diag_serial(const char *fmt, ...) {
  char buf[128];
  va_list args;
  va_start(args, fmt);
  vsnprintf(buf, sizeof(buf), fmt, args);
  va_end(args);
  Serial.print(buf);  // ✅ Use Serial here in C++ only
}

somewhere

[bnarit]

I have found another issue, regarding the difference between data packing in lwip on C and CPP.

For Arm compiler (rp2350), it seems to have a different data packing, which it should be guarded with 'extern "C"'

For ex. the netif has the pointer at 0x2000A988, on lwIpInit() (C++) same as ethernet_input().(C)
However, for netif.flahs the address are 0x2000a9db (C++) and 0x2000a9bb (C).

I not sure how to fix this paddind problem.

[bnarit]

OK, it because the conflict between Pico's lwip and the T1S.

Just rename old Pico's lwip lwipopts.h and create a dummy one.

and force build_flags in platformio.ini with

-include lib/Arduino_10BASE_T1S/src/lib/liblwip/cfg/lwipopts.h

also add some configs in lwipopt.h in t1s /cfg folder.

#define LWIP_NETIF_LINK_CALLBACK      0
#define LWIP_NETIF_HOSTNAME           1
#define LWIP_NUM_NETIF_CLIENT_DATA    5
#define LWIP_RAND() ((u32_t)rand()) 

you will be able to receive UDP package.

I am thinking that may be should we fork the T1S lib, which strip-off lwip?
Then we link T1S to arduino's lwip instead, so we can bridge and use the other libs with ease.

[leonardocavagnis]

I am thinking that may be should we fork the T1S lib, which strip-off lwip?
Then we link T1S to arduino's lwip instead, so we can bridge and use the other libs with ease.

Hey @bnarit,
Thanks a lot for the huge effort in porting this library to work with the RP2050 and the https://github.com/earlephilhower/arduino-pico core.
Please consider forking it and opening a PR to add preliminary RP2050 support. Also, feel free to evaluate removing the built-in lwip dependency.
Your contribution would be very valuable for everyone!