[coreboot-gerrit] Change in coreboot[master]: cpu/intel/common: rework AES-NI locking

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46275 )

Change subject: cpu/intel/common: rework AES-NI locking
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/c/coreboot/+/46275/4/src/cpu/intel/common/common... 
File src/cpu/intel/common/common_init.c:

https://review.coreboot.org/c/coreboot/+/46275/4/src/cpu/intel/common/common... 
PS4, Line 269: /*
             :  * Lock AES-NI feature (MSR_FEATURE_CONFIG) to prevent unintended changes
             :  * to the enablement state as suggested in Intel document 325384-070US.
             :  */
> So, where would you like to have this? here or in the header?

Preferably just the header, but we're not consistent enough about this 😐