[U-Boot] [PATCH v3] arm: printf() is not available in some SPL configurations
Andreas Bießmann
andreas.devel at googlemail.com
Tue Nov 22 09:49:46 CET 2011
Dear Christian,
sorry I haven't followed the discussion so far. I have a little pointer too.
Am 22.11.2011 09:26, schrieb Christian Riesch:
> This patch avoids build breakage for SPLs that do not support printf.
>
> Signed-off-by: Christian Riesch <christian.riesch at omicron.at>
> Cc: Wolfgang Denk <wd at denx.de>
> Cc: Tom Rini <trini at ti.com>
> Acked-by: Tom Rini <trini at ti.com>
> ---
> Changes for v3:
> - Removed extra white space
> - Separated patch from patchset [1]
>
> [1] http://lists.denx.de/pipermail/u-boot/2011-November/110635.html
>
> arch/arm/lib/eabi_compat.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/lib/eabi_compat.c b/arch/arm/lib/eabi_compat.c
> index eb3e26d..e1b87be 100644
> --- a/arch/arm/lib/eabi_compat.c
> +++ b/arch/arm/lib/eabi_compat.c
> @@ -13,7 +13,9 @@
>
> int raise (int signum)
> {
> +#if !defined(CONFIG_SPL_BUILD) || defined(CONFIG_SPL_LIBCOMMON_SUPPORT)
> printf("raise: Signal # %d caught\n", signum);
> +#endif
Well, you can do it that way but I guess there are several places where
printf() will be needed by some drivers in SPL. Therefore I think it is
better to provide printf() for SPL than to ifdef out all the printf's.
So how about providing an empty
#if !defined(CONFIG_SPL_BUILD) || defined(CONFIG_SPL_LIBCOMMON_SUPPORT)
#define printf(x)
#endif
somewhere in the SPL code?
best regards
Andreas Bießmann
More information about the U-Boot
mailing list