Skip to content

Commit

Permalink
Fix cmake configure for STM32F4XX and STM32F7XX (#59)
Browse files Browse the repository at this point in the history
Fixed bug that prevented STM32F4XX and STM32F7XX from correctly
configure with cmake.

Description
-----------
Setting `FREERTOS_PLUS_FAT_PORT` to STM32F4XX would throw fatal error at
line 57 of root CMakeLists.txt since not in
`FREERTOS_PLUS_FAT_PORT_LIST`.
Setting `FREERTOS_PLUS_FAT_PORT` to STM32FXX as in the list of ports
(notice no number before XX), would throw error since at
`portable/CMakeLists.txt:38` it would NOT mach with any generator and
result in no sources for `freertos_plus_fat_port` target.

Test Steps
-----------
<!-- Describe the steps to reproduce. -->
set `FREERTOS_PLUS_FAT_PORT` to either STM32F4XX or STM32F7XX

Checklist:
----------
<!--- Go over all the following points, and put an `x` in all the boxes
that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
- [ ] I have tested my changes. No regression in existing tests.
- [ ] I have modified and/or added unit-tests to cover the code changes
in this Pull Request.

Related Issue
-----------
<!-- If any, please provide issue ID. -->

By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.
  • Loading branch information
safesintesi authored Dec 20, 2023
1 parent 90281cb commit bc13a8a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,8 @@ set(FREERTOS_PLUS_FAT_PORT_LIST
AVR32_UC3
LPC18XX
POSIX
STM32FXX
STM32F4XX
STM32F7XX
STM32HXX
ZYNQ
ZYNQ_2019_3
Expand Down

0 comments on commit bc13a8a

Please sign in to comment.