[M] Change in coreboot[main]: mb/google/brox: Boot to kernel without battery

[Sowmya Aralguppe (Code Review)]

Attention is currently required from: Caveh Jalali, Dinesh Gehlot, Eric Herrmann, Forest Mittelberg, Kapil Porwal, Karthik Ramasubramanian, Keith Short, Nick Vaccaro, Rishika Raj, Shelly Chang, Subrata Banik.

Sowmya Aralguppe has posted comments on this change by Sowmya Aralguppe. ( https://review.coreboot.org/c/coreboot/+/83735?usp=email )

Change subject: mb/google/brox: Boot to kernel without battery
......................................................................

Patch Set 4:

(1 comment)

File src/ec/google/chromeec/ec.h:

https://review.coreboot.org/c/coreboot/+/83735/comment/cf86183b_c77800c5?usp... :
PS4, Line 433: google_chromeec_is_battery_present
> please split the addition of a new chromeec API into a separate patch. […]

Yes - Thanks I will create a separate patch .
Regarding google_chromeec_is_battery_present_and_above_critical_threshold (I am considering when (physical) battery present /not present here through some GPIO signal because for this scenario - we can retain optimized PL4 value even after boot .IMO - This is not a typical use case for users but only needed for factory testing.
If we check critical threshold and reduce Power limits - we should bring it back after battery is sufficiently charged so that our PNP KPIs are not impacted .So that will be a different use case .We have a bug on that and actively discussing this issue there .
I am checking - How other programs brought back PL to its original values. AFAIK it continued to operate in degraded mode - I might be wrong about this but i will confirm.