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

Fix DMA assignments for AIKONF4V2 #635

Merged
merged 2 commits into from
Dec 10, 2024
Merged

Fix DMA assignments for AIKONF4V2 #635

merged 2 commits into from
Dec 10, 2024

Conversation

ot0tot
Copy link
Contributor

@ot0tot ot0tot commented Dec 10, 2024

M1-3 were all assigned to the same DMA stream.
ADC1 conflicts with SPI1_RX and M1, must use ADC3.
Features should not be defined in the config.

@ot0tot
Copy link
Contributor Author

ot0tot commented Dec 10, 2024

Additionally, motors 5 and 6 conflict with SPI3_TX for the OSD. Should we remove them?

@sugaarK
Copy link
Member

sugaarK commented Dec 10, 2024

Additionally, motors 5 and 6 conflict with SPI3_TX for the OSD. Should we remove them?

yep do it

@ot0tot ot0tot requested review from nerdCopter and sugaarK December 10, 2024 13:59
@haslinghuis haslinghuis merged commit 0d8fbab into master Dec 10, 2024
1 check passed
@haslinghuis haslinghuis deleted the ot0tot-patch-1 branch December 10, 2024 14:30
@sugaarK
Copy link
Member

sugaarK commented Dec 10, 2024

@haslinghuis was waiting on confirmation from Aikon that they target had been tested

@haslinghuis
Copy link
Member

haslinghuis commented Dec 10, 2024

Sure @ot0tot got it right this time. If not we always can do another iteration.

My initial PR was merged before getting test results.

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