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

Helloradio modification: #5937

Closed

Conversation

helloradiosky
Copy link

@helloradiosky helloradiosky commented Feb 21, 2025

Summary of changes:

  1. Redefine the V14 toggle switch
  2. Increase voltage measurement for V12 and V14
  3. Add voice recognition chip drivers to V14 and V16
  4. Add gyroscope chip driver to V14 and V16

1. Redefine the V14 toggle switch
2. Increase voltage measurement for V12 and V14
3. Add voice recognition chip drivers to V14 and V16
4. Add gyroscope chip driver to V14 and V16
@richardclli
Copy link
Collaborator

richardclli commented Mar 7, 2025

@rotorman I remembered that you have modded a gyro with tx16s, how did you map the readings in EdgeTX? I think the gyro in V16 should follow your approach, could you please give some comments about the IMU integration in this PR?

Copy link
Collaborator

@richardclli richardclli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Normally hardware drivers should be put in the target folder e.g. targets/horus , and for those that are used by more than one target should goes to targets/common instead.

@rotorman
Copy link
Member

rotorman commented Mar 7, 2025

Regarding IMU, see: #2113 (also #1744 and #617)

@helloradiosky
Copy link
Author

Normally hardware drivers should be put in the target folder e.g. targets/horus , and for those that are used by more than one target should goes to targets/common instead

Okay, I will follow these rules in the future

@pfeerick
Copy link
Member

pfeerick commented Mar 9, 2025

Unless there is a real need to (i.e. major re-write or unable to rebase on main), changes can continue to be made in the one branch and PR. But for simplicity, this is continued in #5977

@pfeerick pfeerick closed this Mar 9, 2025
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.

4 participants