From d57e03c94c76a0bd0cdba9b8486d370dc772a753 Mon Sep 17 00:00:00 2001 From: Benjamin Valentin Date: Sun, 8 Dec 2019 22:07:57 +0100 Subject: [PATCH] cpu/lpc2387: use the same default stack sizes as cortexm_common Both architectures are variants of the ARM architecture and use the same toolchain. There is no reason to have such wildly different defaults. This results in some tests passing that would crash before: - [x] `tests/pkg_libcose` - [x] `tests/pkg_qdsa` - [x] `tests/pkg_relic` - [x] `tests/pkg_tweetnacl` - [x] `tests/pthread_tls` `THREAD_EXTRA_STACKSIZE_PRINTF_FLOAT` is not used anywhere in RIOT anymore, so just drop it. --- cpu/lpc2387/include/cpu_conf.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/cpu/lpc2387/include/cpu_conf.h b/cpu/lpc2387/include/cpu_conf.h index b7a385ec25..03daad9037 100644 --- a/cpu/lpc2387/include/cpu_conf.h +++ b/cpu/lpc2387/include/cpu_conf.h @@ -42,11 +42,10 @@ extern "C" { * @name Kernel configuration * @{ */ -#define THREAD_EXTRA_STACKSIZE_PRINTF_FLOAT (4096) -#define THREAD_EXTRA_STACKSIZE_PRINTF (2048) +#define THREAD_EXTRA_STACKSIZE_PRINTF (512) #ifndef THREAD_STACKSIZE_DEFAULT -#define THREAD_STACKSIZE_DEFAULT (512) +#define THREAD_STACKSIZE_DEFAULT (1024) #endif #define THREAD_STACKSIZE_IDLE (160)