[S] Change in coreboot[main]: mb/google/rex/var/rex: Configure cpu power limits by battery status

[Jamie Ryu (Code Review)]

Attention is currently required from: Brandon Breitenstein, Caveh Jalali, Eran Mitrani, Hannah Williams, Kapil Porwal, Subrata Banik, Sukumar Ghorai, Tarun, Tim Van Patten.

Jamie Ryu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78322?usp=email )

Change subject: mb/google/rex/var/rex: Configure cpu power limits by battery status
......................................................................

Patch Set 6:

(4 comments)

File src/mainboard/google/rex/variants/baseboard/rex/ramstage.c:

https://review.coreboot.org/c/coreboot/+/78322/comment/ab7e99d0_feb27e6d :
PS5, Line 25: p
> 'optimized' sounds like the preferred/nominal profile, […]

this is updated to patchset#6. please let me know if you have any suggestion.

https://review.coreboot.org/c/coreboot/+/78322/comment/8b33a615_998b4060 :
PS5, Line 41: google_chromeec_read_battery_status
> shouldn't this be < 0 ?? […]

This is updated to patchset#6.

https://review.coreboot.org/c/coreboot/+/78322/comment/53ed30ad_825c804c :
PS5, Line 42: 		printk(BIOS_ERR, "Failed to read EC battery info\n");
> > I was thinking to let the boot flow goes with power_optimized_limits profile, but if you have a su […]

This is updated to patchset#6.

https://review.coreboot.org/c/coreboot/+/78322/comment/5b48876d_c4300059 :
PS5, Line 48: battery_status
> > isn't the battery cut-off or no battery is the major concern for us? […]

the patchset#6 is updated to consider 'no battery' and 'level_critical' cases to lower PL4 configuration. Please let me know if you have any comments. Thanks.