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

Add iFlight Commando8 ELRS PR fixed version2 #1982

Merged
merged 4 commits into from
Aug 16, 2022
Merged

Conversation

XING-IF
Copy link
Contributor

@XING-IF XING-IF commented May 21, 2022

Add iFlight Commando8 ELRS PR fixed version2
PR1902 seems to be unfixable broken for some reason, so we started all over and integrated latest changes and corrections.
Sorry for this approach, I hope this will be OK now.

Companion files will be updated with another commit.
Based on the EdgeTX PR #1902

[edit: pfeerick]
To power on or off this radio, you double press and hold the power button.

  • The first press shows the battery charge level
  • The second actually powers it on or off

To access DFU or bootloader, you press the left joystick down (not centre press) and then while holding that down, hold down the power button. After about a second, the battery indicator will start animating. If USB is connected, it will go into DFU mode. If not, the bootloader menu will be shown.

To access the USB (for storage, joystick mode), you need to press and hold the power button after plugging in the USB until the battery leds animate. On initial connect it appears to be in charge mode. If you want to disconnect and resume charging, press and hold the power button again until the animation stops.
[/end edit]

image

@deadbytefpv
Copy link

Tested the binary from this PR.
All seems to work on my initial and short testing.

None of the issues I've reported on the older PR #1902
Will have a play with the radio a bit more and report back should I notice anything amiss.

@XING-IF
Copy link
Contributor Author

XING-IF commented Jun 3, 2022

I think we're finally there, as well as all changes from PR1902 have been successfully ported to this PR.

@raphaelcoeffic raphaelcoeffic force-pushed the IF_com8_v2 branch 3 times, most recently from 7cfee58 to 29d1feb Compare June 19, 2022 13:34
Copy link
Member

@raphaelcoeffic raphaelcoeffic left a comment

Choose a reason for hiding this comment

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

The changes for the radio firmware are now as expected and already reviewed (in #1902).

However, I had to rebase / squash commits to cleanup this mess. In future, I expect things to be already done this way.

Apart from that, the original code had support for Companion, which is not the case anymore, and thus missing.

@pfeerick pfeerick self-assigned this Aug 3, 2022
@pfeerick pfeerick added this to the 2.8 milestone Aug 5, 2022
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.

Seems ready to roll. Just waiting on final checks from iFlight crew now.

XING-IF and others added 4 commits August 16, 2022 18:11
@pfeerick pfeerick merged commit e1ea57c into EdgeTX:main Aug 16, 2022
@pfeerick pfeerick mentioned this pull request Jul 3, 2024
1 task
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.

5 participants