[coreboot-gerrit] Change in coreboot[master]: cpu/intel/common: Fix regression

[Michael Niewöhner (Code Review)]

Michael Niewöhner has submitted this change. ( https://review.coreboot.org/c/coreboot/+/46634 )

Change subject: cpu/intel/common: Fix regression
......................................................................

cpu/intel/common: Fix regression

Fix the logic introduced in CB:46276
"cpu/intel/common: only lock AES-NI when supported"
which needs to be negated.

Change-Id: Icaf882625529842ea0aedf39147fc9a9e6081e43
Signed-off-by: Patrick Rudolph 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/46634
Reviewed-by: Christian Walter 
Reviewed-by: Angel Pons 
Reviewed-by: Michael Niewöhner 
Tested-by: build bot (Jenkins) 
---
M src/cpu/intel/common/common_init.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved
  Christian Walter: Looks good to me, approved
  Michael Niewöhner: Looks good to me, approved

diff --git a/src/cpu/intel/common/common_init.c b/src/cpu/intel/common/common_init.c
index 4568014..a54e891 100644
--- a/src/cpu/intel/common/common_init.c
+++ b/src/cpu/intel/common/common_init.c
@@ -273,7 +273,7 @@
    if (!CONFIG(SET_MSR_AESNI_LOCK_BIT))
    	return;

-	if (cpu_get_feature_flags_ecx() & CPUID_AES)
+	if (!(cpu_get_feature_flags_ecx() & CPUID_AES))
    	return;

/* Only run once per core as specified in the MSR datasheet */