Hi Kevin,
On 9 December 2014 at 16:03, Kevin Hilman <khil...@kernel.org> wrote:
> From: Kevin Hilman <khil...@linaro.org>
>
> When CONFIG_TRACE is disabled, linking fails with:
>
> common/built-in.o:(.data.init_sequence_f+0x8): undefined reference to
> `trace_early_init'
>
> To fix, wrap the call to trace_early_init() with #ifdef CONFIG_TRACE.
>
> Cc: Simon Glass <s...@chromium.org>
> Cc: Tom Rini <tr...@ti.com>
> Signed-off-by: Kevin Hilman <khil...@linaro.org>
> ---
> Applies to v2015.01-rc3
>
> common/board_f.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/common/board_f.c b/common/board_f.c
> index 98c9c728ce73..cfd77f865361 100644
> --- a/common/board_f.c
> +++ b/common/board_f.c
> @@ -813,7 +813,9 @@ static init_fnc_t init_sequence_f[] = {
> #endif
> setup_mon_len,
> setup_fdt,
> +#ifdef CONFIG_TRACE
> trace_early_init,
> +#endif
> initf_malloc,
> #if defined(CONFIG_MPC85xx) || defined(CONFIG_MPC86xx)
> /* TODO: can this go into arch_cpu_init()? */
Thanks - can you please also try removing the static inline stuff for
this in trace.h? It doesn't work.
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
