Skip to content

EFM32GG: Add support for uVisor #2875

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

Merged
merged 3 commits into from
Oct 12, 2016
Merged

EFM32GG: Add support for uVisor #2875

merged 3 commits into from
Oct 12, 2016

Conversation

AlessandroA
Copy link
Contributor

This PR adds support for uVisor to the EFM32GG target.

Changes:

  • Linker script and startup file, as documented by the uVisor porting guide.
  • Apply NVIC virtualization (backwards-compatible when uVisor is not present).
  • Use SECURE_ACCESS() APIs to read/write from the device ROM table at startup time. Those APIs map to a regular memory access when uVisor is not present.

@sg- @0xc0170 @meriac @Patater

@Patater
Copy link
Contributor

Patater commented Sep 30, 2016

Specifically, see https://github.com/ARMmbed/uvisor/blob/master/docs/core/PORTING.md#linker-script for the linker script porting recommendation and reasoning.

@sg-
Copy link
Contributor

sg- commented Oct 7, 2016

@c1728p9 Can you review

@sg-
Copy link
Contributor

sg- commented Oct 7, 2016

@AlessandroA can you resolve the file move conflicts?

@c1728p9
Copy link
Contributor

c1728p9 commented Oct 7, 2016

PR looks good to me

@sg- sg- removed the needs: review label Oct 7, 2016
@0xc0170
Copy link
Contributor

0xc0170 commented Oct 10, 2016

bump @AlessandroA

With CMSIS 5, the NVIC functions are virtualizable. Rename the EFM32
NVIC functions to support being virtualized.
To allow enabling of uVisor on EFM32, the ROM table must be accessed
through the CMSIS-provided SECURE_ACCESS macro.
@AlessandroA
Copy link
Contributor Author

Updated.

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 10, 2016

/morph test

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1091

All builds and test passed!

@sg-
Copy link
Contributor

sg- commented Oct 10, 2016

/morph test-nightly

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 1092

Test failed!

@bridadan
Copy link
Contributor

Results look ok, failure is from the TLS and CFSTORE tests that are currently failing in the nightly.

@sg- sg- merged commit 348a155 into ARMmbed:master Oct 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants