-
Notifications
You must be signed in to change notification settings - Fork 340
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
Integration of H5 family #320
Merged
Merged
Changes from all commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
a4ff415
Added h5.cmake file
Hish15 82ff509
docs:Added doc to add new family
Hish15 ecaefbe
common.cmake:Added H5 to STM32_SUPPORTED_FAMILIES_LONG_NAME
Hish15 0ab5956
CI: Added H5 family to CI run
Hish15 ae613dc
TRASH: CI: Removed all but H5 from tests
Hish15 93725b7
maintiain.md added more info
Hish15 9ffdd49
utilities.cmake: Added versions for fetch feature
Hish15 8bf0520
h5: fixed cpu core to cortex-m33
Hish15 f270cb0
Merge branch 'master' into P2H/develop_H5
Hish15 9e2a580
Updated CMSIS version to v5.9.0 for H5 family
Hish15 c6c84f4
Corrected CMSIS and HAL versions
Hish15 9b57e2e
common.cmake includes h5 properly
Hish15 e37e0de
Added H5 devices to list of known devices
Hish15 d9f8585
Added conf file for all tests
Hish15 c7e9b42
Updated doc to have a working CI for new families
Hish15 fb8cbae
h5.cmake:Corrected fpu flag
Hish15 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| set(STM32_H5_TYPES | ||
| H503xx H562xx H563xx H573xx | ||
| ) | ||
|
|
||
| set(STM32_H5_TYPE_MATCH | ||
| "H503.." "H562.." "H563.." "H573.." | ||
| ) | ||
| set(STM32_H5_RAM_SIZES | ||
| 32K 640K 640K 640K | ||
| ) | ||
| set(STM32_H5_CCRAM_SIZES | ||
| 0K 0K 0K 0K | ||
| ) | ||
|
|
||
| stm32_util_create_family_targets(H5) | ||
|
|
||
| target_compile_options(STM32::H5 INTERFACE | ||
| -mcpu=cortex-m33 -mfloat-abi=hard -mfpu=fpv5-sp-d16 -mthumb | ||
| ) | ||
|
|
||
| target_link_options(STM32::H5 INTERFACE | ||
| -mcpu=cortex-m33 -mfloat-abi=hard -mfpu=fpv5-sp-d16 -mthumb | ||
| ) |
This file contains 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
This file contains 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| # Supporting new families | ||
|
|
||
| ## Write device file | ||
|
|
||
| a file <family>.cmake must be added to the folder cmake/stm32 | ||
| This file containes the differents devices and the regex used to parse them. | ||
| It also give information on the RAM and CCRAM available | ||
|
|
||
| ## Add family to list of know families | ||
|
|
||
| Update the list `STM32_SUPPORTED_FAMILIES_LONG_NAME` located in the cmake/stm32/common.cmake file | ||
|
|
||
| ## Add devices to list of devices in devices.cmake | ||
|
|
||
Hish15 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| Add all known devices to the list of all devices | ||
|
|
||
| ## Update cube, cmsis and hal version in utilities.cmake | ||
|
|
||
| The versions to use can be found as follow: | ||
| - Cube version : a valid tag from the repo https://github.com/STMicroelectronics/STM32Cube${FAMILY} | ||
| - Cmsis version : a valid tag from the repo https://github.com/STMicroelectronics/cmsis_device_${FAMILY_L} | ||
| - Hal version : a valid tag from the repo https://github.com/STMicroelectronics/stm32${FAMILY_L}xx_hal_driver | ||
|
|
||
| ## Add family to CI | ||
|
|
||
| Add the file stm32${FAMILY_L}xx_hal_conf.h file to the root folder of each test. (A template is provided in the corresponding HAL) | ||
| Update the cmake.yml workflow file to include the new family | ||
atsju marked this conversation as resolved.
Show resolved
Hide resolved
|
||
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you make a suggestion with all the changes you want here ?