Skip to content

Conversation

@aggarg
Copy link
Member

@aggarg aggarg commented Sep 22, 2024

Description

Support for 16 MPU regions was added to Cortex-M33, M35P, M55 and M85 ports was added in this PR but the compile time error check was not removed. This results in compilation error when 16 MPU regions are used. This PR removes the not needed compile time error check.

Test Steps

Build IAR demo with configTOTAL_MPU_REGIONS set to 16.

Checklist:

  • I have tested my changes. No regression in existing tests.
  • [NA] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

https://forums.freertos.org/t/cortex-m55-and-16-region-mpu-support/21470

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Support for 16 MPU regions was added to Cortex-M33, M35P, M55 and M85
ports was added but the compile time error check was not removed. This
results in compilation error when 16 MPU regions are used. This PR
removes the not needed compile time error check.

It was reported here - https://forums.freertos.org/t/cortex-m55-and-16-region-mpu-support/21470.

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
@aggarg aggarg requested a review from a team as a code owner September 22, 2024 07:58
@sonarqubecloud
Copy link

@aggarg aggarg merged commit c0bfbb9 into FreeRTOS:main Sep 23, 2024
@aggarg aggarg deleted the armv8m_mpu_16_regions branch September 23, 2024 05:09
@aggarg aggarg changed the title Remove error for 16 MPU resgion from IAR ports Remove error for 16 MPU regions from IAR ports Sep 23, 2024
moninom1 pushed a commit to moninom1/FreeRTOS-Kernel that referenced this pull request Sep 30, 2025
* Add in a Cortex R5F MPU demo for the Hercules RM57 Development Kit.
* Add in a Cortex R4F MPU demo for the Hercules RM46 Development Kit.
* Provide a Code Composer Studio (CCS) project for running these demos.
* Provide a CMakeLists.txt file to allow for compilation of the demos without use of an IDE.
* Add a CI-CD build of these demos using CMake with Fetch-Content.
* Include necessary README to explain the new demos.
---------
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