[01/10] cpu/msr: remove EBL_CR_POWERON (2ah) from IA32_MSRs
diff mbox series

Message ID 20190711174635.770-2-alex.hung@canonical.com
State Accepted
Headers show
Series
  • Updates architectural MSRs in IA32_MSRs
Related show

Commit Message

Alex Hung July 11, 2019, 5:46 p.m. UTC
This MSR is not an "architectural MSR" in Table 2-2 in Intel's software
developer's manual

Signed-off-by: Alex Hung <alex.hung@canonical.com>
---
 src/cpu/msr/msr.c | 1 -
 1 file changed, 1 deletion(-)

Comments

ivanhu July 24, 2019, 7:20 a.m. UTC | #1
On 7/12/19 1:46 AM, Alex Hung wrote:
> This MSR is not an "architectural MSR" in Table 2-2 in Intel's software
> developer's manual
> 
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>   src/cpu/msr/msr.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/src/cpu/msr/msr.c b/src/cpu/msr/msr.c
> index 1030a264..a29e7cde 100644
> --- a/src/cpu/msr/msr.c
> +++ b/src/cpu/msr/msr.c
> @@ -343,7 +343,6 @@ static const msr_info IA32_MSRs[] = {
>   	{ "MONITOR_FILTER_SIZE",	0x00000006,	0xffffffffffffffffULL, NULL },
>   	//{ "TIME_STAMP_COUNTER",	0x00000010,	0xffffffffffffffffULL, NULL },
>   	{ "PLATFORM_ID",		0x00000017,	0x001c000000000000ULL, NULL },
> -	{ "EBL_CR_POWERON",		0x0000002a,	0xffffffffffffffffULL, NULL },
>   	{ "APIC_BASE",			0x0000001b,	0xfffffffffffffeffULL, NULL },
>   	{ "FEATURE_CONTROL",		0x0000003a,	0x000000000000ff07ULL, NULL },
>   	{ "BIOS_SIGN_ID",		0x0000008b,	0xffffffff00000000ULL, NULL },
> 

Acked-by: Ivan Hu <ivan.hu@canonical.com>
Colin King July 24, 2019, 3:03 p.m. UTC | #2
On 11/07/2019 18:46, Alex Hung wrote:
> This MSR is not an "architectural MSR" in Table 2-2 in Intel's software
> developer's manual
> 
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>  src/cpu/msr/msr.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/src/cpu/msr/msr.c b/src/cpu/msr/msr.c
> index 1030a264..a29e7cde 100644
> --- a/src/cpu/msr/msr.c
> +++ b/src/cpu/msr/msr.c
> @@ -343,7 +343,6 @@ static const msr_info IA32_MSRs[] = {
>  	{ "MONITOR_FILTER_SIZE",	0x00000006,	0xffffffffffffffffULL, NULL },
>  	//{ "TIME_STAMP_COUNTER",	0x00000010,	0xffffffffffffffffULL, NULL },
>  	{ "PLATFORM_ID",		0x00000017,	0x001c000000000000ULL, NULL },
> -	{ "EBL_CR_POWERON",		0x0000002a,	0xffffffffffffffffULL, NULL },
>  	{ "APIC_BASE",			0x0000001b,	0xfffffffffffffeffULL, NULL },
>  	{ "FEATURE_CONTROL",		0x0000003a,	0x000000000000ff07ULL, NULL },
>  	{ "BIOS_SIGN_ID",		0x0000008b,	0xffffffff00000000ULL, NULL },
> 

Acked-by: Colin Ian King <colin.king@canonical.com>

Patch
diff mbox series

diff --git a/src/cpu/msr/msr.c b/src/cpu/msr/msr.c
index 1030a264..a29e7cde 100644
--- a/src/cpu/msr/msr.c
+++ b/src/cpu/msr/msr.c
@@ -343,7 +343,6 @@  static const msr_info IA32_MSRs[] = {
 	{ "MONITOR_FILTER_SIZE",	0x00000006,	0xffffffffffffffffULL, NULL },
 	//{ "TIME_STAMP_COUNTER",	0x00000010,	0xffffffffffffffffULL, NULL },
 	{ "PLATFORM_ID",		0x00000017,	0x001c000000000000ULL, NULL },
-	{ "EBL_CR_POWERON",		0x0000002a,	0xffffffffffffffffULL, NULL },
 	{ "APIC_BASE",			0x0000001b,	0xfffffffffffffeffULL, NULL },
 	{ "FEATURE_CONTROL",		0x0000003a,	0x000000000000ff07ULL, NULL },
 	{ "BIOS_SIGN_ID",		0x0000008b,	0xffffffff00000000ULL, NULL },