<p>Peter Lemenkov has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/29160">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mb/lenovo/t60: Fix typo in ACPI C-state<br><br>We have 3 similar Lenovo mainboards - x60 (oldest), t60, and z61t (most<br>recent addition).  Their ACPI C-state tables were reworked with commit<br>4cc8c70c (Change-Id Icb87418d44d355f607c4a67300107b40f40b3b3f) to<br>something like that:<br><br>static acpi_cstate_t cst_entries[] = {<br>       { 1,  1, 1000, { 0x7f, 1, 2, { 0 }, 1, 0 } },<br>       { 2,  1,  500, { 0x01, 8, 0, { 0 }, DEFAULT_PMBASE + LV2, 0 } },<br>       { 2, 17,  250, { 0x01, 8, 0, { 0 }, DEFAULT_PMBASE + LV3, 0 } },<br>};<br><br>where acpi_cstate_t is defined as follows:<br><br>typedef struct acpi_cstate {<br>        u8  ctype;<br>        u16 latency;<br>        u32 power;<br>        acpi_addr_t resource;<br>} __packed acpi_cstate_t;<br><br>Please, note two consequent 2s as ctype values.<br><br>Later this was corrected for x60 mainboard in commit e1f0ac4b (Change-Id<br>I9eeb5b008e2ddc2193725667f2c13582a4877e3c). Since then the last line<br>starts from 3 as ctype value.<br><br>And z61t was introduced with 3 as the last CST entry's ctype - see<br>commit 60d9ce39 (Change-Id I84c89cc47d3db126d827f92d50270954bc42f224).<br><br>For me it looks like typo, but I am not sure here. Could someone please<br>clarify whether t60 should have the ACPI C-state table listed above or<br>the following one (as two other boards have):<br><br>static acpi_cstate_t cst_entries[] = {<br>       { 1,  1, 1000, { 0x7f, 1, 2, { 0 }, 1, 0 } },<br>       { 2,  1,  500, { 0x01, 8, 0, { 0 }, DEFAULT_PMBASE + LV2, 0 } },<br>       { 3, 17,  250, { 0x01, 8, 0, { 0 }, DEFAULT_PMBASE + LV3, 0 } },<br>};<br><br>Change-Id: I090e82d5f4ae25c768ff45a01a8dd76ff8a96a90<br>Signed-off-by: Peter Lemenkov <lemenkov@gmail.com><br>---<br>M src/mainboard/lenovo/t60/mainboard.c<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/29160/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/mainboard/lenovo/t60/mainboard.c b/src/mainboard/lenovo/t60/mainboard.c</span><br><span>index bb52c87..5a565e0 100644</span><br><span>--- a/src/mainboard/lenovo/t60/mainboard.c</span><br><span>+++ b/src/mainboard/lenovo/t60/mainboard.c</span><br><span>@@ -29,7 +29,7 @@</span><br><span> static acpi_cstate_t cst_entries[] = {</span><br><span>        { 1,  1, 1000, { 0x7f, 1, 2, { 0 }, 1, 0 } },</span><br><span>        { 2,  1,  500, { 0x01, 8, 0, { 0 }, DEFAULT_PMBASE + LV2, 0 } },</span><br><span style="color: hsl(0, 100%, 40%);">-        { 2, 17,  250, { 0x01, 8, 0, { 0 }, DEFAULT_PMBASE + LV3, 0 } },</span><br><span style="color: hsl(120, 100%, 40%);">+      { 3, 17,  250, { 0x01, 8, 0, { 0 }, DEFAULT_PMBASE + LV3, 0 } },</span><br><span> };</span><br><span> </span><br><span> int get_cst_entries(acpi_cstate_t **entries)</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/29160">change 29160</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/29160"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I090e82d5f4ae25c768ff45a01a8dd76ff8a96a90 </div>
<div style="display:none"> Gerrit-Change-Number: 29160 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Peter Lemenkov <lemenkov@gmail.com> </div>