[coreboot] [commit] r6430 - trunk/src/cpu/amd/model_fxx

[repository service]

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 at kollasch.net>
Acked-by: Peter Stuge <peter at 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,