[coreboot-gerrit] [S] Change in coreboot[main]: ec/google/chromeec: Add is_battery_present()

[Karthik Ramasubramanian (Code Review)]

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

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

The change is no longer submittable: All-Comments-Resolved is unsatisfied now.

Change subject: ec/google/chromeec: Add is_battery_present()
......................................................................

Patch Set 7:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/83735/comment/513e3c8b_1731eef5?usp... :
PS4, Line 433: google_chromeec_is_battery_present
> i'm not convinced about using […]

My preference is to wait until we make the decision. This API by itself will not help much if we cannot dynamically update the PL4 value. Once we know for sure that this API is required, then we can move forward with this CL.