> On 25 Jan 2018, at 18:37, Lukasz Majewski <lu...@denx.de> wrote:
>
> From: Anatolij Gustschin <ag...@denx.de>
>
> Selecting this option will reduce SPL boot time by approx. 6 ms
> (e. g. with 70 bytes long banner string at 115200 baud).
>
> Signed-off-by: Anatolij Gustschin <ag...@denx.de>
> Tested-by: Lukasz Majewski <lu...@denx.de>
Reviewed-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com>
See below for a nitpick.
> ---
>
> common/spl/Kconfig | 7 +++++++
> common/spl/spl.c | 2 ++
> 2 files changed, 9 insertions(+)
>
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index d686b1ecbd..d1f9cae6b7 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -118,6 +118,13 @@ config SPL_SEPARATE_BSS
> location is used. Normally we put the device tree at the end of BSS
> but with this option enabled, it goes at _image_binary_end.
>
> +config SPL_DISABLE_BANNER_PRINT
> + bool "Disable output of the SPL banner 'U-Boot SPL ...'"
> + help
> + If this option is enabled, SPL will not print the banner with version
> + info. Selecting this option could be useful to reduce SPL boot time
> + (e.g. approx. 6 ms slower, when output on i.MX6 with 1152000 baud).
You seem to be off by “times 10” for the baud-rate.
> +
> config SPL_DISPLAY_PRINT
> bool "Display a board-specific message in SPL"
> help
> diff --git a/common/spl/spl.c b/common/spl/spl.c
> index 76c1963611..898c416a66 100644
> --- a/common/spl/spl.c
> +++ b/common/spl/spl.c
> @@ -477,8 +477,10 @@ void preloader_console_init(void)
>
> gd->have_console = 1;
>
> +#ifndef CONFIG_SPL_DISABLE_BANNER_PRINT
> puts("\nU-Boot SPL " PLAIN_VERSION " (" U_BOOT_DATE " - " \
> U_BOOT_TIME ")\n");
> +#endif
> #ifdef CONFIG_SPL_DISPLAY_PRINT
> spl_display_print();
> #endif
> --
> 2.11.0
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
