cpu/intel/speedstep: Fix the PNOT ACPI method

The PNOT method never notifies the CPU to update it's _CST methods due
to reliance on inexisting variable (PDCx).

Add a method in the speedstep ssdt generator to notify all available
CPU nodes and hook this up in this file.

The cpu.asl file is moved to cpu/intel/speedstep/acpi since it now
relies on code generated in the speedstep ssdt generator. CPUs not
using the speedstep code never included this PNOT method so this is
a logical place for this code to be.

Change-Id: Ie2ba5e07b401d6f7c80c31f2bfcd9ef3ac0c1ad1
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/23144
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>

src/arch/x86/include/arch/acpigen.h

@@ -82,6 +82,7 @@ enum {
 	AND_OP			= 0x7B,
 	OR_OP			= 0x7D,
 	NOT_OP			= 0x80,
+	NOTIFY_OP		= 0x86,
 	LEQUAL_OP		= 0x93,
 	LGREATER_OP		= 0x94,
 	LLESS_OP		= 0x95,

src/cpu/intel/speedstep/acpi.c

@@ -23,6 +23,7 @@
 #include <cpu/x86/msr.h>
 #include <cpu/intel/speedstep.h>
 #include <device/device.h>
+#include <string.h>
 
 static int determine_total_number_of_cores(void)
 {
@@ -164,4 +165,16 @@ void generate_cpu_entries(device_t device)
 	/* PPKG is usually used for thermal management
 	   of the first and only package. */
 	acpigen_write_processor_package("PPKG", 0, cores_per_package);
+
+	/* Add a method to notify processor nodes */
+	acpigen_write_method("\\_PR.CNOT", 1);
+	for (coreID = 0; coreID < cores_per_package; coreID++) {
+		char buffer[DEVICE_PATH_MAX];
+		snprintf(buffer, sizeof(buffer), "\\_PR.CP%c%c",
+			'0' + coreID / 10, '0' + coreID % 10);
+		acpigen_emit_byte(NOTIFY_OP);
+		acpigen_emit_namestring(buffer);
+		acpigen_emit_byte(ARG0_OP);
+	}
+	acpigen_pop_len();
 }

src/cpu/intel/speedstep/acpi/cpu.asl

@@ -12,36 +12,17 @@
  */
 
 /* These come from the dynamically created CPU SSDT */
-External(PDC0)
+External (\_PR.CNOT, MethodObj)
-External(PDC1)
-
-// Power notification
-
 External (\_PR_.CP00, DeviceObj)
-External (\_PR_.CP01, DeviceObj)
 External (\_PR_.CP00._PPC)
 External (\_PR_.CP01._PPC)
 
 Method (PNOT)
 {
 	If (MPEN) {
-		If(And(PDC0, 0x08)) {
+		\_PR.CNOT (0x80) // _PPC
-			Notify (\_PR_.CP00, 0x80)	 // _PPC
-
-			If (And(PDC0, 0x10)) {
 		Sleep(100)
-				Notify(\_PR_.CP00, 0x81) // _CST
+		\_PR.CNOT (0x81) // _CST
-			}
-		}
-
-		If(And(PDC1, 0x08)) {
-			Notify (\_PR_.CP01, 0x80)	 // _PPC
-			If (And(PDC1, 0x10)) {
-				Sleep(100)
-				Notify(\_PR_.CP01, 0x81) // _CST
-			}
-		}
-
 	} Else { // UP
 		Notify (\_PR_.CP00, 0x80)
 		Sleep(0x64)

src/mainboard/apple/macbook21/dsdt.asl

@@ -34,7 +34,7 @@ DefinitionBlock(
 	#include <southbridge/intel/i82801gx/acpi/globalnvs.asl>
 	#include <southbridge/intel/common/acpi/platform.asl>
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/asus/p5gc-mx/dsdt.asl

@@ -38,7 +38,7 @@ DefinitionBlock(
 	// Thermal Zone
 	//#include "acpi/thermal.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/getac/p470/dsdt.asl

@@ -43,7 +43,7 @@ DefinitionBlock(
 	// Thermal Zone
 	#include "acpi/thermal.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/gigabyte/ga-945gcm-s2l/dsdt.asl

@@ -38,7 +38,7 @@ DefinitionBlock(
 	// Thermal Zone
 	//#include "acpi/thermal.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/ibase/mb899/dsdt.asl

@@ -35,7 +35,7 @@ DefinitionBlock(
 
 	//#include "acpi/thermal.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/intel/d510mo/dsdt.asl

@@ -26,7 +26,7 @@ DefinitionBlock(
 	#include "acpi/platform.asl"
 	#include <southbridge/intel/i82801gx/acpi/globalnvs.asl>
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/intel/d945gclf/dsdt.asl

@@ -38,7 +38,7 @@ DefinitionBlock(
 	// Thermal Zone
 	//#include "acpi/thermal.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/kontron/986lcd-m/dsdt.asl

@@ -34,7 +34,7 @@ DefinitionBlock(
 
 	//#include "acpi/thermal.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/lenovo/t400/dsdt.asl

@@ -39,7 +39,7 @@ DefinitionBlock(
 	// General Purpose Events
 	#include "acpi/gpe.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/lenovo/t60/dsdt.asl

@@ -47,7 +47,7 @@ DefinitionBlock(
 		#include "acpi/video.asl"
 	}
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/lenovo/x200/dsdt.asl

@@ -39,7 +39,7 @@ DefinitionBlock(
 	// General Purpose Events
 	#include "acpi/gpe.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/lenovo/x60/dsdt.asl

@@ -43,7 +43,7 @@ DefinitionBlock(
 	// mainboard specific devices
 	#include "acpi/mainboard.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/roda/rk886ex/dsdt.asl

@@ -39,7 +39,7 @@ DefinitionBlock(
 	// Thermal Zone
 	#include "acpi/thermal.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	Scope (\_SB) {
 		Device (PCI0)

src/mainboard/roda/rk9/dsdt.asl

@@ -38,7 +38,7 @@ DefinitionBlock(
 	// mainboard specific devices
 	#include "acpi/mainboard.asl"
 
-	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <cpu/intel/speedstep/acpi/cpu.asl>
 
 	// Thermal Zone
 	#include "acpi/thermal.asl"