Correct off-by-one problem in AMD pre-rev-F model-F PowerNow code.
With this change the last P-state entry of the last CPU in the table
is successfully conveyed into the SSDT.
Signed-off-by: Jonathan Kollasch <[email protected]>
----
Index: src/cpu/amd/model_fxx/powernow_acpi.c
===================================================================
--- src/cpu/amd/model_fxx/powernow_acpi.c (revision 6429)
+++ src/cpu/amd/model_fxx/powernow_acpi.c (working copy)
@@ -622,7 +622,7 @@
Pstate_vid[Pstate_num] = vid_to_reg(data->pstates[Pstate_num -
1].voltage);
Pstate_power[Pstate_num] = data->pstates[Pstate_num - 1].tdp *
100;
Pstate_num++;
- } while ((Pstate_num < MAXP) && (data->pstates[Pstate_num].freqMhz !=
0));
+ } while ((Pstate_num < MAXP) && (data->pstates[Pstate_num - 1].freqMhz
!= 0));
for (i=0;i<Pstate_num;i++)
printk(BIOS_DEBUG, "P#%d freq %d [MHz] voltage %d [mV] TDP %d
[mW]\n", i,
--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot
