[coreboot-gerrit] [S] Change in coreboot[master]: intel/cmn/smm: Introduce PERIODIC_SMI_RATE_SELECTION_IN_GEN_PMCON_B

[Michał Żygowski (Code Review)]

Attention is currently required from: Sean Rhodes, Krystian Hebel.

Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69870 )

Change subject: intel/cmn/smm: Introduce PERIODIC_SMI_RATE_SELECTION_IN_GEN_PMCON_B
......................................................................

Patch Set 1:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/69870/comment/ec383115_810d871f 
PS1, Line 9: Certain CPUs/SoCs like Apollo Lake use GEN_PMCON_B for periodic SMI
> Do they use the same socket as CPUs using `GEN_PMCON_A`? If yes, maybe this should be handled in cod […]

`Do they use the same socket as CPUs` I don;t understand what you mean.

Certain uarchs have different register layout for GEN_PMCON fields, don't know why