Skip to content

Commit 7ee2d2a

Browse files
committed
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 <[email protected]>
1 parent 03600f6 commit 7ee2d2a

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

targets/TARGET_Realtek/TARGET_AMEBA/sdk/os/os_dep/osdep_service.c

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ int RTW_STATUS_CODE(int error_code)
5353
u32 rtw_atoi(u8* s)
5454
{
5555
int num=0,flag=0;
56-
int i;
56+
size_t i;
5757

5858
for(i=0;i<=strlen((char *)s);i++)
5959
{
@@ -202,10 +202,11 @@ void add_mem_usage(_list *pmem_table, void *ptr, int size, int *used_num, int fl
202202
return;
203203
}
204204
else{
205-
if(flag == MEM_MONITOR_FLAG_WPAS)
205+
if(flag == MEM_MONITOR_FLAG_WPAS) {
206206
DBG_INFO("Alloc memory at %p with size of %d", ptr, size);
207-
else
207+
} else {
208208
DBG_INFO("Alloc memory at %p with size of %d", ptr, size);
209+
}
209210
}
210211
#if CONFIG_MEM_MONITOR & MEM_MONITOR_LEAK
211212
mem_entry = (struct mem_entry *) _rtw_malloc(sizeof(struct mem_entry));
@@ -1060,7 +1061,7 @@ static void worker_thread_main( void *arg )
10601061
message.function(message.buf, message.buf_len, message.flags, message.user_data);
10611062
if(message.buf){
10621063
//printf("\n!!!!!Free %p(%d)\n", message.buf, message.buf_len);
1063-
_rtw_mfree(message.buf, message.buf_len);
1064+
_rtw_mfree((u8 *)message.buf, message.buf_len);
10641065
}
10651066
}
10661067
}

0 commit comments

Comments
 (0)