search

eclipse-threadx / netxduo

Eclipse ThreadX - NetXDuo is an advanced, industrial-grade TCP/IP network stack designed specifically for deeply embedded real-time and IoT applications
https://github.com/eclipse-threadx/rtos-docs/blob/main/rtos-docs/netx-duo/index.md
MIT License
230 stars 131 forks source link

ULONG64 definition failing for rx architecture #253

Open ryanwinter opened 4 months ago

ryanwinter commented 4 months ago

Describe the bug

The RX65N architecture does not have the ULONG64 defined in its tx_port.h which causes compilation to fail on this line: https://github.com/eclipse-threadx/netxduo/blob/master/addons/ptp/nxd_ptp_client.c#L1191

To Reproduce

Build the RX65N getting started guide.

/tmp/tmp-2136C7K0DphsKYJY/gcc-8.3.0.202004/bin/rx-elf-gcc -DNX_INCLUDE_USER_DEFINE_FILE -DTX_INCLUDE_USER_DEFINE_FILE -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/ports/rxv2/gnu/inc -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/common/inc -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/auto_ip -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/azure_iot -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/BSD -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/cloud -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/dhcp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/dns -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/ftp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/http -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/mdns -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/mqtt -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/nat -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/pop3 -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/ppp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/pppoe -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/ptp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/smtp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/snmp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/sntp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/telnet -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/tftp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/web -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/websocket -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/rtp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/rtsp -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/crypto_libraries/inc -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/nx_secure/inc -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/nx_secure/ports -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/tsn/inc -I/home/runner/work/getting-started/getting-started/Renesas/RSK_RX65N_2MB/build/lib/netxduo/custom_inc -I/home/runner/work/getting-started/getting-started/shared/lib/threadx/ports/rxv2/gnu/inc -I/home/runner/work/getting-started/getting-started/Renesas/RSK_RX65N_2MB/build/lib/threadx/custom_inc -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/azure_iot/azure-sdk-for-c/sdk/inc -I/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/azure_iot/azure-sdk-for-c/sdk/tests/core/inc -isystem /home/runner/work/getting-started/getting-started/shared/lib/threadx/common/inc -m64bit-doubles -mcpu=rx64m -misa=v2 -mlittle-endian-data  -g3 -ffunction-sections -fdata-sections -fdiagnostics-parseable-fixits -fno-strict-aliasing -fno-builtin -fno-common -Wuninitialized -Wdouble-promotion -Werror -Wno-unused-function -Wno-unused-parameter -Wno-incompatible-pointer-types  -O0 -std=gnu99 -MD -MT lib/netxduo/CMakeFiles/netxduo.dir/addons/ptp/nxd_ptp_client.c.obj -MF lib/netxduo/CMakeFiles/netxduo.dir/addons/ptp/nxd_ptp_client.c.obj.d -o lib/netxduo/CMakeFiles/netxduo.dir/addons/ptp/nxd_ptp_client.c.obj -c /home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/ptp/nxd_ptp_client.c
/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/ptp/nxd_ptp_client.c: In function ‘_nx_ptp_client_sync_received’:
/home/runner/work/getting-started/getting-started/shared/lib/netxduo/addons/ptp/nxd_ptp_client.c:1191:1: error: unknown type name ‘ULONG64’; did you mean ‘ULONG’?
 ULONG64     correctionNS = (hdr -> cFieldHigh << 16) | (hdr -> cFieldLow >> 16);
Vinzenz82 commented 1 month ago

Same error for cortex_m0. With adding a typedef for ULONG64 as unsigned long long it compiles without error.