Skip to content

arch: arm: minor clean-up in interrupt stack size derivation #16355

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

Conversation

ioannisg
Copy link
Member

@ioannisg ioannisg commented May 23, 2019

CONFIG_MPU_REQUIRES_POWER_OF_TWO_ALIGNMENT is taken into
account when allocating the area for the interrupt stack
with the K_THREAD_STACK_DEFINE macro. Therefore, we can
simpllify how the top of the stack is derived during
system initialization.

Signed-off-by: Ioannis Glaropoulos [email protected]

@ioannisg ioannisg requested a review from andrewboie May 23, 2019 13:56
@ioannisg ioannisg marked this pull request as ready for review May 23, 2019 14:49
@ioannisg ioannisg requested a review from MaureenHelm as a code owner May 23, 2019 14:49
@ioannisg ioannisg added area: ARM ARM (32-bit) Architecture Enhancement Changes/Updates/Additions to existing features labels May 24, 2019
CONFIG_MPU_REQUIRES_POWER_OF_TWO_ALIGNMENT is taken into
account when allocating the area for the interrupt stack
using the K_THREAD_STACK_DEFINE macro. Therefore, we can
simplify how the top of the stack is derived during the
initialization of the system, by removing one more
occurrence of CONFIG_MPU_REQUIRES_POWER_OF_TWO_ALIGNMENT.

Signed-off-by: Ioannis Glaropoulos <[email protected]>
@ioannisg ioannisg force-pushed the arch_arm_interrupt_stack_cleanup branch from 5f39b94 to 8e4743a Compare May 25, 2019 11:28
@ioannisg ioannisg added this to the v2.0.0 milestone May 25, 2019
@nashif nashif merged commit 7f26009 into zephyrproject-rtos:master May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ARM ARM (32-bit) Architecture Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants