[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge: Move CPU report to cpu folder

[Patrick Rudolph (Code Review)]

Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/27666


Change subject: nb/intel/sandybridge: Move CPU report to cpu folder
......................................................................

nb/intel/sandybridge: Move CPU report to cpu folder

Change-Id: Ie973923b90eca0bfabd474fed85a6cc33fce7e19
Signed-off-by: Patrick Rudolph <patrick.rudolph at 9elements.com>
---
M src/cpu/intel/model_206ax/model_206ax_init.c
M src/northbridge/intel/sandybridge/report_platform.c
2 files changed, 27 insertions(+), 46 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/66/27666/1

diff --git a/src/cpu/intel/model_206ax/model_206ax_init.c b/src/cpu/intel/model_206ax/model_206ax_init.c
index 4be3d87..75631c1 100644
--- a/src/cpu/intel/model_206ax/model_206ax_init.c
+++ b/src/cpu/intel/model_206ax/model_206ax_init.c
@@ -498,9 +498,33 @@
 	}
 }
 
+static void model_206ax_report(void)
+{
+	static const char *const mode[] = {"NOT ", ""};
+	struct cpuid_result cpuidr;
+	char processor_name[49];
+	int vt, txt, aes;
+
+	/* Print processor name */
+	fill_processor_name(processor_name);
+	printk(BIOS_INFO, "CPU: %s.\n", processor_name);
+
+	/* Print platform ID */
+	printk(BIOS_INFO, "CPU: platform id %x\n", get_platform_id());
+
+	/* CPUID and features */
+	cpuidr = cpuid(1);
+	printk(BIOS_INFO, "CPU: cpuid(1) 0x%x\n", cpuidr.eax);
+	aes = (cpuidr.ecx & (1 << 25)) ? 1 : 0;
+	txt = (cpuidr.ecx & (1 << 6)) ? 1 : 0;
+	vt = (cpuidr.ecx & (1 << 5)) ? 1 : 0;
+	printk(BIOS_INFO, "CPU: AES %ssupported\n", mode[aes]);
+	printk(BIOS_INFO, "CPU: TXT %ssupported\n", mode[txt]);
+	printk(BIOS_INFO, "CPU: VT %ssupported\n", mode[vt]);
+}
+
 static void model_206ax_init(struct device *cpu)
 {
-	char processor_name[49];
 
 	/* Turn on caching if we haven't already */
 	x86_enable_cache();
@@ -510,12 +534,8 @@
 	/* Clear out pending MCEs */
 	configure_mca();
 
-	/* Print processor name */
-	fill_processor_name(processor_name);
-	printk(BIOS_INFO, "CPU: %s.\n", processor_name);
-
-	/* Print platform ID */
-	printk(BIOS_INFO, "CPU: platform id %x\n", get_platform_id());
+	/* Print infos */
+	model_206ax_report();
 
 	/* Setup MTRRs based on physical address size */
 	x86_setup_mtrrs_with_detect();
diff --git a/src/northbridge/intel/sandybridge/report_platform.c b/src/northbridge/intel/sandybridge/report_platform.c
index 6dd760d..7e647d8 100644
--- a/src/northbridge/intel/sandybridge/report_platform.c
+++ b/src/northbridge/intel/sandybridge/report_platform.c
@@ -14,47 +14,9 @@
  */
 
 #include <console/console.h>
-#include <arch/cpu.h>
-#include <string.h>
 #include <southbridge/intel/bd82x6x/pch.h>
-#include <arch/io.h>
 #include "sandybridge.h"
 
-static void report_cpu_info(void)
-{
-	struct cpuid_result cpuidr;
-	u32 i, index;
-	char cpu_string[50], *cpu_name = cpu_string; /* 48 bytes are reported */
-	int vt, txt, aes;
-	const char *mode[] = {"NOT ", ""};
-
-	index = 0x80000000;
-	cpuidr = cpuid(index);
-	if (cpuidr.eax < 0x80000004) {
-		strcpy(cpu_string, "Platform info not available");
-	} else {
-		u32 *p = (u32*) cpu_string;
-		for (i = 2; i <= 4; i++) {
-			cpuidr = cpuid(index + i);
-			*p++ = cpuidr.eax;
-			*p++ = cpuidr.ebx;
-			*p++ = cpuidr.ecx;
-			*p++ = cpuidr.edx;
-		}
-	}
-	/* Skip leading spaces in CPU name string */
-	while (cpu_name[0] == ' ')
-		cpu_name++;
-
-	cpuidr = cpuid(1);
-	printk(BIOS_DEBUG, "CPU id(%x): %s\n", cpuidr.eax, cpu_name);
-	aes = (cpuidr.ecx & (1 << 25)) ? 1 : 0;
-	txt = (cpuidr.ecx & (1 << 6)) ? 1 : 0;
-	vt = (cpuidr.ecx & (1 << 5)) ? 1 : 0;
-	printk(BIOS_DEBUG, "AES %ssupported, TXT %ssupported, VT %ssupported\n",
-	       mode[aes], mode[txt], mode[vt]);
-}
-
 static struct {
 	u16 dev_id;
 	const char *dev_name;
@@ -125,6 +87,5 @@
 
 void report_platform_info(void)
 {
-	report_cpu_info();
 	report_pch_info();
 }

-- 
To view, visit https://review.coreboot.org/27666
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie973923b90eca0bfabd474fed85a6cc33fce7e19
Gerrit-Change-Number: 27666
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph <patrick.rudolph at 9elements.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180727/3f4dcf60/attachment.html>