Skip to content

Commit

Permalink
RTOS: use default boot-stack-size for nRF as SoftDevice is no longer …
Browse files Browse the repository at this point in the history
…used

Previously we overrode nRF targets to have a larger stack
due to memory required by SoftDevice. Having deprecated SoftDevice
in favour of Cordio for BLE (ARMmbed#12674), such requirement does not
apply anymore.
  • Loading branch information
LDong-Arm committed Apr 27, 2020
1 parent 28ef753 commit 81e676b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 16 deletions.
5 changes: 1 addition & 4 deletions TESTS/mbed_hal/stack_size_unification/main.cpp
Expand Up @@ -31,10 +31,7 @@ extern osThreadAttr_t _main_thread_attr;
#endif
extern uint32_t mbed_stack_isr_size;

/* Exception for Nordic boards - BLE requires 2KB ISR stack. */
#if defined(TARGET_NRF5x)
#define EXPECTED_ISR_STACK_SIZE (2048)
#elif !defined(MBED_CONF_RTOS_PRESENT)
#if !defined(MBED_CONF_RTOS_PRESENT)
#define EXPECTED_ISR_STACK_SIZE (4096)
#else
#define EXPECTED_ISR_STACK_SIZE (1024)
Expand Down
12 changes: 0 additions & 12 deletions rtos/source/TARGET_CORTEX/mbed_lib.json
Expand Up @@ -76,18 +76,6 @@
"STM32F072RB": {
"main-thread-stack-size": 3072
},
"MCU_NRF51": {
"target.boot-stack-size": "0x800"
},
"MCU_NRF52840": {
"target.boot-stack-size": "0x800"
},
"MCU_NRF52832": {
"target.boot-stack-size": "0x800"
},
"MCU_NRF51_UNIFIED": {
"target.boot-stack-size": "0x800"
},
"NUVOTON": {
"idle-thread-stack-size-debug-extra": 512
}
Expand Down

0 comments on commit 81e676b

Please sign in to comment.