Attention is currently required from: Dinesh Gehlot, Eric Lai, Jamie Chen, Kapil Porwal, Nick Vaccaro, SH Kim, Sumeet R Pawnikar.
Hello Dinesh Gehlot, Eric Lai, Jamie Chen, Kapil Porwal, Nick Vaccaro, Subrata Banik, Sumeet R Pawnikar,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/81614?usp=email
to look at the new patch set (#3).
Change subject: mb/google/brya/var/xol: Configure power limits by battery status ......................................................................
mb/google/brya/var/xol: Configure power limits by battery status
When battery level is below critical level or battery is not present, cpus need to run with a power optimized configuration to avoid platform instabilities such as system power down.
This will check the current battery status and configure cpu power limits using current PD power value.
BUG=b:328729536 TEST=built and veified MSR PL2/PL4 values using PTAT.
Original power limit configuration for Xol: MSR PL2: 55W, MSR PL4: 114W.
[When connected 60W adaptor without battery] Before: MSR PL2: 55W, MSR PL4: 114W After: MSR PL2: 55W, MSR PL4: 60W
[When connected 45W adaptor without battery] Before: MSR PL2: 55W, MSR PL4: 114W After: MSR PL2: 45W, MSR PL4: 45W
Change-Id: I5d71e9edde0ecbd7aaf316cd754a6ebcff9da77d Signed-off-by: Seunghwan Kim sh_.kim@samsung.corp-partner.google.com --- M src/mainboard/google/brya/variants/baseboard/brya/ramstage.c M src/mainboard/google/brya/variants/baseboard/include/baseboard/variants.h M src/mainboard/google/brya/variants/xol/Makefile.mk A src/mainboard/google/brya/variants/xol/ramstage.c 4 files changed, 65 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/14/81614/3