Re: [PATCH V4 10/12] trace: Add osnoise tracer

From: Steven Rostedt
Date: Fri Jun 18 2021 - 14:27:39 EST


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.

-- Steve