[coreboot-gerrit] [M] Change in coreboot[main]: soc/amd/genoa: Add SMI support

[Varshit Pandya (Code Review)]

Attention is currently required from: Felix Held, Martin Roth, Paul Menzel.

Varshit Pandya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78221?usp=email )

Change subject: soc/amd/genoa: Add SMI support
......................................................................

Patch Set 4:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/78221/comment/65826f69_ff086d16 :
PS1, Line 8:
> Please elaborate. […]

Done

File src/soc/amd/genoa/include/soc/smi.h:

https://review.coreboot.org/c/coreboot/+/78221/comment/15e5d351_e6cec1b3 :
PS1, Line 8: /* @Todo:  Update the SMI details for Genoa */
> Any reason this is left as a TODO instead of doing it before the patch is submitted?

Done

https://review.coreboot.org/c/coreboot/+/78221/comment/a66968d8_c94d95b7 :
PS1, Line 26: #define SMITYPE_G_GENINT2_L		1
> this table doesn't seem to be updated for genoa yet; this entry should for example be […]

Done