-
-
Notifications
You must be signed in to change notification settings - Fork 339
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: support for HelloRadioSky V16 #5317
Conversation
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.
Few minor corrections / changes from initial skim of the PR. Will make some of the changes later today, and add missing companion skeleton code.
80b6e6f
to
8fd3b73
Compare
a57363c
to
79102ed
Compare
Needs simulator support for full function
Not putting any real effort into simulator graphics given auto-generated UI is now on the agenda
... taranis boards file is the same...
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.
After some final corrections... like re-enabling the S1 pot for X10 radios (probably got lost in rebase) 😆 ... LGTM... MT12 as another WS2812 equiped radio appears to be working fine, simulator is working now, although not pretty (but given moving to auto-generated simulator UI is in the works, I'm not putting any more effort into it beyond it being functional), companion appears to be working fine with the radio, and core radio functionality seems to be working.
Initial support of HelloRadioSky V16:
Extra features will be submitted by other PRs.