[coreboot-gerrit] Change in coreboot[master]: security/intel/stm: Add STM support

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33234 )

Change subject: security/intel/stm: Add STM support
......................................................................

Patch Set 48: Code-Review+1

(3 comments)

https://review.coreboot.org/c/coreboot/+/33234/48/src/cpu/x86/mp_init.c 
File src/cpu/x86/mp_init.c:

https://review.coreboot.org/c/coreboot/+/33234/48/src/cpu/x86/mp_init.c@357 
PS48, Line 357: abiility
why is a typo being added?

https://review.coreboot.org/c/coreboot/+/33234/48/src/cpu/x86/mp_init.c@758 
PS48, Line 758: 		} else
Code style says that, since the if-branch needs braces, the else-branch should use braces as well:

https://doc.coreboot.org/coding_style.html

https://review.coreboot.org/c/coreboot/+/33234/48/src/cpu/x86/mp_init.c@1040 
PS48, Line 1040: 	// make sure there is enough room for the SMM descriptor
nit: comment style should be consistent