-
-
Notifications
You must be signed in to change notification settings - Fork 350
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(GT911): Add CLI command to reset touch panel config #2351
Conversation
@wimalopaan it seems the config can be perfectly reseted with this CLI command. On my TX16S, this leads to the following output:
On the next reboot, the config number read is |
Related to #2063
Note: config reset with CLI "reset_gt911" required.
70d73e5
to
d87305f
Compare
Thats great! I had no luck with that on my controller ... and I verified all communication with a LogicAnalyzer. But anyway, very cool to have the intercative setting command! |
This has been tested today on a X12S and allows for removing the software inversion. For those who installed @wimalopaan ’s builds, it might work already (when software inversion is gone), for the others, a config reset via CLI will be needed. |
I’ll try to have something more “automatic” if I get to it, but that should already release the pain. |
Given how often this would be needed, this should be enough. And/or we add a button to the hardware tab (for that radio, if touchscreen enabled) to reset the touchscreen config (might also be good to show the current setting on the diags screen)... |
FWIW, mine was 109 ... I won't make any IQ jokes as I'll surely lose! 🤣 explorer_Y3GzZe1NsO.mp4 |
This is normal, see current definition: #define GT911_CFG_NUMBER 0x6D |
Perfect! :) I'll leave it for a day to give Kevin from the related issue a chance to respond, and then merge. Actually, scratch that... it can get merged now... |
Related to #2063