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

Update AIRBOTSUPERF4 SPI DMA #687

Merged
merged 1 commit into from
Feb 6, 2025
Merged

Update AIRBOTSUPERF4 SPI DMA #687

merged 1 commit into from
Feb 6, 2025

Conversation

ot0tot
Copy link
Contributor

@ot0tot ot0tot commented Feb 6, 2025

Defining DMA for only SPI_SDO results in SPI_SDO not being assigned DMA. Removing the unnecessary assignment allows both streams to be assigned properly.

Before change:

# dma show

Currently active DMA:
--------------------
DMA1 Channel 1: ADC 1
DMA1 Channel 2: LED_STRIP
DMA1 Channel 3: DSHOT_BITBANG 3
DMA1 Channel 4: DSHOT_BITBANG 2
DMA1 Channel 5: SPI_SDO 1
DMA1 Channel 6: SPI_SDI 1
DMA1 Channel 7: SPI_SDI 3
DMA2 Channel 1: FREE
DMA2 Channel 2: FREE
DMA2 Channel 3: FREE
DMA2 Channel 4: FREE
DMA2 Channel 5: FREE
DMA2 Channel 6: FREE
DMA2 Channel 7: FREE

After change:

# dma show

Currently active DMA:
--------------------
DMA1 Channel 1: ADC 1
DMA1 Channel 2: LED_STRIP
DMA1 Channel 3: DSHOT_BITBANG 3
DMA1 Channel 4: DSHOT_BITBANG 2
DMA1 Channel 5: SPI_SDO 1
DMA1 Channel 6: SPI_SDI 1
DMA1 Channel 7: SPI_SDO 3
DMA2 Channel 1: SPI_SDI 3
DMA2 Channel 2: FREE
DMA2 Channel 3: FREE
DMA2 Channel 4: FREE
DMA2 Channel 5: FREE
DMA2 Channel 6: FREE
DMA2 Channel 7: FREE

Defining DMA for only SPI_SDO results in SPI_SDI not being assigned DMA. Removing the unnecessary assignment allows both streams to be assigned properly.
@haslinghuis haslinghuis merged commit ebfa401 into master Feb 6, 2025
1 check passed
@haslinghuis haslinghuis deleted the ot0tot-patch-1 branch February 6, 2025 14:06
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.

3 participants