Add Support stm32h5 for stm32h573i based on stm32_fsdev G0 implementa…#2373
Merged
hathach merged 6 commits intohathach:masterfrom Dec 18, 2023
Merged
Add Support stm32h5 for stm32h573i based on stm32_fsdev G0 implementa…#2373hathach merged 6 commits intohathach:masterfrom
hathach merged 6 commits intohathach:masterfrom
Conversation
…tion Tested on Windows with IAR and gcc toolchain ( via cubeide) Tested only with audio examples, but sould be ok for other examples Cmake pacthed : but not tested Linux build : not tested Added a temporary workaround in make file to support H5 HAL repo ( ARMCC_VERSION=0)
Contributor
Author
|
i can send some boards if you need it, send me the adresse via the private email |
Collaborator
|
Nice PR ! I'm trying to push a commit to fix CI & typo but I got permission denied, could you give maintainers write permission or simply apply the patch ? |
Contributor
Author
|
done: you should have the access |
Contributor
Author
|
patch applied |
hathach
reviewed
Dec 15, 2023
Owner
There was a problem hiding this comment.
thank you for your PR. I have push some update to
- correct some cmake build
- rename board to stm32h573i_dk (dash does not play well with some build system)
- use linker script from st cmsis since it is available.
- add h5 to cmake ci build
- add h567nucleo since I have this board. Though usb isn't showed up yet. Probably due to clock setup (just copied from your dk just now), will fix in later push.
hathach
approved these changes
Dec 18, 2023
Owner
hathach
left a comment
hathach
left a comment
There was a problem hiding this comment.
thank you for your PR, everything is working well now. The usb issue previously is due to h563 need 8Mhz clock out put from st-link onboard, while I tried to use my jlink. Will probably try to get it running without stlink later on.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe the PR
Add Support stm32h5 for stm32h573i based on stm32_fsdev stm32g0
Board supported : STM32H573I-DK
Implementation Tested on Windows with IAR and GCC toolchain ( via cubeide) and working properly
Additional context
Tested only with audio examples, but sould be ok for other examples
cmake patched : but not tested Linux build : not tested
Added a temporary workaround in make file to support H5 HAL repo ( ARMCC_VERSION=0)