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
Store touchscreen config #169
Conversation
Could you document somewhere how those values can be obtained? |
master-firmware/src/config.c
Outdated
parameter_scalar_declare_with_default(&screen.ay, &screen.ns, "ay", 0); | ||
parameter_scalar_declare_with_default(&screen.bx, &screen.ns, "bx", 0); | ||
parameter_scalar_declare_with_default(&screen.by, &screen.ns, "by", 0); | ||
parameter_scalar_declare_with_default(&screen.cx, &screen.ns, "cx", 0); |
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.
Might be better not to have any defaults as the default value of 0. will cause the screen to act in weird ways. I would prefer to have an assert fail.
8e774c7
to
a9750d6
Compare
a9750d6
to
b5111e9
Compare
Tested and working now |
Where should I put the file ? A README.md in the gui folder (../src/gui/) ? |
Yes I think this would work. |
1 similar comment
Yes I think this would work. |
Each touchscreen need to have his own configuration to work well.
Next step will be to implement touch screen calibration.