Skip to content
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

Angle: add support for MT6816, MT6826S sensors #6735

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

nefelim4ag
Copy link
Contributor

@nefelim4ag nefelim4ag commented Nov 12, 2024

For now, this is just a proposal - till there are other real users.

MT6816 - simple 14-bit angle sensor, can be read and that's it.
MT6826S - Has calibration capabilities - implemented, hysteresis, and 15-bit output
MT6835 - (21 bit) should work with code from MT6826S, we anyway will ignore additional bits.

For now, I mostly play around with MT6826S.

Thanks.


BTW, it would be nice to have a generic query command for the angle sensor, like ACCELEROMETER_QUERY.

Signed-off-by: Timofey Titovets <nefelim4ag@gmail.com>
Signed-off-by: Timofey Titovets <nefelim4ag@gmail.com>
@KevinOConnor
Copy link
Collaborator

Thanks. As very high-level feedback, we can certainly add support for these additional sensors. The code would need to pass the regression tests though.

Cheers,
-Kevin

@nefelim4ag
Copy link
Contributor Author

nefelim4ag commented Nov 20, 2024

But for now it is failing because of image limitations:

...  ../../or1k-linux-musl/bin/ld: Klipper image is too large

The only idea that I have is to disable angle sensors for ar100.

Signed-off-by: Timofey Titovets <nefelim4ag@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants