Arch arm v8m mpu remove hal #8467
Merged
Conversation
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
Codecov Report
@@ Coverage Diff @@
## master #8467 +/- ##
=======================================
Coverage 52.35% 52.35%
=======================================
Files 195 195
Lines 24694 24694
Branches 5128 5128
=======================================
Hits 12929 12929
Misses 9695 9695
Partials 2070 2070Continue to review full report at Codecov.
|
cd401ce to
c70a67a
Compare
c70a67a to
624e8cf
Compare
d79ee82 to
6303405
Compare
3cedcde to
13c4dff
Compare
This was referenced Jun 25, 2018
bf31ddb to
dd6451f
Compare
pizi-nordic
approved these changes
Jun 29, 2018
|
Looks good to me @ioannisg. Will let others approve though. |
d6138bb to
812fa08
Compare
andrewboie
approved these changes
Jul 3, 2018
nashif
approved these changes
Jul 3, 2018
812fa08 to
33aeb07
Compare
This commit removes the redundant HAL definition for the ARM Cortex-M MPU registers, and modifies the ARM MPU driver implementation to directly use the provided HAL from CMSIS. Signed-off-by: Ioannis Glaropoulos <Ioannis.Glaropoulos@nordicsemi.no>
This commit modifies the interal macro defitions for MPU region attributes in arm_mpu.h to use bitset flags directly from CMSIS. Signed-off-by: Ioannis Glaropoulos <Ioannis.Glaropoulos@nordicsemi.no>
Duplicate the ARM MPU-related registers' definitions specifically for Beetle SOC which is not compliant with ARM CMSIS and does not include the ARM CMSIS headers. Signed-off-by: Ioannis Glaropoulos <Ioannis.Glaropoulos@nordicsemi.no>
This commit moves the macro definitions and convenience wrappers for ARM MPU that are specific to ARMv6-m, and ARMv7-m to an arch-specific MPU header file. It leaves only the generic content in arm_mpu.h, i.e. the content that is supposed to be common for ARMv8-M MPU. Signed-off-by: Ioannis Glaropoulos <Ioannis.Glaropoulos@nordicsemi.no>
33aeb07 to
960de32
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This pull request refactors the arm_mpu.h header, as part of the preparations for adding support for ARMv8-M MPU (#8018 ). Contains the following:
The PR does not change the current functionality.
Advice to review commit by commit.
Super-set of #8603 (rebased over #8603)