John Su has uploaded this change for review. ( https://review.coreboot.org/27020
Change subject: mb/google/poppy/variants/nami: Solve thermal sensor QT2 PSV is not work ......................................................................
mb/google/poppy/variants/nami: Solve thermal sensor QT2 PSV is not work
Modify DPTF TRT parameters to solve thermal sensor QT2 PSV problem, and update DPTF table from the thermal team.
BUG=b:109941652 TEST=Thermal sensor QT2 PSV is work
Change-Id: Iea3a790945ce542890e51b61e07a4202c4d19064 Signed-off-by: John Su john_su@compal.corp-partner.google.com --- M src/mainboard/google/poppy/variants/nami/include/variant/acpi/dptf.asl 1 file changed, 11 insertions(+), 11 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/20/27020/1
diff --git a/src/mainboard/google/poppy/variants/nami/include/variant/acpi/dptf.asl b/src/mainboard/google/poppy/variants/nami/include/variant/acpi/dptf.asl index 05d5552..c1a13d5 100644 --- a/src/mainboard/google/poppy/variants/nami/include/variant/acpi/dptf.asl +++ b/src/mainboard/google/poppy/variants/nami/include/variant/acpi/dptf.asl @@ -23,23 +23,23 @@
#define DPTF_TSR0_SENSOR_ID 0 #define DPTF_TSR0_SENSOR_NAME "Thermal_Sensor_Remote_CPU" -#define DPTF_TSR0_PASSIVE 81 +#define DPTF_TSR0_PASSIVE 75 #define DPTF_TSR0_CRITICAL 125 #define DPTF_TSR0_ACTIVE_AC0 50 #define DPTF_TSR0_ACTIVE_AC1 47 #define DPTF_TSR0_ACTIVE_AC2 45 -#define DPTF_TSR0_ACTIVE_AC3 43 -#define DPTF_TSR0_ACTIVE_AC4 41 +#define DPTF_TSR0_ACTIVE_AC3 42 +#define DPTF_TSR0_ACTIVE_AC4 39
#define DPTF_TSR1_SENSOR_ID 1 #define DPTF_TSR1_SENSOR_NAME "Thermal_Sensor_Remote_PMIC" -#define DPTF_TSR1_PASSIVE 78 +#define DPTF_TSR1_PASSIVE 75 #define DPTF_TSR1_CRITICAL 125 #define DPTF_TSR1_ACTIVE_AC0 50 #define DPTF_TSR1_ACTIVE_AC1 47 #define DPTF_TSR1_ACTIVE_AC2 45 -#define DPTF_TSR1_ACTIVE_AC3 43 -#define DPTF_TSR1_ACTIVE_AC4 41 +#define DPTF_TSR1_ACTIVE_AC3 42 +#define DPTF_TSR1_ACTIVE_AC4 39
#define DPTF_ENABLE_CHARGER #define DPTF_ENABLE_FAN_CONTROL @@ -63,7 +63,7 @@ /* Control, Trip Point, Speed, NoiseLevel, Power */ Package () {90, 0xFFFFFFFF, 6700, 220, 2200}, Package () {69, 0xFFFFFFFF, 5800, 180, 1800}, - Package () {52, 0xFFFFFFFF, 5000, 145, 1450}, + Package () {56, 0xFFFFFFFF, 5000, 145, 1450}, Package () {46, 0xFFFFFFFF, 4900, 115, 1150}, Package () {36, 0xFFFFFFFF, 3900, 90, 900} }) @@ -76,15 +76,15 @@ * Source, Target, Weight, AC0, AC1, AC2, AC3, AC4, AC5, AC6, * AC7, AC8, AC9 */ - _SB.DPTF.TFN1, _SB.PCI0.B0D4, 100, 90, 69, 52, 46, 36, 0, 0, + _SB.DPTF.TFN1, _SB.PCI0.B0D4, 100, 90, 69, 56, 46, 36, 0, 0, 0, 0, 0 }, Package () { - _SB.DPTF.TFN1, _SB.DPTF.TSR0, 100, 90, 69, 52, 46, 36, 0, 0, + _SB.DPTF.TFN1, _SB.DPTF.TSR0, 100, 90, 69, 56, 46, 36, 0, 0, 0, 0, 0 }, Package () { - _SB.DPTF.TFN1, _SB.DPTF.TSR1, 100, 90, 69, 52, 46, 36, 0, 0, + _SB.DPTF.TFN1, _SB.DPTF.TSR1, 100, 90, 69, 56, 46, 36, 0, 0, 0, 0, 0 } }) @@ -98,7 +98,7 @@ Package () { _SB.PCI0.B0D4, _SB.DPTF.TSR0, 100, 1, 0, 0, 0, 0 }, #ifdef DPTF_ENABLE_CHARGER /* Charger Throttle Effect on Charger (TSR1) */ - Package () { _SB.DPTF.TCHG, _SB.DPTF.TSR1, 100, 1, 0, 0, 0, 0 }, + Package () { _SB.PCI0.B0D4, _SB.DPTF.TSR1, 100, 1, 0, 0, 0, 0 }, #endif })