Author: jakllsch Date: Fri Mar 4 00:09:43 2011 New Revision: 6430 URL: https://tracker.coreboot.org/trac/coreboot/changeset/6430
Log: 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 jakllsch@kollasch.net Acked-by: Peter Stuge peter@stuge.se
Modified: trunk/src/cpu/amd/model_fxx/powernow_acpi.c
Modified: trunk/src/cpu/amd/model_fxx/powernow_acpi.c ============================================================================== --- trunk/src/cpu/amd/model_fxx/powernow_acpi.c Thu Mar 3 21:52:50 2011 (r6429) +++ trunk/src/cpu/amd/model_fxx/powernow_acpi.c Fri Mar 4 00:09:43 2011 (r6430) @@ -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,