summaryrefslogtreecommitdiff
path: root/src/cpu/intel
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2019-02-11 10:12:35 +0200
committerPatrick Georgi <pgeorgi@google.com>2019-02-13 13:01:54 +0000
commit1a8387eabadb16adae2b5a032611813587957ed5 (patch)
tree66e718b42afbf3a44f34059458d274cb28912c04 /src/cpu/intel
parent61e18ebdf1d1f16af79bcdd735a26a5a9aedee69 (diff)
downloadcoreboot-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>
Diffstat (limited to 'src/cpu/intel')
-rw-r--r--src/cpu/intel/common/fsb.c10
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)