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

Attention is currently required from: Dinesh Gehlot, Eric Lai, Jamie Chen, Kapil Porwal, Nick Vaccaro, Paul Menzel, 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 (#4). 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 BRANCH=brya 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/xol/Makefile.mk A src/mainboard/google/brya/variants/xol/ramstage.c 2 files changed, 59 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/14/81614/4 -- To view, visit https://review.coreboot.org/c/coreboot/+/81614?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: main Gerrit-Change-Id: I5d71e9edde0ecbd7aaf316cd754a6ebcff9da77d Gerrit-Change-Number: 81614 Gerrit-PatchSet: 4 Gerrit-Owner: SH Kim <sh_.kim@samsung.corp-partner.google.com> Gerrit-Reviewer: Dinesh Gehlot <digehlot@google.com> Gerrit-Reviewer: Eric Lai <ericllai@google.com> Gerrit-Reviewer: Jamie Chen <jamie.chen@intel.com> Gerrit-Reviewer: Kapil Porwal <kapilporwal@google.com> Gerrit-Reviewer: Nick Vaccaro <nvaccaro@chromium.org> Gerrit-Reviewer: Subrata Banik <subratabanik@google.com> Gerrit-Reviewer: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com> Gerrit-CC: Paul Menzel <paulepanter@mailbox.org> Gerrit-Attention: Eric Lai <ericllai@google.com> Gerrit-Attention: Jamie Chen <jamie.chen@intel.com> Gerrit-Attention: SH Kim <sh_.kim@samsung.corp-partner.google.com> Gerrit-Attention: Paul Menzel <paulepanter@mailbox.org> Gerrit-Attention: Kapil Porwal <kapilporwal@google.com> Gerrit-Attention: Dinesh Gehlot <digehlot@google.com> Gerrit-Attention: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com> Gerrit-Attention: Nick Vaccaro <nvaccaro@chromium.org> Gerrit-MessageType: newpatchset
participants (1)
-
SH Kim (Code Review)