Change in coreboot[master]: soc/intel/common: Implement power-failure-state handling

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34724 )

Change subject: soc/intel/common: Implement power-failure-state handling
......................................................................

Patch Set 5:

(3 comments)

https://review.coreboot.org/c/coreboot/+/34724/5/src/soc/intel/common/block/... 
File src/soc/intel/common/block/include/intelblocks/pmclib.h:

https://review.coreboot.org/c/coreboot/+/34724/5/src/soc/intel/common/block/... 
PS5, Line 222: ,
no param name?

https://review.coreboot.org/c/coreboot/+/34724/5/src/soc/intel/common/block/... 
PS5, Line 223: )
same here?

https://review.coreboot.org/c/coreboot/+/34724/5/src/soc/intel/common/block/... 
PS5, Line 223: pmc_set_power_failure_state
Can you please add description to each of the functions as to how they differ? I believe one is a callback that SoC is supposed to provide? Also, it would be good to provide information about what params are expected.