[coreboot-gerrit] Change in coreboot[master]: mb/google/hatch/var/jinlon: Remove DPTF fan control

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/38492 )

Change subject: mb/google/hatch/var/jinlon: Remove DPTF fan control
......................................................................

mb/google/hatch/var/jinlon: Remove DPTF fan control

Jinlon will use EC to control fan, so remove DPTF fan control.

BUG=b:141259174
TEST=emerge-hatch coreboot chromeos-bootimage

Change-Id: I8ada4fe72eee260fecf45d00510da8b91e3f10a4
Signed-off-by: Wisley Chen 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38492
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
---
M src/mainboard/google/hatch/variants/jinlon/include/variant/acpi/dptf.asl
1 file changed, 0 insertions(+), 34 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved

diff --git a/src/mainboard/google/hatch/variants/jinlon/include/variant/acpi/dptf.asl b/src/mainboard/google/hatch/variants/jinlon/include/variant/acpi/dptf.asl
index 74195e1..ffa7590 100644
--- a/src/mainboard/google/hatch/variants/jinlon/include/variant/acpi/dptf.asl
+++ b/src/mainboard/google/hatch/variants/jinlon/include/variant/acpi/dptf.asl
@@ -32,7 +32,6 @@
 #define DPTF_TSR1_CRITICAL	105

#define DPTF_ENABLE_CHARGER
-#define DPTF_ENABLE_FAN_CONTROL

/* Charger performance states, board-specific values from charger and EC */
 Name (CHPS, Package () {
@@ -42,39 +41,6 @@
    Package () { 0, 0, 0, 0, 8, 0x200, "mA", 0 },	/* 0.5A */
 })

-/* DFPS: Fan Performance States */
-Name (DFPS, Package () {
-	0,	// Revision
-	/*
-	 * TODO : Need to update this Table after characterization.
-	 *	  These are initial reference values.
-	 */
-	/* Control, Trip Point, Speed, NoiseLevel, Power */
-	Package () {90,		0xFFFFFFFF,	6700,	220,	2200},
-	Package () {80,		0xFFFFFFFF,	5800,	180,	1800},
-	Package () {70,		0xFFFFFFFF,	5000,	145,	1450},
-	Package () {67,		0xFFFFFFFF,	4900,	115,	1150},
-	Package () {60,		0xFFFFFFFF,	3838,	90,	900},
-	Package () {50,		0xFFFFFFFF,	2904,	55,	550},
-	Package () {40,		0xFFFFFFFF,	2337,	30,	300},
-	Package () {20,		0xFFFFFFFF,	1608,	15,	150},
-	Package () {10,		0xFFFFFFFF,	800,	10,	100},
-	Package () {0,		0xFFFFFFFF,	0,	0,	50}
-})
-
-Name (DART, Package () {
-	/* Fan effect on CPU */
-	0,	// Revision
-	Package () {
-		/*
-		 * Source, Target, Weight, AC0, AC1, AC2, AC3, AC4, AC5, AC6,
-		 *	AC7, AC8, AC9
-		 */
-		_SB.DPTF.TFN1, _SB.PCI0.TCPU, 100, 70, 67, 60, 50, 40, 0, 0,
-			0, 0, 0
-	},
-})
-
 Name (DTRT, Package () {
    /* CPU Throttle Effect on CPU */
    Package () { _SB.PCI0.TCPU, _SB.PCI0.TCPU, 100, 50, 0, 0, 0, 0 },