diff options
author | Subrata Banik <subrata.banik@intel.com> | 2018-12-10 14:11:35 +0530 |
---|---|---|
committer | Subrata Banik <subrata.banik@intel.com> | 2018-12-13 04:32:57 +0000 |
commit | 53b08c347f54970cf5f6ce35b8e1a1fe5f8a11f9 (patch) | |
tree | b4eed19fbf1b9f35dfbdddcc43fc84469176ec59 /src/arch/x86/mpspec.c | |
parent | e3110b862071ae0797afb0688495315a7dd47739 (diff) |
cpuid: Add helper function for cpuid(1) functions
This patch introduces 3 helper function for cpuid(1) :
1. cpu_get_cpuid() -> to get processor id (from cpuid.eax)
2. cpu_get_feature_flags_ecx -> to get processor feature flag (from cpuid.ecx)
3. cpu_get_feature_flags_edx -> to get processor feature flag (from cpuid.edx)
Above 3 helper functions are targeted to replace majority of cpuid(1)
references.
Change-Id: Ib96a7c79dadb1feff0b8d58aa408b355fbb3bc50
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/c/30123
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Diffstat (limited to 'src/arch/x86/mpspec.c')
-rw-r--r-- | src/arch/x86/mpspec.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/arch/x86/mpspec.c b/src/arch/x86/mpspec.c index b44662d38b..28a9080e2e 100644 --- a/src/arch/x86/mpspec.c +++ b/src/arch/x86/mpspec.c @@ -161,14 +161,12 @@ void smp_write_processors(struct mp_config_table *mc) unsigned int apic_version; unsigned int cpu_features; unsigned int cpu_feature_flags; - struct cpuid_result result; struct device *cpu; boot_apic_id = lapicid(); apic_version = lapic_read(LAPIC_LVR) & 0xff; - result = cpuid(1); - cpu_features = result.eax; - cpu_feature_flags = result.edx; + cpu_features = cpu_get_cpuid(); + cpu_feature_flags = cpu_get_feature_flags_edx(); /* order the output of the cpus to fix a bug in kernel 2.6.11 */ for (order_id = 0; order_id < 256; order_id++) { for (cpu = all_devices; cpu; cpu = cpu->next) { |