[U-Boot] [PATCH v4 05/66] rockchip: back-to-bootrom: simplify the #ifdef-check for LIBGENERIC in TPL/SPL

[Andy Yan]

Hi Philipp：

2017-08-03 4:34 GMT+08:00 Philipp Tomsich <philipp.tomsich at theobroma-
systems.com>:

> With the finer-grained control over LIBGENERIC_SUPPORT for TPL/SPL (i.e.
> with the newly introduced distinction between TPL_LIBGENERIC_SUPPORT and
> SPL_LIBGENERIC_SUPPORT), we can simplify the #ifdef-check to simply use
> CONFIG_IS_ENABELD.
>
> Signed-off-by: Philipp Tomsich <philipp.tomsich at theobroma-systems.com>
>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> Reviewed-by: Tom Rini <trini at konsulko.com>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2: None
>
>  arch/arm/mach-rockchip/bootrom.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-rockchip/bootrom.c
> b/arch/arm/mach-rockchip/bootrom.c
> index 4ca9962..4d38ed6 100644
> --- a/arch/arm/mach-rockchip/bootrom.c
> +++ b/arch/arm/mach-rockchip/bootrom.c
> @@ -9,8 +9,8 @@
>
>  void back_to_bootrom(void)
>  {
> -#if defined(CONFIG_SPL_LIBCOMMON_SUPPORT) && !defined(CONFIG_TPL_BUILD)
> -       puts("Returning to boot ROM...");
> +#if CONFIG_IS_ENABLED(LIBGENERIC_SUPPORT)
> +       printf("Returning to boot ROM...\n");
>




We need to use puts()  instead of printf() here， or the code size will be
increased to much for the sram limited platform， such as rk3036、rv1108.

And also should be CONFIG_IS_ENABLED(LIBOMMON_SUPPORT), or it will
introduce compile error for kylin-rk3036.





>  #endif
>         _back_to_bootrom_s();
>  }
> --
> 2.1.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
>