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

STM32H7 BDMA driver #1115

Merged
merged 4 commits into from
Feb 2, 2024
Merged

Conversation

chris-durand
Copy link
Member

  • Add STM32H7 BDMA driver
  • Make DAC and SPI drivers compatible with BDMA
  • Add hardware unit test for BDMA and SPI6 on Nucleo-H723ZG

A separate driver is added because the DMA type differs between BDMA and regular DMA instances on H7, but the driver assumes common bit definitions in DmaBase.

Direct dependencies to DmaBase must be removed from drivers for the same reason to be compatible to BDMA. Common definitions are accessed through the DMA channel types instead.

@chris-durand chris-durand added this to the 2024q1 milestone Jan 2, 2024
@chris-durand chris-durand force-pushed the feature/stm32h7_bdma branch 2 times, most recently from 3b20146 to 0439add Compare January 2, 2024 22:05
@chris-durand chris-durand added the ci:hal Triggers the exhaustive HAL compile CI jobs label Jan 2, 2024
@chris-durand chris-durand added ci:hal Triggers the exhaustive HAL compile CI jobs and removed ci:hal Triggers the exhaustive HAL compile CI jobs labels Jan 3, 2024
Copy link
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

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

Yes!

src/modm/platform/dma/stm32-bdma/module.lb Outdated Show resolved Hide resolved
src/modm/platform/dma/stm32-bdma/module.lb Outdated Show resolved Hide resolved
src/modm/platform/dma/stm32-bdma/module.lb Outdated Show resolved Hide resolved
src/modm/platform/dma/stm32-bdma/module.lb Show resolved Hide resolved
@chris-durand chris-durand added ci:hal Triggers the exhaustive HAL compile CI jobs and removed ci:hal Triggers the exhaustive HAL compile CI jobs labels Feb 1, 2024
@chris-durand chris-durand merged commit 2e40ab4 into modm-io:develop Feb 2, 2024
39 checks passed
@chris-durand chris-durand deleted the feature/stm32h7_bdma branch February 2, 2024 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
advanced 🤯 ci:hal Triggers the exhaustive HAL compile CI jobs feature 🚧
Development

Successfully merging this pull request may close these issues.

None yet

2 participants