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 <[email protected]>
Acked-by: Stefan Reinauer <[email protected]>



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);
 }
 


-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to