cedarhouse1@comcast.net has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33234 )
Change subject: security/intel/stm: Add STM support ......................................................................
Patch Set 51:
(2 comments)
random spell check...
https://review.coreboot.org/c/coreboot/+/33234/51/src/cpu/x86/mp_init.c File src/cpu/x86/mp_init.c:
https://review.coreboot.org/c/coreboot/+/33234/51/src/cpu/x86/mp_init.c@1051 PS51, Line 1051: * that require alignment of the save state on 32K boundries.
'boundries' may be misspelled - perhaps 'boundaries'?
Done
https://review.coreboot.org/c/coreboot/+/33234/51/src/security/intel/stm/Stm... File src/security/intel/stm/StmApi.h:
https://review.coreboot.org/c/coreboot/+/33234/51/src/security/intel/stm/Stm... PS51, Line 486: // Intepreter of DomainType
'Intepreter' may be misspelled - perhaps 'Interpreter'?
Done