On 6/18/21 8:27 PM, Steven Rostedt wrote: > On Tue, 15 Jun 2021 11:28:49 +0200 > Daniel Bristot de Oliveira <bristot@xxxxxxxxxx> wrote: > >> diff --git a/kernel/trace/trace_osnoise.h b/kernel/trace/trace_osnoise.h >> new file mode 100644 >> index 000000000000..34c7b80f9e12 >> --- /dev/null >> +++ b/kernel/trace/trace_osnoise.h >> @@ -0,0 +1,9 @@ >> +int __weak osnoise_arch_register(void) >> +{ >> + return 0; >> +} >> + >> +void __weak osnoise_arch_unregister(void) >> +{ >> + return; >> +} > No reason to put the above in a header file. In fact, as it is code, it > should not be in a header file. Keep it in the trace_osnoise.c. Ack! -- Daniel > -- Steve >