[coreboot-gerrit] Patch set updated for coreboot: apollolake: Set RAPL power limit PL1 value to 12W.

Sumeet R Pawnikar (sumeet.r.pawnikar@intel.com) gerrit at coreboot.org
Tue Sep 13 20:27:26 CEST 2016


Sumeet R Pawnikar (sumeet.r.pawnikar at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16596

-gerrit

commit ae52d5c671fe092eaa977b126a4cc7eb7e66fc2b
Author: Sumeet Pawnikar <sumeet.r.pawnikar at intel.com>
Date:   Tue Sep 13 11:54:03 2016 +0530

    apollolake: Set RAPL power limit PL1 value to 12W.
    
    This patch sets tuned RAPL power limit PL1 value to
    12W in acpi/dptf.asl for RAPL MSR register. With PL1
    as 12W for WebGL and stream case, we measured SoC power
    reaching upto 6W. Above 12W PL1 value, we observed that
    Soc power going above 6W. With PL1 as 12W, system is
    able to leverage full TDP capacity.
    
    BUG=chrome-os-partner:56524
    TEST=Built, booted on reef and verifed the package
    power with heavy workload.
    
    Change-Id: I8185ce890f27e29bc138ea568af536bc274fe7b8
    Signed-off-by: Sumeet Pawnikar <sumeet.r.pawnikar at intel.com>
---
 src/mainboard/google/reef/acpi/dptf.asl  | 2 +-
 src/mainboard/intel/amenia/acpi/dptf.asl | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/mainboard/google/reef/acpi/dptf.asl b/src/mainboard/google/reef/acpi/dptf.asl
index 33d9b4d..1f9bfb2 100644
--- a/src/mainboard/google/reef/acpi/dptf.asl
+++ b/src/mainboard/google/reef/acpi/dptf.asl
@@ -73,7 +73,7 @@ Name (MPPC, Package ()
 	Package () {	/* Power Limit 1 */
 		0,	/* PowerLimitIndex, 0 for Power Limit 1 */
 		1600,	/* PowerLimitMinimum */
-		15000,	/* PowerLimitMaximum */
+		12000,	/* PowerLimitMaximum */
 		1000,	/* TimeWindowMinimum */
 		1000,	/* TimeWindowMaximum */
 		200	/* StepSize */
diff --git a/src/mainboard/intel/amenia/acpi/dptf.asl b/src/mainboard/intel/amenia/acpi/dptf.asl
index 35bbecb..a75d668 100644
--- a/src/mainboard/intel/amenia/acpi/dptf.asl
+++ b/src/mainboard/intel/amenia/acpi/dptf.asl
@@ -73,7 +73,7 @@ Name (MPPC, Package ()
 	Package () {	/* Power Limit 1 */
 		0,	/* PowerLimitIndex, 0 for Power Limit 1 */
 		1600,	/* PowerLimitMinimum */
-		15000,	/* PowerLimitMaximum */
+		12000,	/* PowerLimitMaximum */
 		1000,	/* TimeWindowMinimum */
 		1000,	/* TimeWindowMaximum */
 		200	/* StepSize */



More information about the coreboot-gerrit mailing list