[coreboot] [v2] r4843 - trunk/coreboot-v2/src/cpu/intel/speedstep

svn at coreboot.org svn at coreboot.org
Sat Oct 24 20:36:52 CEST 2009


Author: stepan
Date: 2009-10-24 20:36:51 +0200 (Sat, 24 Oct 2009)
New Revision: 4843

Modified:
   trunk/coreboot-v2/src/cpu/intel/speedstep/acpi.c
Log:
Some ACPI implementations don't like to see full paths within a scope for
CPU power management, so don't add the scope name

Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
Acked-by: Stefan Reinauer <stepan at coresystems.de>



Modified: trunk/coreboot-v2/src/cpu/intel/speedstep/acpi.c
===================================================================
--- trunk/coreboot-v2/src/cpu/intel/speedstep/acpi.c	2009-10-24 18:15:07 UTC (rev 4842)
+++ trunk/coreboot-v2/src/cpu/intel/speedstep/acpi.c	2009-10-24 18:36:51 UTC (rev 4843)
@@ -62,10 +62,9 @@
 
 void generate_cpu_entries(void)
 {
-	int len_sc, len_pr, len_ps;
+	int len_pr, len_ps;
 	int coreID, cpuID, pcontrol_blk=0x510, plen=6;
 	msr_t msr;
-	len_sc = acpigen_write_scope("\\_PR_");
 	int totalcores = determine_total_number_of_cores();
 	int cores_per_package = (cpuid_ebx(1)>>16) & 0xff;
 	int numcpus = totalcores/cores_per_package; // this assumes that all CPUs share the same layout
@@ -129,9 +128,7 @@
 		len_pr += len_ps;
 		len_pr--;
 		acpigen_patch_len(len_pr);
-		len_sc += len_pr;
 		}
 	}
-	acpigen_patch_len(len_sc-1);
 }
 





More information about the coreboot mailing list