From 5742a508a299ea4ce65c1c675543b23b77c426d7 Mon Sep 17 00:00:00 2001 From: Kumar Gala Date: Fri, 21 Apr 2017 11:41:26 -0500 Subject: [PATCH] kernel: cleanup use of naked unsigned in _new_thread There are a few places that we used an naked unsigned type, lets be explicit and make it 'unsigned int'. Change-Id: I33fcbdec4a6a1c0b1a2defb9a5844d282d02d80e Signed-off-by: Kumar Gala --- arch/arc/core/thread.c | 2 +- arch/arm/core/thread.c | 2 +- arch/nios2/core/thread.c | 2 +- arch/riscv32/core/thread.c | 2 +- arch/x86/core/thread.c | 6 +++--- kernel/include/nano_internal.h | 2 +- 6 files changed, 8 insertions(+), 8 deletions(-) diff --git a/arch/arc/core/thread.c b/arch/arc/core/thread.c index ae849e5f1e3..c0e1419f3b7 100644 --- a/arch/arc/core/thread.c +++ b/arch/arc/core/thread.c @@ -76,7 +76,7 @@ static ALWAYS_INLINE void thread_monitor_init(struct k_thread *thread) void _new_thread(char *pStackMem, size_t stackSize, _thread_entry_t pEntry, void *parameter1, void *parameter2, void *parameter3, - int priority, unsigned options) + int priority, unsigned int options) { _ASSERT_VALID_PRIO(priority, pEntry); diff --git a/arch/arm/core/thread.c b/arch/arm/core/thread.c index 090155aa63f..a64896d48fe 100644 --- a/arch/arm/core/thread.c +++ b/arch/arm/core/thread.c @@ -71,7 +71,7 @@ static ALWAYS_INLINE void thread_monitor_init(struct k_thread *thread) void _new_thread(char *pStackMem, size_t stackSize, _thread_entry_t pEntry, void *parameter1, void *parameter2, void *parameter3, - int priority, unsigned options) + int priority, unsigned int options) { _ASSERT_VALID_PRIO(priority, pEntry); diff --git a/arch/nios2/core/thread.c b/arch/nios2/core/thread.c index b259fa6c586..cd011242edb 100644 --- a/arch/nios2/core/thread.c +++ b/arch/nios2/core/thread.c @@ -53,7 +53,7 @@ struct init_stack_frame { void _new_thread(char *stack_memory, size_t stack_size, _thread_entry_t thread_func, void *arg1, void *arg2, void *arg3, - int priority, unsigned options) + int priority, unsigned int options) { _ASSERT_VALID_PRIO(priority, thread_func); diff --git a/arch/riscv32/core/thread.c b/arch/riscv32/core/thread.c index cbc1b266e57..d209e172f81 100644 --- a/arch/riscv32/core/thread.c +++ b/arch/riscv32/core/thread.c @@ -37,7 +37,7 @@ void _thread_entry_wrapper(_thread_entry_t thread, void _new_thread(char *stack_memory, size_t stack_size, _thread_entry_t thread_func, void *arg1, void *arg2, void *arg3, - int priority, unsigned options) + int priority, unsigned int options) { _ASSERT_VALID_PRIO(priority, thread_func); diff --git a/arch/x86/core/thread.c b/arch/x86/core/thread.c index ff4ebeaf1a5..66abe031edc 100644 --- a/arch/x86/core/thread.c +++ b/arch/x86/core/thread.c @@ -65,9 +65,9 @@ static ALWAYS_INLINE void thread_monitor_init(struct k_thread *thread) * * @return N/A */ -static void _new_thread_internal(char *pStackMem, unsigned stackSize, +static void _new_thread_internal(char *pStackMem, unsigned int stackSize, int priority, - unsigned options) + unsigned int options) { unsigned long *pInitialCtx; /* ptr to the new task's k_thread */ @@ -230,7 +230,7 @@ __asm__("\t.globl _thread_entry\n" void _new_thread(char *pStackMem, size_t stackSize, _thread_entry_t pEntry, void *parameter1, void *parameter2, void *parameter3, - int priority, unsigned options) + int priority, unsigned int options) { _ASSERT_VALID_PRIO(priority, pEntry); diff --git a/kernel/include/nano_internal.h b/kernel/include/nano_internal.h index 7e7d9baf3e5..42f518cfd1c 100644 --- a/kernel/include/nano_internal.h +++ b/kernel/include/nano_internal.h @@ -48,7 +48,7 @@ extern void _thread_entry(void (*)(void *, void *, void *), extern void _new_thread(char *pStack, size_t stackSize, void (*pEntry)(void *, void *, void *), void *p1, void *p2, void *p3, - int prio, unsigned options); + int prio, unsigned int options); /* context switching and scheduling-related routines */