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 Support stm32h5 for stm32h573i based on stm32_fsdev G0 implementa… #2373

Merged
merged 6 commits into from
Dec 18, 2023

Conversation

Lurcy38
Copy link
Contributor

@Lurcy38 Lurcy38 commented Dec 12, 2023

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)

…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)
@Lurcy38
Copy link
Contributor Author

Lurcy38 commented Dec 12, 2023

i can send some boards if you need it, send me the adresse via the private email
regards

@HiFiPhile
Copy link
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 ?

0001-Fix-CI-typo.patch

@Lurcy38
Copy link
Contributor Author

Lurcy38 commented Dec 13, 2023

done: you should have the access

@Lurcy38
Copy link
Contributor Author

Lurcy38 commented Dec 13, 2023

patch applied

@Dolphindalt Dolphindalt mentioned this pull request Dec 15, 2023
1 task
Copy link
Owner

@hathach hathach left a comment

Choose a reason for hiding this comment

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

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.

Copy link
Owner

@hathach hathach left a comment

Choose a reason for hiding this comment

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

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.

@hathach hathach merged commit 7c181c4 into hathach:master Dec 18, 2023
49 checks passed
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