diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2019-02-11 10:12:35 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2019-02-13 13:01:54 +0000 |
| commit | 1a8387eabadb16adae2b5a032611813587957ed5 (patch) | |
| tree | 66e718b42afbf3a44f34059458d274cb28912c04 | |
| parent | 61e18ebdf1d1f16af79bcdd735a26a5a9aedee69 (diff) | |
| download | coreboot-1a8387eabadb16adae2b5a032611813587957ed5.tar.xz | |
cpu/intel/common: Split get_ia32_fsb()
It is desireable to not have printk() inside a
function body that can be used for udelay().
This avoids potential infinite recursion.
Change-Id: Ie67fc2a8da8351f22794e4d36c55b887c298e8ca
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/31338
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
| -rw-r--r-- | src/cpu/intel/common/fsb.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/cpu/intel/common/fsb.c b/src/cpu/intel/common/fsb.c index 1f7c391a8f..92967859dd 100644 --- a/src/cpu/intel/common/fsb.c +++ b/src/cpu/intel/common/fsb.c @@ -18,7 +18,7 @@ #include <console/console.h> #include <commonlib/helpers.h> -int get_ia32_fsb(void) +static int get_fsb(void) { struct cpuinfo_x86 c; static const short core_fsb[8] = { -1, 133, -1, 166, -1, 100, -1, -1 }; @@ -60,6 +60,14 @@ int get_ia32_fsb(void) break; } } + return ret; +} + +int get_ia32_fsb(void) +{ + int ret; + + ret = get_fsb(); if (ret == -1) printk(BIOS_ERR, "FSB not found\n"); if (ret == -2) |