Skip to content

Commit 4f161d2

Browse files
committed
fix(hal.cpu) compiler warn about printf-format for pointers >
warning was: ``` framework-arduinoespressif32/cores/esp32/esp32-hal-cpu.c:132:9: note: in expansion of macro 'log_e' log_e("not found func=%08X arg=%08X",cb,arg); ^~~~~ framework-arduinoespressif32/tools/sdk/esp32/include/log/include/esp_log.h:276:27: warning: format '%X' expects argument of type 'unsigned int', but argument 7 has type 'void *' [-Wformat=] ```
1 parent 0b3f1a9 commit 4f161d2

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

cores/esp32/esp32-hal-cpu.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ bool addApbChangeCallback(void * arg, apb_change_cb_t cb){
107107
// look for duplicate callbacks
108108
while( (r != NULL ) && !((r->cb == cb) && ( r->arg == arg))) r = r->next;
109109
if (r) {
110-
log_e("duplicate func=%08X arg=%08X",c->cb,c->arg);
110+
log_e("duplicate func=%8p arg=%8p",c->cb,c->arg);
111111
free(c);
112112
xSemaphoreGive(apb_change_lock);
113113
return false;
@@ -129,7 +129,7 @@ bool removeApbChangeCallback(void * arg, apb_change_cb_t cb){
129129
// look for matching callback
130130
while( (r != NULL ) && !((r->cb == cb) && ( r->arg == arg))) r = r->next;
131131
if ( r == NULL ) {
132-
log_e("not found func=%08X arg=%08X",cb,arg);
132+
log_e("not found func=%8p arg=%8p",cb,arg);
133133
xSemaphoreGive(apb_change_lock);
134134
return false;
135135
}

0 commit comments

Comments
 (0)