[coreboot-gerrit] [S] Change in coreboot[main]: soc/intel/common: Add low battery shutdown function

[Julius Werner (Code Review)]

Attention is currently required from: Angel Pons, Intel coreboot Reviewers, Jérémy Compostella, Karthik Ramasubramanian, Subrata Banik.

Julius Werner has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/86361?usp=email )

Change subject: soc/intel/common: Add low battery shutdown function
......................................................................

Patch Set 4:

(1 comment)

File src/soc/intel/common/reset.c:

https://review.coreboot.org/c/coreboot/+/86361/comment/fc56a014_74d91e37?usp... :
PS4, Line 27: 	poweroff();
Wait, where did the rest of this function go? Who is doing the delay and the elog entry now? (If this function does nothing more than `poweroff()`, it doesn't need to exist, the caller could just call `poweroff()` directly.)