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@coresystems.de Acked-by: Stefan Reinauer stepan@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); }