[S] Change in coreboot[main]: soc/intel/adl: Handle critical low battery early in romstage

[Jérémy Compostella (Code Review)]

Attention is currently required from: Dinesh Gehlot, Intel coreboot Reviewers, Jayvik Desai, Kapil Porwal, Karthik Ramasubramanian, Nick Vaccaro, Subrata Banik.

Jérémy Compostella has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/86227?usp=email )

Change subject: soc/intel/adl: Handle critical low battery early in romstage
......................................................................

Patch Set 1:

(2 comments)

File src/soc/intel/alderlake/romstage/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/86227/comment/164bf92e_42b2c126?usp... :
PS1, Line 427: 		google_chromeec_reboot(EC_REBOOT_COLD_AP_OFF,0);
> `space required after that ',' (ctx:VxV)`

Please fix.

https://review.coreboot.org/c/coreboot/+/86227/comment/2536b4ae_e2c92583?usp... :
PS1, Line 446: is_early_critical_shutdown_needed
A function start with `is_` is commonly expected to return a boolean and to not take any action. I would suggest to rename it. What about `early_shutdown_if_battery_below_critical_threshold` ?