[coreboot-gerrit] Patch set updated for coreboot: model_fxx/powernow: add dual core Socket F TDPs

Jonathan A. Kollasch (jakllsch@kollasch.net) gerrit at coreboot.org
Thu Oct 22 20:34:01 CEST 2015


Jonathan A. Kollasch (jakllsch at kollasch.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10926

-gerrit

commit 48a3e428207ce45317a7e833d55ebcb543b1dd69
Author: Jonathan A. Kollasch <jakllsch at kollasch.net>
Date:   Wed Jul 15 14:34:17 2015 -0500

    model_fxx/powernow: add dual core Socket F TDPs
    
    Values based on correlation of brand strings, brand numbers and the TDP
    listings on AMD's web site (Wikipedia for Athlon 64 FX-7x TDPs).
    
    Change-Id: I7e6d12d0b6cc4fefc3f84076234c62c40e08304c
    Signed-off-by: Jonathan A. Kollasch <jakllsch at kollasch.net>
---
 src/cpu/amd/model_fxx/powernow_acpi.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/cpu/amd/model_fxx/powernow_acpi.c b/src/cpu/amd/model_fxx/powernow_acpi.c
index ac505ee..1ce746d 100644
--- a/src/cpu/amd/model_fxx/powernow_acpi.c
+++ b/src/cpu/amd/model_fxx/powernow_acpi.c
@@ -106,7 +106,12 @@ static void pstates_algorithm(u32 pcontrol_blk, u8 plen, u8 onlyBSP)
 	u8 index;
 	msr_t msr;
 	u32 fid_multiplier;
-	static struct power_limit_encoding TDP[20] = {
+	static const struct power_limit_encoding TDP[] = {
+		{0x10, 0x1, 0x2, 45}, /* Opteron EE */
+		{0x10, 0x1, 0x6, 68}, /* Opteron HE */
+		{0x10, 0x1, 0xa, 95}, /* Opteron */
+		{0x10, 0x1, 0xc, 119}, /* Opteron SE */
+		{0x10, 0x1, 0xe, 125}, /* Athlon 64 FX-7x */
 		{0x11, 0x0, 0x8, 62},
 		{0x11, 0x1, 0x8, 89},
 		{0x11, 0x1, 0xa, 103},



More information about the coreboot-gerrit mailing list