[U-Boot] [PATCH] common/board_f.c: fix compile error when tracing disabled

[U-Boot] [PATCH] common/board_f.c: fix compile error when tracing disabled

[Kevin Hilman]

From: Kevin Hilman <khilman@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 <sjg@chromium.org>
Cc: Tom Rini <trini@ti.com>
Signed-off-by: Kevin Hilman <khilman@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()? */
-- 
2.1.3

[U-Boot] [PATCH] common/board_f.c: fix compile error when tracing disabled

[Simon Glass]

Hi Kevin,

On 9 December 2014 at 16:03, Kevin Hilman <khilman@kernel.org> wrote:
> From: Kevin Hilman <khilman@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 <sjg@chromium.org>
> Cc: Tom Rini <trini@ti.com>
> Signed-off-by: Kevin Hilman <khilman@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] common/board_f.c: fix compile error when tracing disabled

[Tom Rini]

On Tue, Dec 09, 2014 at 03:03:58PM -0800, Kevin Hilman wrote:

> From: Kevin Hilman <khilman@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 <sjg@chromium.org>
> Cc: Tom Rini <trini@ti.com>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141229/2d01eaf8/attachment.pgp>