Attention is currently required from: Patrick Rudolph, Paul Menzel.
Hello Paul Menzel, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/78194?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed: Code-Review+1 by Paul Menzel, Verified+1 by build bot (Jenkins)
Change subject: cpu/intel/model_206ax: Only use supported C-states ......................................................................
cpu/intel/model_206ax: Only use supported C-states
When advertising C-state using the ACPI _CST object, make sure to only advertise those that are supported by the CPU. Downgrade if it's not and make sure to not advertise duplicate states.
Add debug prints for the finally selected mapping of ACPI C-state vs Intel CPU C-state.
Test: Tested on Lenovo X220. All C-states are still advertised as all are supported.
Change-Id: Iaaee050e0ce3c29c12e97f5819a29f485a7946c2 Signed-off-by: Patrick Rudolph patrick.rudolph@9elements.com --- M src/cpu/intel/model_206ax/acpi.c 1 file changed, 57 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/94/78194/2