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

feat!: update DmaStm to support new dma peripherals in G0, G4 and WB devices #169

Merged
merged 57 commits into from
Feb 21, 2024

Conversation

daantimmer
Copy link
Contributor

@daantimmer daantimmer commented Jul 20, 2023

  • fully refactored hal::DmaStm
    • peripherals now require to own a hal::TransmitDmaChannel, hal::ReceiveDmaChannel or hal::TransceiveDmaChannel.
    • peripherals now have a dependency on hal::DmaStm::TransmitStream, hal::DmaStm::ReceiveStream or hal::DmaStm::TransceiveStream
  • hal::DmaStm now properly supports asymetric transfer sizes
  • refactored hal::UartStm/hal::UartStmDma/hal::UartStmDuplexDma:
    • removed ctor duplication
    • removed dmachannels from Config
    • removed empty ctor from Config to allow for designated initializers
    • removed hwFlowControl from Config. This is now infered from which ctor is used
    • removed infra::Optional from uartRts and uartCts members, they are now initialized to dummyPinStm instead if no flow control is required
  • refactored hal::SpiMasterStmDma/hal::SpiSlaveStmDma removed Config
  • added DefaultClockNuckleoG474ret. Configured as 170MHz from an 24MHz HSE
  • added helloworld example

@github-actions
Copy link
Contributor

github-actions bot commented Jul 20, 2023

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 5 0 0.06s
✅ CPP clang-format 15 2 0 1.14s
✅ DOCKERFILE hadolint 1 0 0.19s
✅ JSON eslint-plugin-jsonc 8 0 0 4.05s
✅ JSON jsonlint 8 0 0.28s
✅ JSON prettier 8 4 0 1.36s
⚠️ MARKDOWN markdownlint 6 1 4 2.02s
⚠️ MARKDOWN markdown-link-check 6 2 54.25s
✅ MARKDOWN markdown-table-formatter 6 1 0 1.17s
✅ REPOSITORY checkov yes no 25.46s
✅ REPOSITORY git_diff yes no 1.07s
✅ REPOSITORY grype yes no 20.01s
✅ REPOSITORY secretlint yes no 42.72s
✅ REPOSITORY trivy yes no 6.28s
✅ REPOSITORY trivy-sbom yes no 4.43s
✅ REPOSITORY trufflehog yes no 11.24s
⚠️ SPELL lychee 30 2 87.08s
✅ YAML prettier 7 1 0 1.68s
✅ YAML v8r 7 0 6.17s
✅ YAML yamllint 7 0 0.32s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@daantimmer daantimmer changed the title Feature/refactor dmastm refactor: rework DmaStm to support new dma peripherals in G0, G4 and WB devices Jul 20, 2023
@daantimmer daantimmer changed the title refactor: rework DmaStm to support new dma peripherals in G0, G4 and WB devices feat: update DmaStm to support new dma peripherals in G0, G4 and WB devices Jul 20, 2023
# Conflicts:
#	hal_st/stm32fxxx/AnalogToDigitalPinStm.cpp
# Conflicts:
#	hal_st/stm32fxxx/DmaStm.hpp
#	hal_st/stm32fxxx/UartStmDma.hpp
#	hal_st/stm32fxxx/UartStmDuplexDma.hpp
.vscode/settings.json Outdated Show resolved Hide resolved
CMakePresets.json Outdated Show resolved Hide resolved
CMakePresets.json Outdated Show resolved Hide resolved
@richardapeters richardapeters merged commit d62e03e into main Feb 21, 2024
23 checks passed
@richardapeters richardapeters deleted the feature/refactor-dmastm branch February 21, 2024 11:54
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