-
Notifications
You must be signed in to change notification settings - Fork 26
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
[Firmware] Use async wifi #199
Conversation
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.
clippy found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
You have successfully added a new clippy configuration |
You have successfully added a new clippy configuration |
You have successfully added a new clippy configuration |
5bca120
to
be151a4
Compare
I have tested such that it works on wifi on esp32c3. @ImUrX any chance you can test esp32? I also noticed that latency went from 1ms to 7ms. That really sucks, but I don't really know what to do about it. Under the hood, both embassy-net and esp-wifi were using smoltcp, so idk what could have changed other than the async runtime itself or the way that tasks are interleaving. |
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.
seems fine aside from the clippy ignores
This simultaneously fixes the broken wifi code, and overhauls it to use
embassy-net
.This is a WIP.