Skip to content

Commit b3be1b5

Browse files
committed
tcpip_adapter: make log output at debug level less noisy
At debug log level, tcpip_adapter would print logs for each function call scheduled onto the tcpip task. These logs contained pointers which idf_monitor decoded, adding even more noise and useless vertical space in logs. This change moves these log statements to verbose level.
1 parent a5f9563 commit b3be1b5

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

components/tcpip_adapter/include/tcpip_adapter.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,10 +184,10 @@ typedef struct tcpip_adapter_dns_param_s {
184184
msg.data = (void*)(_data);\
185185
msg.api_fn = (_fn);\
186186
if (TCPIP_ADAPTER_IPC_REMOTE == tcpip_adapter_ipc_check(&msg)) {\
187-
ESP_LOGD(TAG, "check: remote, if=%d fn=%p\n", (_if), (_fn));\
187+
ESP_LOGV(TAG, "check: remote, if=%d fn=%p\n", (_if), (_fn));\
188188
return msg.ret;\
189189
} else {\
190-
ESP_LOGD(TAG, "check: local, if=%d fn=%p\n", (_if), (_fn));\
190+
ESP_LOGV(TAG, "check: local, if=%d fn=%p\n", (_if), (_fn));\
191191
}\
192192
}while(0)
193193

components/tcpip_adapter/tcpip_adapter_lwip.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ static sys_sem_t api_sync_sem = NULL;
6767
static bool tcpip_inited = false;
6868
static sys_sem_t api_lock_sem = NULL;
6969
extern sys_thread_t g_lwip_task;
70-
#define TAG "tcpip_adapter"
70+
static const char* TAG = "tcpip_adapter";
7171

7272
static void tcpip_adapter_api_cb(void* api_msg)
7373
{
@@ -79,7 +79,7 @@ static void tcpip_adapter_api_cb(void* api_msg)
7979
}
8080

8181
msg->ret = msg->api_fn(msg);
82-
ESP_LOGD(TAG, "call api in lwip: ret=0x%x, give sem", msg->ret);
82+
ESP_LOGV(TAG, "call api in lwip: ret=0x%x, give sem", msg->ret);
8383
sys_sem_signal(&api_sync_sem);
8484

8585
return;

0 commit comments

Comments
 (0)