[coreboot-gerrit] Change in coreboot[master]: [RFC]soc/intel/common/block/pmc: Don't link get_uint_option in SMM

[Patrick Rudolph (Code Review)]

Attention is currently required from: Jonathan Zhang, Angel Pons, Arthur Heymans, Michael Niewöhner, Kyösti Mälkki, Andrey Petrov, Patrick Rudolph, Nico Huber, Anjaneya "Reddy" Chagam, Johnny Lin, Tim Wawrzynczak, Morgan Jang, Felix Held.
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55114 )

Change subject: [RFC]soc/intel/common/block/pmc: Don't link get_uint_option in SMM
......................................................................

Patch Set 2:

(6 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/55114/comment/fb6f2638_7db39ece 
PS1, Line 7: int
> *u*int since CB:52672 happened

Done

https://review.coreboot.org/c/coreboot/+/55114/comment/0c53edb5_deb16f41 
PS1, Line 7: src/
> nit: drop `src/`

Done

https://review.coreboot.org/c/coreboot/+/55114/comment/96b98e59_02184738 
PS1, Line 14: SMM handler
> over 72 characters

Done

https://review.coreboot.org/c/coreboot/+/55114/comment/85cdfa6b_e9576bb3 
PS1, Line 16: including
> over 72 characters

Done

File src/soc/intel/common/block/include/intelblocks/pmclib.h:

https://review.coreboot.org/c/coreboot/+/55114/comment/1cc22d66_dba9cb9d 
PS1, Line 223: 				 const enum pmc_after_g3_power_state power_on_after_fail);
> To avoid having to change each and every , I wouldn't change the signature of the `pmc_set_power_fai […]

Done

File src/soc/intel/common/block/pmc/pmclib.c:

https://review.coreboot.org/c/coreboot/+/55114/comment/1d17ab14_94a212a0 
PS1, Line 582: power_on_after_fail
> How about `state` for the sake of brevity?

Done