[coreboot-gerrit] Change in coreboot[master]: soc/amd/psp: Add SmmInfo command

[Marshall Dawson (Code Review)]

Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40295 )

Change subject: soc/amd/psp: Add SmmInfo command
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/40295/8//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/40295/8//COMMIT_MSG@17 
PS8, Line 17: The soc should call the new function only from an SMI handler. 
Only a nit here.  AFAIK the x86 can be in any mode when the SmmInfo call is sent to the PSP.  The main reason we put it in the SMI handler is so that buffers, addresses, etc. are easily identifiable.  If we tried calling it from ramstage instead, it would be difficult to know where a buffer was going to wind up inside the smm.elf.