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

Add STM32G0B/C support #1036

Merged
merged 7 commits into from
Jun 7, 2023
Merged

Conversation

chris-durand
Copy link
Member

@chris-durand chris-durand commented Jun 5, 2023

  • Enable devices
  • Support G0B/C shared interrupt handlers
    • I2C
    • FDCAN, unsupported and deactivated on G0B/C for now
    • SPI
  • Fix DMA driver (fixed by device data correction in modm-devices)

Not tested yet.

The channel ids were incorrectly sorted as strings and then converted
to int. The conversion must happen first for the correct ordering.
The bug has no practical effect since no device has more than 8
channels.
@chris-durand
Copy link
Member Author

@klsc-zeat Do you require FDCAN support on STM32G0B/C? I have disabled the driver on those devices for now.

@chris-durand chris-durand added the ci:hal Triggers the exhaustive HAL compile CI jobs label Jun 6, 2023
@klsc-zeat
Copy link

@klsc-zeat Do you require FDCAN support on STM32G0B/C? I have disabled the driver on those devices for now.

Unfortunately yes but only FDCAN1. But i think i will manage that with the examples from the branch. I will post a PR but it will probably take me some time.

@salkinium salkinium linked an issue Jun 7, 2023 that may be closed by this pull request
@chris-durand chris-durand marked this pull request as ready for review June 7, 2023 18:37
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.

Awesome, this has been on my todo list for a long time!

@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 Jun 7, 2023
@salkinium salkinium merged commit 768d749 into modm-io:develop Jun 7, 2023
39 checks passed
@chris-durand chris-durand deleted the feature/stm32g0b_c branch June 7, 2023 21:35
@salkinium salkinium added this to the 2023q2 milestone Oct 1, 2023
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.

STM32G0B1 currently black listed
3 participants