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

Refactor TIMER/DMA framework #3833

Merged
merged 14 commits into from Sep 24, 2018

Conversation

Projects
None yet
3 participants
@digitalentity
Copy link
Member

commented Sep 1, 2018

Preparation for DSHOT

@digitalentity digitalentity added this to the 2.1 milestone Sep 1, 2018

@digitalentity digitalentity referenced this pull request Sep 1, 2018

Closed

Add DSHOT support #1393

@digitalentity digitalentity force-pushed the de_timer_dma_and_cleanups branch from 959b584 to 30e355d Sep 1, 2018

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Sep 6, 2018

F3/F4/F7 works. F7 refactoring needed.

@digitalentity digitalentity force-pushed the de_timer_dma_and_cleanups branch from 078c086 to 6ebec53 Sep 17, 2018

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Sep 17, 2018

@TwoToneEddy code is generally ready to start implementing DSHOT on top of this branch.

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Sep 17, 2018

F4 & F7 mostly verified to work. F3 not tested

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Sep 18, 2018

Almost done 😄

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Sep 19, 2018

Ok, this should be ready for testing. SoftSerial + PPM in not tested. Motor output + Ledstrip works

digitalentity added some commits Sep 19, 2018

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Sep 19, 2018

Ready for testing

@digitalentity digitalentity referenced this pull request Sep 22, 2018

Merged

DSHOT #3879

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Sep 23, 2018

All motor protocols verified on COLIBRI_RACE (F3), REVO (F4) and OMNIBUSF7NXT (F7).
SoftSerial verified to work on REVO (F4, SmartPort, LTM)
If nobody objects I intend to merge this.

@digitalentity digitalentity merged commit 78bc53f into development Sep 24, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@digitalentity digitalentity deleted the de_timer_dma_and_cleanups branch Sep 24, 2018

@TwoToneEddy

This comment has been minimized.

Copy link

commented Sep 24, 2018

Daft question, if I now build the development branch is dshot ready to try out?

@TwoToneEddy

This comment has been minimized.

Copy link

commented Sep 24, 2018

Sorry I'm an idiot, I see you have a branch for dshot. Can I test this if I add the USE_DSHOT to the target of the board I want to test with?

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Sep 24, 2018

@TwoToneEddy Defining USE_DSHOT should be enough to allow DSHOT for a target (if all timer mappings are compatible). Use caution - I didn't test the code with actual ESCs

@stronnag

This comment has been minimized.

Copy link
Collaborator

commented Sep 28, 2018

I flew the new timer/dma stuff today (without USE_DHSOT) on a couple of F4 FCs. No adverse affects, no regressions.

@digitalentity

This comment has been minimized.

Copy link
Member Author

commented Sep 28, 2018

@stronnag good to know, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.