Name of user not set #1002358 has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33234 )
Change subject: security/intel/stm: Add STM support ......................................................................
Patch Set 11:
(4 comments)
https://review.coreboot.org/c/coreboot/+/33234/1/src/security/intel/stm/StmP... File src/security/intel/stm/StmPlatformResource.c:
https://review.coreboot.org/c/coreboot/+/33234/1/src/security/intel/stm/StmP... PS1, Line 180: // Find max bus number and PCIEX length
Done
Done
https://review.coreboot.org/c/coreboot/+/33234/1/src/security/intel/stm/StmP... PS1, Line 180: // Find max bus number and PCIEX length
This is part of the code that is generating the access list for the SMI handler. […]
Done
https://review.coreboot.org/c/coreboot/+/33234/1/src/security/intel/stm/StmP... File src/security/intel/stm/StmPlatformSmm.c:
https://review.coreboot.org/c/coreboot/+/33234/1/src/security/intel/stm/StmP... PS1, Line 27: extern bool StmCheckStmImage(void *StmImage, uint32_t StmImageSize);
will be in updated changes
Done
https://review.coreboot.org/c/coreboot/+/33234/1/src/security/intel/stm/StmP... PS1, Line 38: int LoadStmImage(uint32_t mseg)
done - in next change update
Done