Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35188 )
Change subject: soc/skylake: lock aesni msr ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/35188/4//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/35188/4//COMMIT_MSG@7 PS4, Line 7: msr MSR
https://review.coreboot.org/c/coreboot/+/35188/4/src/soc/intel/skylake/cpu.c File src/soc/intel/skylake/cpu.c:
https://review.coreboot.org/c/coreboot/+/35188/4/src/soc/intel/skylake/cpu.c... PS4, Line 429: /* Lock it */ This comment is really obnoxious... maybe explain what locking this is useful for?