summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorSuriyan Ramasami <suriyan.r@gmail.com>2015-08-18 09:25:33 -0700
committerTom Rini <trini@konsulko.com>2015-08-28 12:32:36 -0400
commite9015b304ad2a39a03f2684dadeabef474b7addb (patch)
tree5446c7248bea3a9f47a39afc86877749b670ddc8 /lib
parentb3537c08e14b9d7e3b3207de2d280dab2dc499fa (diff)
lib/display_options: Fix print_freq
Build without CONFIG_SPL_SERIAL_SUPPORT does not print the cpu freq. I have seen this in the odroid U3 board, where on boot one sees this: CPU: Exynos4412 @ GHz instead of: CPU: Exynos4412 @ 1 GHz I am assuming that this change was done to get rid of compiler warnings related to unused variables when building with CONFIG_SPL_SERIAL_SUPPORT not being defined in an SPL build. Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com> Acked-by: Simon Glass <sjg@chromium.org> Acked-by: Heiko Schocher <hs@denx.de> Reviewed-by: Joe Hershberger <joe.hershberger@ni.com>
Diffstat (limited to 'lib')
-rw-r--r--lib/display_options.c10
1 files changed, 1 insertions, 9 deletions
diff --git a/lib/display_options.c b/lib/display_options.c
index df134cd23a9..83ea4decb53 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);