Hello Suriyan,

Am 25.08.2015 um 07:59 schrieb Suriyan Ramasami:

Build without CONFIG_SPL_SERIAL_SUPPORT does not print the cpu freq. Booting an odroid U3 board, one sees this: CPU: Exynos4412 @ GHz instead of: CPU: Exynos4412 @ 1 GHz

This change was done to get rid of compiler warnings related to the unused variable 'n' when CONFIG_SPL_SERIAL_SUPPORT is not defined in an SPL build. Example board: smartweb

Signed-off-by: Suriyan Ramasami suriyan.r@gmail.com

v2:

• Get rid of variable 'n' altogether

v1:

• Remove #ifdef CONFIG_SPL_SERIAL_SUPPORT

  lib/display_options.c | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-)

Thanks!

Acked-by: Heiko Schocherhs@denx.de

bye, Heiko

diff --git a/lib/display_options.c b/lib/display_options.c index df134cd..83ea4de 100644 --- a/lib/display_options.c +++ b/lib/display_options.c @@ -26,9 +26,6 @@ int display_options (void) void print_freq(uint64_t freq, const char *s) { unsigned long m = 0; -#if defined(CONFIG_SPL_SERIAL_SUPPORT)

• unsigned long n;

-#endif uint32_t f; static const char names[] = {'G', 'M', 'K'}; unsigned long d = 1e9; @@ -48,9 +45,6 @@ void print_freq(uint64_t freq, const char *s) }

f = do_div(freq, d); -#if defined(CONFIG_SPL_SERIAL_SUPPORT)

• n = freq;

-#endif

/* If there's a remainder, show the first few digits */ if (f) { @@ -63,9 +57,7 @@ void print_freq(uint64_t freq, const char *s) m = (m / 10) + (m % 100 >= 50); }

-#if defined(CONFIG_SPL_SERIAL_SUPPORT)

• printf("%lu", n);

-#endif

• printf("%lu", (unsigned long) freq); if (m) printf(".%ld", m); printf(" %cHz%s", c, s);