-
Notifications
You must be signed in to change notification settings - Fork 97
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
esp32 support #68
esp32 support #68
Conversation
fbiego
commented
Nov 28, 2024
- add esp32 support (arduino framework)
- change main file to c++ (arduino compatibility)
- add support for esp32 (tested with wt32 480x320)
- add esp32 support (arduino framework) - change main file to c++ (arduino compatibility) - add support for esp32 (tested with wt32 480x320)
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.
Great, thank you! I've added a few formatting related comments.
#endif | ||
|
||
|
||
void hal_setup(void); |
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.
void hal_setup(void); | |
/** | |
* Add some description please | |
*/ | |
void hal_setup(void); |
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 meant add some description please as API comments 🙂
Co-authored-by: Gabor Kiss-Vamosi <kisvegabor@gmail.com>
Co-authored-by: Gabor Kiss-Vamosi <kisvegabor@gmail.com>
Co-authored-by: Gabor Kiss-Vamosi <kisvegabor@gmail.com>
Co-authored-by: Gabor Kiss-Vamosi <kisvegabor@gmail.com>
Co-authored-by: Gabor Kiss-Vamosi <kisvegabor@gmail.com>
add comments
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.
Thank you!
As we have discussed I've sent you an invite for write access. Feel free to merge this PR when it's accepted 🙂
🎉 |