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

Support for zephyr #319

Open
GwnDaan opened this issue Sep 6, 2023 · 4 comments
Open

Support for zephyr #319

GwnDaan opened this issue Sep 6, 2023 · 4 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request investigating Looking into this issue / need more info

Comments

@GwnDaan
Copy link
Member

GwnDaan commented Sep 6, 2023

It would be nice to be able to integrate AgIsoStack++ in a zephyr based project. It probably can be done with a link to a git submodule in the cmake. But it may as well be more difficult, like for Arduino #264. Further research on this is needed

@GwnDaan GwnDaan added documentation Improvements or additions to documentation enhancement New feature or request investigating Looking into this issue / need more info labels Sep 6, 2023
@GwnDaan GwnDaan self-assigned this Sep 6, 2023
@tomstix
Copy link
Member

tomstix commented Nov 2, 2023

I was thinking about that as well a while ago. The main problem I saw back then was the lack of std::thread in zephyr. But afaik, support for that is on the way. Or we might just work around that using native zephyr threads / wrappers or something.
What are your findings so far @GwnDaan?

@ad3154
Copy link
Member

ad3154 commented Nov 2, 2023

We do now support single-threaded operation as well, which could be an option in the meantime if they are working on std::thread. It's what we do on Arduino at the moment

@GwnDaan
Copy link
Member Author

GwnDaan commented Nov 3, 2023

What are your findings so far @GwnDaan?

To be honest, I have not looked into this yet. I initially was planning on integrating the stack in an existing zephyr project, but never got to it. So if anyone wants to look into this before that, feel free to do so 😄

@GwnDaan GwnDaan removed their assignment Nov 3, 2023
@ad3154
Copy link
Member

ad3154 commented Mar 31, 2024

#458 Adds partial Zephyr support, see here

We'll just have to make some zephyr specific tweaks when we detect their OS specific macro in the cmsis_os header path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request investigating Looking into this issue / need more info
Projects
None yet
Development

No branches or pull requests

3 participants