-
Notifications
You must be signed in to change notification settings - Fork 238
[nrf noup] boot: zephyr: Disable self RWX #491
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
Conversation
918b809
to
64f45fd
Compare
460752d
to
a25c7c4
Compare
@@ -163,6 +182,84 @@ struct arm_vector_table { | |||
uint32_t reset; | |||
}; | |||
|
|||
static void __ramfunc jump_in(uint32_t reset) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we have better name for this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, however this is the hardest part. Any suggestions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you seen names I have invented?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope. Bring it on.
a25c7c4
to
d0d8188
Compare
c02faf0
to
3c510f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just one nit
Disables read write and execute on mcuboots NVM at the end of execution. Signed-off-by: Mateusz Michalek <[email protected]>
3c510f1
to
6771d06
Compare
|
Disables read write and execute on mcuboots NVM
at the end of execution.