-
Notifications
You must be signed in to change notification settings - Fork 2k
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
tests: adds 6LoWPAN receiving test #5502
Conversation
Here is a handcrafted hex dump for Wireshark: |
Just thinking if you can use |
|
#5504 was merged, please rebase. |
537613f
to
65d4fb5
Compare
Rebased & squashed. |
char dst[8]; | ||
} netif_hdr = { | ||
.src = { 0x02, 0x00, 0x00, 0xFF, 0xFE, 0x00, 0x00, 0x02 }, | ||
.dst = { 0x02, 0x00, 0x00, 0xFF, 0xFE, 0x00, 0x00, 0x01 }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Building application "gnrc_sixlowpan" for "native" with MCU "native".
/home/martine/Repositories/RIOT-OS/RIOT/tests/gnrc_sixlowpan/main.c: In function ‘_send_packet’:
/home/martine/Repositories/RIOT-OS/RIOT/tests/gnrc_sixlowpan/main.c:61:36: error: overflow in implicit constant conversion [-Werror=overflow]
.src = { 0x02, 0x00, 0x00, 0xFF, 0xFE, 0x00, 0x00, 0x02 },
^
/home/martine/Repositories/RIOT-OS/RIOT/tests/gnrc_sixlowpan/main.c:61:42: error: overflow in implicit constant conversion [-Werror=overflow]
.src = { 0x02, 0x00, 0x00, 0xFF, 0xFE, 0x00, 0x00, 0x02 },
^
/home/martine/Repositories/RIOT-OS/RIOT/tests/gnrc_sixlowpan/main.c:62:36: error: overflow in implicit constant conversion [-Werror=overflow]
.dst = { 0x02, 0x00, 0x00, 0xFF, 0xFE, 0x00, 0x00, 0x01 },
^
/home/martine/Repositories/RIOT-OS/RIOT/tests/gnrc_sixlowpan/main.c:62:42: error: overflow in implicit constant conversion [-Werror=overflow]
.dst = { 0x02, 0x00, 0x00, 0xFF, 0xFE, 0x00, 0x00, 0x01 },
^
cc1: all warnings being treated as errors
/home/martine/Repositories/RIOT-OS/RIOT/Makefile.base:60: recipe for target '/home/martine/Repositories/RIOT-OS/RIOT/tests/gnrc_sixlowpan/bin/native/gnrc_sixlowpan/main.o' failed
make[1]: *** [/home/martine/Repositories/RIOT-OS/RIOT/tests/gnrc_sixlowpan/bin/native/gnrc_sixlowpan/main.o] Error 1
/home/martine/Repositories/RIOT-OS/RIOT/tests/gnrc_sixlowpan/../../Makefile.include:259: recipe for target 'all' failed
I think its recommendable to have these fields of type uint8_t
(see #5497)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amended.
65d4fb5
to
9e172f3
Compare
I'll give my ACK as soon as I have tested it locally :) |
gnrc_netreg_register(GNRC_NETTYPE_UDP, &dump_udp); | ||
gnrc_netreg_register(GNRC_NETTYPE_UDP, &dump_udp_61616); | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There be whitespace ;-)
Apart from the introduced whitespace-error: ACK and go, as soon as Murdock likes it. |
9e172f3
to
f7bd237
Compare
Fixed whitespace error. |
This test sends fragmented 6LoWPAN packets with header compression to itself and checks the received packet. I made this for #5033.