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

feat: basic rtos task api #15

Closed
wants to merge 2 commits into from

Conversation

ForsakenHarmony
Copy link
Contributor

@ForsakenHarmony ForsakenHarmony commented Oct 26, 2021

"threads" without std

I'm not entirely sure if just stopping them is ok, the vTaskDelte docs mention having to free task memory prior to stopping the task

depends on #13

@ivmarkov
Copy link
Collaborator

See my comments in #20
If we abstract away the spawning of a thread with a trait, then we would not need this.

@ivmarkov
Copy link
Collaborator

Closing. Main argument is as follows: using no_std on top of ESP IDF remains to be a solution looking for a problem, really.
Yet - while we still do that for the moment, exposing another thread API (FreeRTOS) in a Rust-safe way (as opposed to just using the unsafe esp-idf-sys bindings), seems to be going too far. Why would we do that?

@ivmarkov ivmarkov closed this Mar 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants