[coreboot-gerrit] [M] Change in coreboot[main]: mb/google/brya/var/xol: Configure power limits by battery status

[Paul Menzel (Code Review)]

Attention is currently required from: Dinesh Gehlot, Eric Lai, Jamie Chen, Kapil Porwal, Nick Vaccaro, SH Kim, Sumeet R Pawnikar.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81614?usp=email )

Change subject: mb/google/brya/var/xol: Configure power limits by battery status
......................................................................

Patch Set 2:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/81614/comment/4c3bc9bf_d12657e9 :
PS2, Line 10: platform
            : instabilities
What instabilities occur? Please elaborate? Doesn’t the battery just drain faster?

https://review.coreboot.org/c/coreboot/+/81614/comment/7b847a46_41c7dc32 :
PS2, Line 18: MSR PL2: 55W, MSR PL4: 114W
Please document the command you used to get these.

File src/mainboard/google/brya/variants/xol/ramstage.c:

https://review.coreboot.org/c/coreboot/+/81614/comment/a6447c05_7635d367 :
PS2, Line 24: 
Log the non-zero return value?

https://review.coreboot.org/c/coreboot/+/81614/comment/32fe9c32_772b8d24 :
PS2, Line 64: performance
The commit message says *power*?