-
Notifications
You must be signed in to change notification settings - Fork 61
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
Add support for zephyr #24
Conversation
Signed-off-by: Esteve Fernandez <esteve@apache.org>
Signed-off-by: Esteve Fernandez <esteve@apache.org>
Signed-off-by: Esteve Fernandez <esteve@apache.org>
Signed-off-by: Esteve Fernandez <esteve@apache.org>
return r; | ||
} | ||
|
||
// NOTE(esteve): SO_KEEPALIVE and SO_LINGER not supported in Zephyr |
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.
I would simply remove the commented code and leave the note saying that those flags are not supported in Zephyr.
src/net/zephyr/network.c
Outdated
|
||
return 0; | ||
} | ||
|
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.
I would simply remove the commented code and leave the note saying that iovec is not supported in Zephyr.
@esteve I went through your PR and it seems fine to me. Thanks for adding the support for Zephyr into zenoh-pico! For what concerns the socket flags, they are not mandatory in zenoh-pico. It's only a matter of tuning that we found beneficial on UNIX systems. If those flags are not supported in Zephyr, let's simply ignore them. |
Signed-off-by: Esteve Fernandez <esteve@apache.org>
Signed-off-by: Esteve Fernandez <esteve@apache.org>
@Mallets this PR has many changes, let me know if you'd like me to break it out:
#if
s to just check if the macros had been defined.strdup
, since Zephyr does not provide it_times()
when callingclock()
, I replaced thesrand()
initialization to just usetime()