[coreboot-gerrit] New patch to review for coreboot: Set the fsb timer correctly for Netburst CPUs

[Arthur Heymans]

Arthur Heymans ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17832

-gerrit

commit dcd3366135272ad9a1d589abadddb4ddc0ea989a
Author: Arthur Heymans 
Date:   Tue Dec 13 15:21:24 2016 +0100

Set the fsb timer correctly for Netburst CPUs

On Netburst (Pentium 4) the fsb cannot be read from MSR_FSB_FREQ (msr 0xcd)
    but one has to use msr 0x2c instead.

Change-Id: I0beccba2e4a8ec5cd23537b2207f9c49a040fd73
    Signed-off-by: Arthur Heymans 
---
 src/cpu/x86/lapic/apic_timer.c | 48 ++++++++++++++++++++++++------------------
 1 file changed, 27 insertions(+), 21 deletions(-)

diff --git a/src/cpu/x86/lapic/apic_timer.c b/src/cpu/x86/lapic/apic_timer.c
index 1930ec4..f22fbfa 100644
--- a/src/cpu/x86/lapic/apic_timer.c
+++ b/src/cpu/x86/lapic/apic_timer.c
@@ -47,31 +47,37 @@ static int set_timer_fsb(void)
    struct cpuinfo_x86 c;
    int core_fsb[8] = { -1, 133, -1, 166, -1, 100, -1, -1 };
    int core2_fsb[8] = { 266, 133, 200, 166, 333, 100, -1, -1 };
+	int netburst_fsb[4] = { 100, 133, 200, -1 };

get_fms(&c, cpuid_eax(1));
-	if (c.x86 != 6)
+	if (c.x86 == 6) {
+
+		switch (c.x86_model) {
+		case 0xe:  /* Core Solo/Duo */
+		case 0x1c: /* Atom */
+			car_set_var(g_timer_fsb,
+				core_fsb[rdmsr(MSR_FSB_FREQ).lo & 7]);
+			break;
+		case 0xf:  /* Core 2 or Xeon */
+		case 0x17: /* Enhanced Core */
+			car_set_var(g_timer_fsb,
+				core2_fsb[rdmsr(MSR_FSB_FREQ).lo & 7]);
+			break;
+		case 0x2a: /* SandyBridge BCLK fixed at 100MHz*/
+		case 0x3a: /* IvyBridge BCLK fixed at 100MHz*/
+		case 0x3c: /* Haswell BCLK fixed at 100MHz */
+		case 0x45: /* Haswell-ULT BCLK fixed at 100MHz */
+			car_set_var(g_timer_fsb, 100);
+			break;
+		default:
+			car_set_var(g_timer_fsb, 200);
+			break;
+		}
+	} else if (c.x86 == 0xf) /* Netburst */
+		car_set_var(g_timer_fsb, netburst_fsb[rdmsr(0x2c).lo & 3]);
+	else
    	return -1;

-	switch (c.x86_model) {
-	case 0xe:  /* Core Solo/Duo */
-	case 0x1c: /* Atom */
-		car_set_var(g_timer_fsb, core_fsb[rdmsr(MSR_FSB_FREQ).lo & 7]);
-		break;
-	case 0xf:  /* Core 2 or Xeon */
-	case 0x17: /* Enhanced Core */
-		car_set_var(g_timer_fsb, core2_fsb[rdmsr(MSR_FSB_FREQ).lo & 7]);
-		break;
-	case 0x2a: /* SandyBridge BCLK fixed at 100MHz*/
-	case 0x3a: /* IvyBridge BCLK fixed at 100MHz*/
-	case 0x3c: /* Haswell BCLK fixed at 100MHz */
-	case 0x45: /* Haswell-ULT BCLK fixed at 100MHz */
-		car_set_var(g_timer_fsb, 100);
-		break;
-	default:
-		car_set_var(g_timer_fsb, 200);
-		break;
-	}
-
    return 0;
 }