[coreboot-gerrit] Change in ...coreboot[master]: mb/lenovo/t60: Align ACPI C-state across the similar boards

[Peter Lemenkov (Code Review)]

Hello Alexander Couzens, Patrick Rudolph, Arthur Heymans, Paul Menzel, build bot (Jenkins), Nico Huber,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/29160

to look at the new patch set (#3).

Change subject: mb/lenovo/t60: Align ACPI C-state across the similar boards
......................................................................

mb/lenovo/t60: Align ACPI C-state across the similar boards

We have 3 similar Lenovo mainboards - x60 (oldest), t60, and z61t (most
recent addition). The only one with two consequent 2s as the C-types
is t60:

static acpi_cstate_t cst_entries[] = {
       { 1,  1, 1000, { 0x7f, 1, 2, { 0 }, 1, 0 } },
       { 2,  1,  500, { 0x01, 8, 0, { 0 }, DEFAULT_PMBASE + LV2, 0 } },
       { 2, 17,  250, { 0x01, 8, 0, { 0 }, DEFAULT_PMBASE + LV3, 0 } },
};

It seems that 3 could be a better choice for the last line here.

UNTESTED on hardware.

Change-Id: I090e82d5f4ae25c768ff45a01a8dd76ff8a96a90
Signed-off-by: Peter Lemenkov 
---
M src/mainboard/lenovo/t60/mainboard.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/29160/3