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

feat: support for HelloRadioSky V16 #5317

Merged
merged 25 commits into from
Sep 1, 2024
Merged

feat: support for HelloRadioSky V16 #5317

merged 25 commits into from
Sep 1, 2024

Conversation

richardclli
Copy link
Collaborator

Initial support of HelloRadioSky V16:

  • basic hardware mapping
  • RGB LED
  • specialized 6 pos switches
  • battery sensors

Extra features will be submitted by other PRs.

@richardclli richardclli marked this pull request as draft July 21, 2024 23:33
@richardclli richardclli self-assigned this Jul 21, 2024
@richardclli richardclli added this to the 2.11 milestone Jul 21, 2024
Copy link
Member

@pfeerick pfeerick left a 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.

.github/ISSUE_TEMPLATE/bug-report.yml Outdated Show resolved Hide resolved
fw.json Outdated Show resolved Hide resolved
radio/src/targets/common/arm/stm32/stm32_ws2812.cpp Outdated Show resolved Hide resolved
radio/src/targets/common/arm/stm32/stm32_ws2812.h Outdated Show resolved Hide resolved
radio/src/targets/horus/CMakeLists.txt Outdated Show resolved Hide resolved
radio/src/targets/horus/csd203_sensor.h Outdated Show resolved Hide resolved
@pfeerick pfeerick changed the title feat(v16): Support of HelloRadioSky V16 feat: support for HelloRadioSky V16 Jul 28, 2024
@pfeerick pfeerick force-pushed the hrs_v16 branch 4 times, most recently from 80b6e6f to 8fd3b73 Compare August 1, 2024 09:00
@pfeerick pfeerick force-pushed the hrs_v16 branch 3 times, most recently from a57363c to 79102ed Compare August 18, 2024 23:44
@pfeerick pfeerick marked this pull request as ready for review August 29, 2024 22:50
Copy link
Member

@pfeerick pfeerick left a 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.

@pfeerick pfeerick merged commit 17bf9dc into main Sep 1, 2024
51 checks passed
@pfeerick pfeerick deleted the hrs_v16 branch September 1, 2024 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants