Skip to content

Commit

Permalink
Remove error for 16 MPU resgion from IAR ports (#1149)
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
aggarg authored Sep 23, 2024
1 parent 9788e7e commit c0bfbb9
Show file tree
Hide file tree
Showing 13 changed files with 0 additions and 62 deletions.
5 changes: 0 additions & 5 deletions portable/ARMv8M/non_secure/portable/IAR/ARM_CM33/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -54,11 +54,6 @@
#define portDONT_DISCARD __root
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif
/*-----------------------------------------------------------*/

/* ARMv8-M common port configurations. */
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,16 +58,11 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif

#ifndef configENABLE_MVE
#define configENABLE_MVE 0
#elif( configENABLE_MVE != 0 )
#error configENABLE_MVE must be left undefined, or defined to 0 for the Cortex-M33.
#endif

/*-----------------------------------------------------------*/

/**
Expand Down
4 changes: 0 additions & 4 deletions portable/ARMv8M/non_secure/portable/IAR/ARM_CM35P/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,6 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif

#ifndef configENABLE_MVE
#define configENABLE_MVE 0
#elif( configENABLE_MVE != 0 )
Expand Down
5 changes: 0 additions & 5 deletions portable/ARMv8M/non_secure/portable/IAR/ARM_CM55/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -63,11 +63,6 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif
/*-----------------------------------------------------------*/

/**
* @brief Critical section management.
*/
Expand Down
5 changes: 0 additions & 5 deletions portable/ARMv8M/non_secure/portable/IAR/ARM_CM85/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -63,11 +63,6 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif
/*-----------------------------------------------------------*/

/**
* @brief Critical section management.
*/
Expand Down
5 changes: 0 additions & 5 deletions portable/IAR/ARM_CM33/non_secure/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -54,11 +54,6 @@
#define portDONT_DISCARD __root
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif
/*-----------------------------------------------------------*/

/* ARMv8-M common port configurations. */
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/
Expand Down
5 changes: 0 additions & 5 deletions portable/IAR/ARM_CM33_NTZ/non_secure/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -58,16 +58,11 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif

#ifndef configENABLE_MVE
#define configENABLE_MVE 0
#elif( configENABLE_MVE != 0 )
#error configENABLE_MVE must be left undefined, or defined to 0 for the Cortex-M33.
#endif

/*-----------------------------------------------------------*/

/**
Expand Down
4 changes: 0 additions & 4 deletions portable/IAR/ARM_CM35P/non_secure/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,6 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif

#ifndef configENABLE_MVE
#define configENABLE_MVE 0
#elif( configENABLE_MVE != 0 )
Expand Down
4 changes: 0 additions & 4 deletions portable/IAR/ARM_CM35P_NTZ/non_secure/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,6 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif

#ifndef configENABLE_MVE
#define configENABLE_MVE 0
#elif( configENABLE_MVE != 0 )
Expand Down
5 changes: 0 additions & 5 deletions portable/IAR/ARM_CM55/non_secure/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -63,11 +63,6 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif
/*-----------------------------------------------------------*/

/**
* @brief Critical section management.
*/
Expand Down
5 changes: 0 additions & 5 deletions portable/IAR/ARM_CM55_NTZ/non_secure/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -63,11 +63,6 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif
/*-----------------------------------------------------------*/

/**
* @brief Critical section management.
*/
Expand Down
5 changes: 0 additions & 5 deletions portable/IAR/ARM_CM85/non_secure/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -63,11 +63,6 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif
/*-----------------------------------------------------------*/

/**
* @brief Critical section management.
*/
Expand Down
5 changes: 0 additions & 5 deletions portable/IAR/ARM_CM85_NTZ/non_secure/portmacro.h
Original file line number Diff line number Diff line change
Expand Up @@ -63,11 +63,6 @@
#include "portmacrocommon.h"
/*-----------------------------------------------------------*/

#if ( configTOTAL_MPU_REGIONS == 16 )
#error 16 MPU regions are not yet supported for this port.
#endif
/*-----------------------------------------------------------*/

/**
* @brief Critical section management.
*/
Expand Down

0 comments on commit c0bfbb9

Please sign in to comment.