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.

Sign up for GitHub

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

Jump to bottom

Minor STM32 fixes #714

Merged
merged 2 commits into from
Jul 29, 2024
Merged

Minor STM32 fixes #714

merged 2 commits into from
Jul 29, 2024

Conversation

patryk3211
Copy link
Contributor

This fixes some minor issues I found.

  • Assign spi_dma_enable to configuration value.
  • Fix interrupt driven spi transfers. The code was causing issues at higher SPI clock speeds.

@patryk3211
STM32 SPI configuration fixes
3058e61 
- Enable DMA assignment is now correct
- Added a volatile flag to port state
@patryk3211
Fix non-DMA SPI bulk transfers on STM32
285dbef 
The code was causing issues at higher clock speeds
@Paciente8159 Paciente8159 added HAL: STM32F1x HAL: STM32F1x HAL: STM32F4x HAL: STM32F4x needs: awaiting review Awaiting review HAL: STM32F0x HAL: STM32F0x labels Jul 29, 2024
@Paciente8159 Paciente8159 removed the needs: awaiting review Awaiting review label Jul 29, 2024
@Paciente8159 Paciente8159 merged commit 23d779e into Paciente8159:master Jul 29, 2024
19 checks passed
@patryk3211 patryk3211 deleted the stm32_spi_fix_2 branch July 30, 2024 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Paciente8159 Paciente8159 Paciente8159 approved these changes

Assignees
No one assigned
Labels
HAL: STM32F0x HAL: STM32F0x HAL: STM32F1x HAL: STM32F1x HAL: STM32F4x HAL: STM32F4x
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@patryk3211 @Paciente8159