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

Remove error for 16 MPU regions from IAR ports #1149

Merged
merged 1 commit into from
Sep 23, 2024

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
Copy link

sonarcloud bot commented Sep 22, 2024

@aggarg aggarg merged commit c0bfbb9 into FreeRTOS:main Sep 23, 2024
16 checks passed
@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
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