From 7ee2d2a862f33562b06f15508f0f58d0af75a7a7 Mon Sep 17 00:00:00 2001 From: Tony Wu Date: Tue, 17 Apr 2018 10:50:36 +0800 Subject: [PATCH] rtl8195am - fix osdep_service.c warnings Fix the following warnings: [Warning] osdep_service.c@58,11: comparison between signed and unsigned integer expressions [-Wsign-compare] [Warning] osdep_service.c@208,61: suggest braces around empty body in an 'else' statement [-Wempty-body] [Warning] osdep_service.c@1063,16: pointer targets in passing argument 1 of '_rtw_mfree' differ in signedness [-Wpointer-sign] Signed-off-by: Tony Wu --- .../TARGET_AMEBA/sdk/os/os_dep/osdep_service.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/targets/TARGET_Realtek/TARGET_AMEBA/sdk/os/os_dep/osdep_service.c b/targets/TARGET_Realtek/TARGET_AMEBA/sdk/os/os_dep/osdep_service.c index 3bd79c2c8d..4b738e54d3 100644 --- a/targets/TARGET_Realtek/TARGET_AMEBA/sdk/os/os_dep/osdep_service.c +++ b/targets/TARGET_Realtek/TARGET_AMEBA/sdk/os/os_dep/osdep_service.c @@ -53,7 +53,7 @@ int RTW_STATUS_CODE(int error_code) u32 rtw_atoi(u8* s) { int num=0,flag=0; - int i; + size_t i; for(i=0;i<=strlen((char *)s);i++) { @@ -202,10 +202,11 @@ void add_mem_usage(_list *pmem_table, void *ptr, int size, int *used_num, int fl return; } else{ - if(flag == MEM_MONITOR_FLAG_WPAS) + if(flag == MEM_MONITOR_FLAG_WPAS) { DBG_INFO("Alloc memory at %p with size of %d", ptr, size); - else + } else { DBG_INFO("Alloc memory at %p with size of %d", ptr, size); + } } #if CONFIG_MEM_MONITOR & MEM_MONITOR_LEAK mem_entry = (struct mem_entry *) _rtw_malloc(sizeof(struct mem_entry)); @@ -1060,7 +1061,7 @@ static void worker_thread_main( void *arg ) message.function(message.buf, message.buf_len, message.flags, message.user_data); if(message.buf){ //printf("\n!!!!!Free %p(%d)\n", message.buf, message.buf_len); - _rtw_mfree(message.buf, message.buf_len); + _rtw_mfree((u8 *)message.buf, message.buf_len); } } }