-
Notifications
You must be signed in to change notification settings - Fork 179
Add multi-touch support for all touch drivers #691
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
Conversation
44372b4 to
38a38e5
Compare
38a38e5 to
129fd10
Compare
Test Results28 tests 28 ✅ 1m 31s ⏱️ Results for commit bb22e99. ♻️ This comment has been updated with latest results. |
b2602ec to
7a90e90
Compare
espzav
left a comment
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.
@PetrESP thank you for these changes, but I think we can make another PR for functions renaming and assert changes.
The main features are hiddent here.
We can keep it for now.
I left some comments - I will try it on HW soon.
7a90e90 to
3574be7
Compare
ee55c50 to
e7c98e5
Compare
e7c98e5 to
bb22e99
Compare
|
|
espzav
left a comment
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.
@PetrESP Thank you! LGTM
|
|
ESP-BSP Pull Request checklist
Change description
Drivers which support detection of multiple touch points have been extended.
Assertions in all touch drivers have been changed to ESP_RETURN_ON_FALSE
Function naming in touch drivers has been aligned